added load image option in forum posts when images are to be displayed

retroshare-gui/src/gui/common/RSTextBrowser.cpp

@@ -74,27 +74,20 @@ void RSTextBrowser::paintEvent(QPaintEvent *event)
 
 QVariant RSTextBrowser::loadResource(int type, const QUrl &name)
 {
-    std::cerr << "TEXTBROWSER: load ressource request: type=" << type << " scheme=" << name.scheme().toStdString() << ", url=" << name.toString().toStdString() << std::endl;
-                 
     // case 1: always trust the image if it comes from an internal resource
     
     if(name.scheme().compare("qrc",Qt::CaseInsensitive)==0 && type == QTextDocument::ImageResource) 
-    {
-        std::cerr << "loading because it's qrc..." << std::endl;
 	    return QTextBrowser::loadResource(type, name);
-    }
         
-    // case 2: only display if the ser allows it
+    // case 2: only display if the user allows it. Data resources can be bad (svg bombs) but we filter them out globally at the network layer.
+    //         It would be good to add here a home-made resource loader that only loads images and not svg crap, just in case.
     
     if(name.scheme().compare("data",Qt::CaseInsensitive)==0 && mShowImages)
-    {
-        std::cerr << "loading because it's data and mShowImages is true..." << std::endl;
 	    return QTextBrowser::loadResource(type, name);
-    }
         
     // case 3: otherwise, do not display
     
-    std::cerr << "not loading " << std::endl;
+    std::cerr << "TEXTBROWSER: refusing load ressource request: type=" << type << " scheme=" << name.scheme().toStdString() << ", url=" << name.toString().toStdString() << std::endl;
     
     if (mImageBlockWidget) 
 	    mImageBlockWidget->show();

retroshare-gui/src/gui/gxsforums/GxsForumThreadWidget.cpp

@@ -213,6 +213,8 @@ GxsForumThreadWidget::GxsForumThreadWidget(const RsGxsGroupId &forumId, QWidget
     ui->line_2->hide() ;
     ui->by_text_label->hide() ;
     ui->by_label->hide() ;
+    ui->postText->setImageBlockWidget(ui->imageBlockWidget);
+    ui->postText->resetImagesStatus(Settings->getForumLoadEmbeddedImages()) ;
 
     ui->subscribeToolButton->setToolTip(tr("<p>Subscribing to the forum will gather \
                                            available posts from your subscribed friends, and make the \
@@ -590,6 +592,7 @@ void GxsForumThreadWidget::changedThread()
 	if (mFillThread) {
 		return;
 	}
+	ui->postText->resetImagesStatus(Settings->getForumLoadEmbeddedImages()) ;
 
 	insertMessage();
 }

retroshare-gui/src/gui/gxsforums/GxsForumThreadWidget.ui

@@ -6,17 +6,14 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>622</width>
+    <width>851</width>
-    <height>412</height>
+    <height>721</height>
    </rect>
   </property>
   <property name="windowTitle">
    <string>Form</string>
   </property>
-  <layout class="QVBoxLayout" name="verticalLayout">
+  <layout class="QVBoxLayout" name="verticalLayout_3">
-   <property name="margin">
-    <number>0</number>
-   </property>
    <item>
     <widget class="QSplitter" name="threadSplitter">
      <property name="orientation">
@@ -117,6 +114,12 @@
             <property name="sizeType">
              <enum>QSizePolicy::MinimumExpanding</enum>
             </property>
+            <property name="sizeHint" stdset="0">
+             <size>
+              <width>0</width>
+              <height>0</height>
+             </size>
+            </property>
            </spacer>
           </item>
           <item>
@@ -446,6 +449,19 @@
        </item>
       </layout>
      </widget>
+     <widget class="QWidget" name="">
+      <layout class="QVBoxLayout" name="verticalLayout">
+       <item>
+        <widget class="RSImageBlockWidget" name="imageBlockWidget" native="true">
+         <property name="sizePolicy">
+          <sizepolicy hsizetype="Preferred" vsizetype="Maximum">
+           <horstretch>0</horstretch>
+           <verstretch>0</verstretch>
+          </sizepolicy>
+         </property>
+        </widget>
+       </item>
+       <item>
         <widget class="RSTextBrowser" name="postText">
          <property name="sizePolicy">
           <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
@@ -457,6 +473,9 @@
           <enum>Qt::CustomContextMenu</enum>
          </property>
         </widget>
+       </item>
+      </layout>
+     </widget>
     </widget>
    </item>
   </layout>
@@ -502,6 +521,12 @@
    <header location="global">gui/common/ElidedLabel.h</header>
    <container>1</container>
   </customwidget>
+  <customwidget>
+   <class>RSImageBlockWidget</class>
+   <extends>QWidget</extends>
+   <header>gui/common/RSImageBlockWidget.h</header>
+   <container>1</container>
+  </customwidget>
  </customwidgets>
  <resources>
   <include location="../images.qrc"/>