commit 483f968fef7423e7120573d3ae2bf3907ac4bb1b
author Olivier SOLDANO <olivier.soldano@savoirfairelinux.com> Wed Jul 26 11:57:49 2017 -0400
committer Andreas Traczyk <andreas.traczyk@savoirfairelinux.com> Thu Jul 27 16:51:41 2017 -0400
tree 11a2b3b8ac1fa98eb48bd0ab0978ebcb624f3c9c
parent 5cb1a42891c549cc5636838f975573649ee74d6e

remove double Ids

If an id was displayed in double (if bestName == bestId)
the bestId is now hidden.

Change-Id: I562d7b1812cda1594fca01cb5bf0d413f4cb478f
Reviewed-by: Andreas Traczyk <andreas.traczyk@savoirfairelinux.com>

smartlistdelegate.cpp

diff --git a/smartlistdelegate.cpp b/smartlistdelegate.cpp
index ac38147..4c164e0 100644
--- a/smartlistdelegate.cpp
+++ b/smartlistdelegate.cpp
@@ -149,22 +149,24 @@
 
         // Display the ID under the name
         QString idStr = index.data(static_cast<int>(Ring::Role::Number)).value<QString>();
-        pen.setColor(RingTheme::grey_);
-        painter->setPen(pen);
-        font.setItalic(true);
-        font.setBold(false);
-        painter->setFont(font);
-        QFontMetrics fontMetrics(font);
-        if (!idStr.isNull()){
-            idStr = fontMetrics.elidedText(idStr, Qt::ElideRight, rectTexts.width()- sizeImage_ - effectiveComBarSize_ - dx_);
-            painter->drawText(QRect(16 + rect.left() + dx_ + sizeImage_,
-                                    rect.top() + rect.height()/7,
-                                    rect.width(),
-                                    rect.height()/2),
-                              Qt::AlignBottom | Qt::AlignLeft, idStr);
+        if (idStr != name.toString()){
+            pen.setColor(RingTheme::grey_);
+            painter->setPen(pen);
+            font.setItalic(true);
+            font.setBold(false);
+            painter->setFont(font);
+            QFontMetrics fontMetrics(font);
+            if (!idStr.isNull()){
+                idStr = fontMetrics.elidedText(idStr, Qt::ElideRight, rectTexts.width()- sizeImage_ - effectiveComBarSize_ - dx_);
+                painter->drawText(QRect(16 + rect.left() + dx_ + sizeImage_,
+                                        rect.top() + rect.height()/7,
+                                        rect.width(),
+                                        rect.height()/2),
+                                  Qt::AlignBottom | Qt::AlignLeft, idStr);
 
-        } else {
-            qDebug() << "This is not a Person";
+            } else {
+                qDebug() << "This is not a Person";
+            }
         }
 
         // Finally, either last interaction date or call state is displayed