Bug 326722

Summary:	Error Text and Title Swapped in Error
Product:	[Applications] kalzium	Reporter:	cmegh1969
Component:	general	Assignee:	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:	http://commits.kde.org/kalzium/23e075bb42ef47de4400a6820f6b058653fab52c	Version Fixed In:	4.11.3
Sentry Crash Report:
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