Review Request 122921: Handle job error when showing job finished notification

[David Rosca]

> On March 22, 2015, 11:36 a.m., Kai Uwe Broulik wrote:
> > applets/notifications/package/contents/ui/Jobs.qml, line 69
> > <https://git.reviewboard.kde.org/r/122921/diff/1/?file=354553#file354553line69>
> >
> >     Just noticed, shouldn't it rather be "Failed"?
> >     
> >     So you get "Copying: Failed" rather than "Error" because the job aborted?

"Failed" would probably be better, I wasn't really sure which word to choose.


- David


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


On March 12, 2015, 5:46 p.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122921/
> -----------------------------------------------------------
> 
> (Updated March 12, 2015, 5:46 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> When job finishes, the notifications applet shows a "%job_message%: Finished" notification with action to open url (extracted from description).
> If the job finishes with error, this notification still displays "Finished" and offers to open url, but as the job finished with error, this url will
> most probably be invalid. There is also no indication that the job actually finished with error.
> 
> This patch checks for job error and in case of error, set notification summary to "%job_message%: Error", body to error text and clears the actions (no open url).
> 
> It only works with jobs that actually sets the error text (KJob::setErrorText), because KJob::error is not exposed in dataengine.
> 
> 
> Diffs
> -----
> 
>   applets/notifications/package/contents/ui/Jobs.qml ee3ec5b 
> 
> Diff: https://git.reviewboard.kde.org/r/122921/diff/
> 
> 
> Testing
> -------
> 
> Error in job is now detected, displayed in notification and there is no "open url" action.
> 
> 
> File Attachments
> ----------------
> 
> error-notification
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/03/12/a7f43698-8c8d-4ad3-aa03-5b527fcdc2d9__job-notification.png
> 
> 
> Thanks,
> 
> David Rosca
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150322/61465a43/attachment.html>