Bug 326722

Summary: Error Text and Title Swapped in Error
Product: [Applications] kalzium Reporter: cmegh1969
Component: generalAssignee: Kalzium Developers <kalzium>
Status: RESOLVED FIXED    
Severity: normal CC: vinc.pii
Priority: NOR Keywords: junior-jobs
Version: 2.4.0   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In: 4.11.3
Attachments: Shows how the Error is Produced.
Title and text swapped
Patch for the issue, simply swap caption and text.

Description cmegh1969 2013-10-27 08:14:44 UTC
I was Exploring the Molecular Editor and Changed Some Options, When Kalzium Gave me an Error. The Error Body was at the App Name Area and Kalzium was in the Error Body Area.

Reproducible: Always

Steps to Reproduce:
1. Open Kalzium,
2. Click 'Molecular Editor',
3. Change Second Style to 'Ring'.
4. Switch To the 'Edit' tab.
5. Click 'Optimize'. 
Actual Results:  
It shown 'Could not...' in the Head and 'Kalzium' in the Body.

Expected Results:  
It should have shown 'Could Not...' in the Body and 'Kalzium' in the Head.
Comment 1 cmegh1969 2013-10-27 08:19:35 UTC
Created attachment 83146 [details]
Shows how the Error is Produced.
Comment 2 Vincenzo 2013-10-29 21:05:06 UTC
Patch submitted to reviewboard: https://git.reviewboard.kde.org/r/113502/
Comment 3 Vincenzo 2013-10-29 21:07:25 UTC
Created attachment 83211 [details]
Title and text swapped
Comment 4 Vincenzo 2013-10-29 21:11:03 UTC
Created attachment 83212 [details]
Patch for the issue, simply swap caption and text.
Comment 5 Jeremy Whiting 2013-10-30 15:47:51 UTC
Git commit 13f734f28fa6b889e368f037a7e183753a43a4a7 by Jeremy Whiting.
Committed on 30/10/2013 at 15:42.
Pushed by whiting into branch 'master'.

Fix KMessageBox usage.
REVIEW:113502
FIXED-IN:4.11.3

M  +3    -2    src/tools/moleculeview.cpp

http://commits.kde.org/kalzium/13f734f28fa6b889e368f037a7e183753a43a4a7
Comment 6 Jeremy Whiting 2013-10-30 15:48:35 UTC
Git commit 23e075bb42ef47de4400a6820f6b058653fab52c by Jeremy Whiting.
Committed on 30/10/2013 at 15:42.
Pushed by whiting into branch 'KDE/4.11'.

Fix KMessageBox usage.
REVIEW:113502
FIXED-IN:4.11.3

(cherry picked from commit 13f734f28fa6b889e368f037a7e183753a43a4a7)

M  +3    -2    src/tools/moleculeview.cpp

http://commits.kde.org/kalzium/23e075bb42ef47de4400a6820f6b058653fab52c