Review Request 108541: KBugReport should open enter_bug.cgi directly instead of wizard.cgi on bugs.kde.org

[David Faure]

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

Ship it!


If some code from drkonqi is needed in kbugreport, feel free to copy it, make it a public method in kbugreport, and call that from drkonqi once it requires the kdelibs version that has the code (thus getting rid of the code duplication at that point).


kdeui/dialogs/kbugreport.cpp
<http://git.reviewboard.kde.org/r/108541/#comment19978>

    optinal -> optional


- David Faure


On Jan. 22, 2013, 11:09 a.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108541/
> -----------------------------------------------------------
> 
> (Updated Jan. 22, 2013, 11:09 a.m.)
> 
> 
> Review request for kdelibs and Ben Cooksley.
> 
> 
> Description
> -------
> 
> Currently kbugreport just can't fill platform information. There are two reasons:
> 
> 1. kbugreport currently pass the results of uname, while bugs.kde.org expects a few
> predefined values like "Gentoo packages", "Chakra", "Ubuntu packages", etc.
> 
> 2. even if kbugreport passes those expected values, wizard.cgi in its current form doesn't correctly redirect those info to enter_bugs.cgi, since the latter expects and understands "sys_os" and "rep_platform" fields, instead of the "additional_info" field.
> 
>     According to this commit[1], the home-made wizard.cgi exists solely for the sake of compatibility. So I wouldn't expect any one has the motivation to improve wizard.cgi.  
> 
> 
> This patch makes kbugreport interact with enter_bug.cgi directly.  It doesn't really improve anything, but it comes as preparation for further work.  There is no functionality lost, since the feature of filling platform information has been broken for a long time.
> 
> Side note: As written in the TODO, I would like to make kbugreport correctly fill OS and Platform information. The plan is stupid, just reuse/copy the systeminformation.cpp[2] from drkonqi, which does a good job in guessing OS and Platform for drkonqi. But I don't know whether such code duplication is allowed or will interfere with the frameworks .
> 
> [1] http://commits.kde.org/websites/bugs-kde-org/491ded267e43faa79723fd31a9b64dd266a6f932
> [2] https://projects.kde.org/projects/kde/kde-runtime/repository/revisions/master/entry/drkonqi/systeminformation.cpp
> 
> 
> This addresses bug 291346.
>     http://bugs.kde.org/show_bug.cgi?id=291346
> 
> 
> Diffs
> -----
> 
>   kdeui/dialogs/kbugreport.cpp 2e72828 
> 
> Diff: http://git.reviewboard.kde.org/r/108541/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

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