Ref T529, added "IndexSeverityAsIcon", use logic sort order

src/blackgui/models/statusmessagelistmodel.cpp

@@ -86,7 +86,7 @@ namespace BlackGui
             case Detailed:
                 {
                     m_columns.addColumn(CColumn("time", CStatusMessage::IndexUtcTimestamp, new CDateTimeFormatter(CDateTimeFormatter::formatHmsz())));
-                    CColumn col = CColumn("severity", CStatusMessage::IndexIcon);
+                    CColumn col = CColumn("severity", CStatusMessage::IndexSeverityAsIcon);
                     col.setSortPropertyIndex(CStatusMessage::IndexSeverityAsString);
                     m_columns.addColumn(col);
                     m_columns.addColumn(CColumn::standardString("message", CStatusMessage::IndexMessage));
@@ -99,7 +99,7 @@ namespace BlackGui
             case Simplified:
                 {
                     m_columns.addColumn(CColumn("time", CStatusMessage::IndexUtcTimestamp, new CDateTimeFormatter(CDateTimeFormatter::formatHmsz())));
-                    CColumn col = CColumn("severity", CStatusMessage::IndexIcon);
+                    CColumn col = CColumn("severity", CStatusMessage::IndexSeverityAsIcon);
                     col.setSortPropertyIndex(CStatusMessage::IndexSeverityAsString);
                     m_columns.addColumn(col);
                     m_columns.addColumn(CColumn::standardString("message", CStatusMessage::IndexMessage));

src/blackmisc/statusmessage.cpp

@@ -413,6 +413,11 @@ namespace BlackMisc
         return severityToString(m_severity);
     }
 
+    const CIcon &CStatusMessage::getSeverityAsIcon() const
+    {
+        return convertToIcon(m_severity);
+    }
+
     const QStringList &CStatusMessage::allSeverityStrings()
     {
         static const QStringList all { severityToString(SeverityDebug), severityToString(SeverityInfo), severityToString(SeverityWarning), severityToString(SeverityError) };
@@ -430,6 +435,7 @@ namespace BlackMisc
         case IndexMessage: return CVariant::from(this->getMessage());
         case IndexSeverity: return CVariant::from(m_severity);
         case IndexSeverityAsString: return CVariant::from(this->getSeverityAsString());
+        case IndexSeverityAsIcon: return CVariant::from(this->getSeverityAsIcon());
         case IndexCategoriesAsString: return CVariant::from(m_categories.toQString());
         case IndexCategoriesHumanReadableAsString: return CVariant::from(this->getHumanReadablePattern());
         case IndexCategoryHumanReadableOrTechnicalAsString: return CVariant::from(this->getHumanOrTechnicalCategoriesAsString());
@@ -467,8 +473,9 @@ namespace BlackMisc
         case IndexMessageAsHtml:
         case IndexMessage:
             return this->getMessage().compare(compareValue.getMessage());
+        case IndexSeverityAsString:
+        case IndexSeverityAsIcon:
         case IndexSeverity: return Compare::compare(this->getSeverity(), compareValue.getSeverity());
-        case IndexSeverityAsString: return this->getSeverityAsString().compare(compareValue.getSeverityAsString());
         case IndexCategoriesAsString: return this->getCategoriesAsString().compare(compareValue.getCategoriesAsString());
         case IndexCategoriesHumanReadableAsString: return this->getHumanReadablePattern().compare(compareValue.getHumanReadablePattern());
         case IndexCategoryHumanReadableOrTechnicalAsString: return this->getHumanOrTechnicalCategoriesAsString().compare(compareValue.getHumanOrTechnicalCategoriesAsString());

src/blackmisc/statusmessage.h

@@ -315,6 +315,7 @@ namespace BlackMisc
             IndexCategoryHumanReadableOrTechnicalAsString,
             IndexSeverity,
             IndexSeverityAsString,
+            IndexSeverityAsIcon,
             IndexMessage,
             IndexMessageNoLineBreaks,
             IndexMessageAsHtml
@@ -464,11 +465,14 @@ namespace BlackMisc
         void setCategories(const CLogCategoryList &categories) { this->m_categories = categories; }
 
         //! Representing icon
-        CIcon toIcon() const { return convertToIcon(*this); }
+        const CIcon &toIcon() const { return convertToIcon(*this); }
 
         //! Severity as string
         const QString &getSeverityAsString() const;
 
+        //! Severity as icon
+        const CIcon &getSeverityAsIcon() const;
+
         //! Severity as string
         static const QString &severityToString(StatusSeverity severity);