Always log message bar messages to message log

NathanW2 · NathanW2 · commit 391712d2d42a · 2017-07-24T15:19:07.000+10:00
Mainly so they don't just disappear on the user
diff --git a/python/gui/qgsmessagebaritem.sip b/python/gui/qgsmessagebaritem.sip
@@ -41,26 +41,56 @@ make out a widget containing a widget to be displayed on the bar
  :rtype: QgsMessageBarItem
 %End
 
+    QString text() const;
+%Docstring
+ Returns the text for the message.
+ :rtype: str
+%End
+
     QgsMessageBarItem *setTitle( const QString &title );
 %Docstring
  :rtype: QgsMessageBarItem
 %End
 
+    QString title() const;
+%Docstring
+ Returns the title for the message.
+ :rtype: str
+%End
+
     QgsMessageBarItem *setLevel( QgsMessageBar::MessageLevel level );
 %Docstring
  :rtype: QgsMessageBarItem
 %End
 
+    QgsMessageBar::MessageLevel level() const;
+%Docstring
+ Returns the message level for the message.
+ :rtype: QgsMessageBar.MessageLevel
+%End
+
     QgsMessageBarItem *setWidget( QWidget *widget );
 %Docstring
  :rtype: QgsMessageBarItem
 %End
 
+    QWidget *widget() const;
+%Docstring
+ Returns the widget for the message.
+ :rtype: QWidget
+%End
+
     QgsMessageBarItem *setIcon( const QIcon &icon );
 %Docstring
  :rtype: QgsMessageBarItem
 %End
 
+    QIcon icon() const;
+%Docstring
+ Returns the icon for the message.
+ :rtype: QIcon
+%End
+
     QgsMessageBarItem *setDuration( int duration );
 %Docstring
  :rtype: QgsMessageBarItem
diff --git a/src/gui/qgsmessagebar.cpp b/src/gui/qgsmessagebar.cpp
@@ -18,6 +18,7 @@
 #include "qgsmessagebar.h"
 #include "qgsmessagebaritem.h"
 #include "qgsapplication.h"
+#include "qgsmessagelog.h"
 
 #include <QWidget>
 #include <QPalette>
@@ -268,6 +269,27 @@ void QgsMessageBar::pushItem( QgsMessageBarItem *item )
   // avoid duplicated widget
   popWidget( item );
   showItem( item );
+
+  // Log all messages that are sent to the message bar into the message log so the
+  // user can get them back easier.
+  QString formattedTitle = QString( "%1 : %2" ).arg( item->title() ).arg( item->text() );
+  QgsMessageLog::MessageLevel level;
+  switch ( item->level() )
+  {
+    case QgsMessageBar::INFO:
+      level = QgsMessageLog::INFO;
+      break;
+    case QgsMessageBar::WARNING:
+      level = QgsMessageLog::WARNING;
+      break;
+    case QgsMessageBar::CRITICAL:
+      level = QgsMessageLog::CRITICAL;
+      break;
+    default:
+      level = QgsMessageLog::NONE;
+      break;
+  }
+  QgsMessageLog::logMessage( formattedTitle, tr( "Messages" ), level );
 }
 
 QgsMessageBarItem *QgsMessageBar::pushWidget( QWidget *widget, QgsMessageBar::MessageLevel level, int duration )
diff --git a/src/gui/qgsmessagebaritem.cpp b/src/gui/qgsmessagebaritem.cpp
@@ -196,13 +196,23 @@ QgsMessageBarItem *QgsMessageBarItem::setText( const QString &text )
   return this;
 }
 
+QString QgsMessageBarItem::text() const
+{
+  return mText;
+}
+
 QgsMessageBarItem *QgsMessageBarItem::setTitle( const QString &title )
 {
   mTitle = title;
   writeContent();
   return this;
 }
 
