Bug 675411

Summary:	Kontact crashes when modifying left sidebars preferences (change to icons only etc.)
Product:	[Fedora] Fedora	Reporter:	Piotrek Juźwiak <piotrek.juzwiak>
Component:	kdepim	Assignee:	Than Ngo <than>
Status:	CLOSED DUPLICATE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	14	CC:	jreznik, kevin, ltinkl, piotrek.juzwiak, rdieter, rnovacek, ry, smparrish, than, thomasj
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2011-02-05 14:17:23 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Piotrek Juźwiak 2011-02-05 14:13:36 UTC

Description of problem:
Right click anywhere in the left sidebar to change the look.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Piotrek Juźwiak 2011-02-05 14:14:41 UTC

Sorry i am not adding any details but the upstream bug report has everything that is needed.

Comment 2 Piotrek Juźwiak 2011-02-05 14:16:38 UTC

Comment #57 contains the solution?

"


------- Comment #57 From Till Adam 2011-01-17 12:59:54 (-) [reply] -------
commit f142cd203cecfc7195b5e5e115c4a66093454e16
branch 4.4
Author: Till Adam <adam>
Date:   Sun Nov 28 19:59:54 2010 +0000

    Backport commit eb37385 from master to 4.4:
    Don't crash when changing the icon or text size or visibility.
    CCBUG: 250138
    FIXED-IN: 4.4.10

diff --git a/kontact/src/iconsidepane.cpp b/kontact/src/iconsidepane.cpp
index 1b71290..c83b95b 100644
--- a/kontact/src/iconsidepane.cpp
+++ b/kontact/src/iconsidepane.cpp
@@ -90,9 +90,9 @@ class Model : public QStringListModel
     {
     }

-    void emitLayoutChanged()
+    void emitReset()
     {
-        emit layoutChanged();
+        emit reset();
     }

     void setPluginList( const QList<KontactInterface::Plugin*> &list ) {
@@ -500,7 +500,7 @@ void Navigator::slotActionTriggered( bool checked )
   Prefs::self()->setSidePaneShowIcons( mShowIcons );
   Prefs::self()->setSidePaneShowText( mShowText );

-  mModel->emitLayoutChanged();
+  mModel->emitReset();

   QTimer::singleShot( 0, this, SLOT(updateNavigatorSize()) );
 }

"

Comment 3 Lukáš Tinkl 2011-02-07 10:48:33 UTC


*** This bug has been marked as a duplicate of bug 675414 ***