Review Request 110687: DrKonqi should check for disabled version as the very first step in the reporting assistant.

[Christoph Feck]

> On May 28, 2013, 11:06 a.m., Martin Gräßlin wrote:
> > Could you please get some feedback from packagers. I'm not sure whether they like words like "unmaintained" and "upgrade". The fact that we as upstream don't accept bugs doesn't mean it's unmaintained by the distro and it's not said that one could upgrade (think of Debian Stable).

Jekyll, has this been discussed on the packagers list?


- Christoph


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110687/#review33280
-----------------------------------------------------------


On June 5, 2013, 10:05 a.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110687/
> -----------------------------------------------------------
> 
> (Updated June 5, 2013, 10:05 a.m.)
> 
> 
> Review request for KDE Runtime and George Kiagiadakis.
> 
> 
> Bugs: 315073
>     http://bugs.kde.org/show_bug.cgi?id=315073
> 
> 
> Repository: kde-runtime
> 
> 
> Description
> -------
> 
> As I have said in https://bugs.kde.org/show_bug.cgi?id=315073#c3 ,  Bugzilla's new and nice behavior (since 4.2.5) of rejecting reports against disabled versions brings a new usability problem to DrKonqi: users spend value time in downloading debug symbols, generating the backtrace, writing all information he/she can recall, but in the end only to find an error dialog telling them (in a not so clear and friendly way) that bugs.kde.org won't accept his/her report. 
> 
> I would propose making version checking the very first step in the reporting assistant: a perfect bug report against an outdated version is still useless. 
> 
> The patch has been created for sometime and works, but unfortunately I don't have much time for coding since then, so it is not as good as what I have planned to make it to be. Nevertheless, I think it is still a good improvement of the current situation.
>  
> 
> 
> Diffs
> -----
> 
>   drkonqi/CMakeLists.txt 39833d7 
>   drkonqi/drkonqi_globals.h d5f098a 
>   drkonqi/productmapping.h f926c9d 
>   drkonqi/productmapping.cpp f4e59e5 
>   drkonqi/reportassistantdialog.cpp c296059 
>   drkonqi/reportassistantpages_bugzilla_versioncheck.h PRE-CREATION 
>   drkonqi/reportassistantpages_bugzilla_versioncheck.cpp PRE-CREATION 
>   drkonqi/reportinterface.h c7e374e 
>   drkonqi/reportinterface.cpp 4190c40 
>   drkonqi/ui/assistantpage_versioncheck.ui PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/110687/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> rejecting disabled version
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/28/drkonqi-version-checking.png
> reject disabled versions (revised edition)
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/30/bugzilla-version-cheking-improved.png
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

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