+QString QgsMessageBarItem::title() const
+{
+  return mTitle;
+}
+
 QgsMessageBarItem *QgsMessageBarItem::setLevel( QgsMessageBar::MessageLevel level )
 {
   mLevel = level;
@@ -211,6 +221,11 @@ QgsMessageBarItem *QgsMessageBarItem::setLevel( QgsMessageBar::MessageLevel leve
   return this;
 }
 
+QgsMessageBar::MessageLevel QgsMessageBarItem::level() const
+{
+  return mLevel;
+}
+
 QgsMessageBarItem *QgsMessageBarItem::setWidget( QWidget *widget )
 {
   if ( mWidget )
@@ -227,12 +242,22 @@ QgsMessageBarItem *QgsMessageBarItem::setWidget( QWidget *widget )
   return this;
 }
 
+QWidget *QgsMessageBarItem::widget() const
+{
+  return mWidget;
+}
+
 QgsMessageBarItem *QgsMessageBarItem::setIcon( const QIcon &icon )
 {
   mUserIcon = icon;
   return this;
 }
 
+QIcon QgsMessageBarItem::icon() const
+{
+  return mUserIcon;
+}
+
 
 QgsMessageBarItem *QgsMessageBarItem::setDuration( int duration )
 {
diff --git a/src/gui/qgsmessagebaritem.h b/src/gui/qgsmessagebaritem.h
@@ -48,14 +48,39 @@ class GUI_EXPORT QgsMessageBarItem : public QWidget
 
     QgsMessageBarItem *setText( const QString &text );
 
+    /**
+     * Returns the text for the message.
+     */
+    QString text() const;
+
     QgsMessageBarItem *setTitle( const QString &title );
 
+    /**
+     * Returns the title for the message.
+     */
+    QString title() const;
+
     QgsMessageBarItem *setLevel( QgsMessageBar::MessageLevel level );
 
+    /**
+     * Returns the message level for the message.
+     */
+    QgsMessageBar::MessageLevel level() const;
+
     QgsMessageBarItem *setWidget( QWidget *widget );
 
+    /**
+     * Returns the widget for the message.
+     */
+    QWidget *widget() const;
+
     QgsMessageBarItem *setIcon( const QIcon &icon );
 
+    /**
+     * Returns the icon for the message.
+     */
+    QIcon icon() const;
+
     QgsMessageBarItem *setDuration( int duration );
 
     //! returns the duration in second of the message

Original file line number	Diff line number	Diff line change
`@@ -196,13 +196,23 @@ QgsMessageBarItem *QgsMessageBarItem::setText( const QString &text )`
`196`	`196`	`return this;`
`197`	`197`	`}`
`198`	`198`
	`199`	`+QString QgsMessageBarItem::text() const`
	`200`	`+{`
	`201`	`+ return mText;`
	`202`	`+}`
	`203`	`+`
`199`	`204`	`QgsMessageBarItem *QgsMessageBarItem::setTitle( const QString &title )`
`200`	`205`	`{`
`201`	`206`	`mTitle = title;`
`202`	`207`	`writeContent();`
`203`	`208`	`return this;`
`204`	`209`	`}`
`205`	`210`
	`211`	`+QString QgsMessageBarItem::title() const`
	`212`	`+{`
	`213`	`+ return mTitle;`
	`214`	`+}`
	`215`	`+`
`206`	`216`	`QgsMessageBarItem *QgsMessageBarItem::setLevel( QgsMessageBar::MessageLevel level )`
`207`	`217`	`{`
`208`	`218`	`mLevel = level;`
`@@ -211,6 +221,11 @@ QgsMessageBarItem *QgsMessageBarItem::setLevel( QgsMessageBar::MessageLevel leve`
`211`	`221`	`return this;`
`212`	`222`	`}`
`213`	`223`
	`224`	`+QgsMessageBar::MessageLevel QgsMessageBarItem::level() const`
	`225`	`+{`
	`226`	`+ return mLevel;`
	`227`	`+}`
	`228`	`+`
`214`	`229`	`QgsMessageBarItem QgsMessageBarItem::setWidget( QWidget widget )`
`215`	`230`	`{`
`216`	`231`	`if ( mWidget )`
`@@ -227,12 +242,22 @@ QgsMessageBarItem QgsMessageBarItem::setWidget( QWidget widget )`
`227`	`242`	`return this;`
`228`	`243`	`}`
`229`	`244`
	`245`	`+QWidget *QgsMessageBarItem::widget() const`
	`246`	`+{`
	`247`	`+ return mWidget;`
	`248`	`+}`
	`249`	`+`
`230`	`250`	`QgsMessageBarItem *QgsMessageBarItem::setIcon( const QIcon &icon )`
`231`	`251`	`{`
`232`	`252`	`mUserIcon = icon;`
`233`	`253`	`return this;`
`234`	`254`	`}`
`235`	`255`
	`256`	`+QIcon QgsMessageBarItem::icon() const`
	`257`	`+{`
	`258`	`+ return mUserIcon;`
	`259`	`+}`
	`260`	`+`
`236`	`261`
`237`	`262`	`QgsMessageBarItem *QgsMessageBarItem::setDuration( int duration )`
`238`	`263`	`{`