[Kde-pim] Review Request 119461: Per attendee customization of email [3] - Add paramenter to ensure the composer to open

[Kevin Krammer]

> On Juli 25, 2014, 12:35 nachm., Kevin Krammer wrote:
> > I don't quite understand the description.
> > isn't that what the "hidden" argument is for?
> 
> Sandro Knauß wrote:
>     Nope the hidden argument is the opposit:
>     hidden == true -> the compose won't be open (automatic sending)
>     hidden == false -> the default value is used ( either MessageViewer::GlobalSettings::self()->automaticSending() or esspecially for itip invations will be always handled automatically) 
>     allowDefaultSend == true (like before)
>     allowDefaultSend == false  && hidden == false -> the composer window is open independently from MessageViewer::GlobalSettings::self()->automaticSending()

Hmm, I am guessing here, but maybe this weird handling of "hidden" is a left over from times when applications could not send messages themselves but had to go through KMail?
I mean, if I call a method called openComposer, explicitly tell it not to be hidden, then I expect the message composer window to show. Anything else is not what the API says it does.
What is the use case of hidden == false if it is ignored?


- Kevin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119461/#review63139
-----------------------------------------------------------


On Juli 25, 2014, 12:22 nachm., Sandro Knauß wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119461/
> -----------------------------------------------------------
> 
> (Updated Juli 25, 2014, 12:22 nachm.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> The actual openComposer method dosen't open an dialog if it is a itip invitation or the user wants automatic sending. 
> But for actually get individual mails for event attendees, we must to make sure that the composer is openend.
> + small bugfix setParameter( QLatin1String(attachParamAttr), attachParamValue ) has to be set to contentType and not to contentDisposition
> 
> 
> Diffs
> -----
> 
>   kmail/kmkernel.h bdad69fadfe1fdd0a7b3734b74673c34d5208977 
>   kmail/kmkernel.cpp f4cb34120c7273158d544a1ee3fa23a02ed815c3 
> 
> Diff: https://git.reviewboard.kde.org/r/119461/diff/
> 
> 
> Testing
> -------
> 
> sent some invations around to make sure they are corret.
> 
> 
> Thanks,
> 
> Sandro Knauß
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/