----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110687/#review33285 -----------------------------------------------------------
drkonqi/reportassistantdialog.cpp <http://git.reviewboard.kde.org/r/110687/#comment24626> There should be a space after the comma. drkonqi/reportassistantpages_bugzilla_versioncheck.h <http://git.reviewboard.kde.org/r/110687/#comment24625> Wrong copyright date. drkonqi/reportassistantpages_bugzilla_versioncheck.h <http://git.reviewboard.kde.org/r/110687/#comment24616> This should be "State" not "Stage". drkonqi/reportassistantpages_bugzilla_versioncheck.h <http://git.reviewboard.kde.org/r/110687/#comment24618> Why add a commented method? This should be removed. drkonqi/reportassistantpages_bugzilla_versioncheck.h <http://git.reviewboard.kde.org/r/110687/#comment24617> "m_state" instead of "m_stage" drkonqi/reportassistantpages_bugzilla_versioncheck.cpp <http://git.reviewboard.kde.org/r/110687/#comment24620> Shouldn't this be a translatable string? drkonqi/reportassistantpages_bugzilla_versioncheck.cpp <http://git.reviewboard.kde.org/r/110687/#comment24621> Translatable string? drkonqi/reportassistantpages_bugzilla_versioncheck.cpp <http://git.reviewboard.kde.org/r/110687/#comment24622> Translatable string? drkonqi/reportinterface.h <http://git.reviewboard.kde.org/r/110687/#comment24623> Code style, there should be just one space between the "*" and the pointer name. Actually I think the current code style says there should be no space between the "*" and the pointer name. I am not sure if the maintainer of this code is ok with mixing code style until everything is converted to the new one. drkonqi/reportinterface.cpp <http://git.reviewboard.kde.org/r/110687/#comment24624> same here. - Lamarque Souza On May 28, 2013, 11:01 a.m., Jekyll Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110687/ > ----------------------------------------------------------- > > (Updated May 28, 2013, 11:01 a.m.) > > > Review request for KDE Runtime and George Kiagiadakis. > > > 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. > > > > This addresses bug 315073. > http://bugs.kde.org/show_bug.cgi?id=315073 > > > 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 > > > Thanks, > > Jekyll Wu > >
