Review Request: More user friendly error handling if a requested help docbook was not found

[David Faure]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5272/#review7438
-----------------------------------------------------------


Looks good, except for one issue.


trunk/KDE/kdelibs/kdoctools/kio_help.cpp
<http://svn.reviewboard.kde.org/r/5272/#comment7645>

    The result of this call is overwritten by the call on line 140. Sounds like a if(empty) is missing?


- David


On 2010-09-06 17:33:36, Burkhard Lück wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5272/
> -----------------------------------------------------------
> 
> (Updated 2010-09-06 17:33:36)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> In case a requested help documentation was not found for whatever reason (not installed, no documentation, wrong/missing X-DocPath etc.) KHelpcenter/Konqueror display the error message "There is no documentation available for %1." and additionally a modal dialog with the error message "The file or folder help://%1 does not exist" pops up.
> With this patch kio_help tries to load a docbook called "documentationnotfound" which will be installed with the khelpcenter documentation from kdebase/runtime. In this docbook the user will find information how to solve this issue and links to kde documentation online (docs.kde.org/userbase/forum).
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdelibs/kdoctools/kio_help.cpp 1170669 
> 
> Diff: http://svn.reviewboard.kde.org/r/5272/diff
> 
> 
> Testing
> -------
> 
> With this patch and no docbook "documentationnotfound" installed no change to the current situation, with an installed docbook "documentationnotfound" this is displayed as expected.
> 
> 
> Thanks,
> 
> Burkhard
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100906/61c71e13/attachment.htm>