Improved arguments parsing in external download manager.

Option to choose where exactly to put url (%d placeholder)
instead of just appending it at the end of arguments.

src/lib/downloads/downloadmanager.cpp

@@ -75,6 +75,10 @@ void DownloadManager::loadSettings()
     m_externalExecutable = settings.value("ExternalManagerExecutable", QString()).toString();
     m_externalArguments = settings.value("ExternalManagerArguments", QString()).toString();
     settings.endGroup();
+
+    if (!m_externalArguments.contains(QLatin1String("%d"))) {
+        m_externalArguments.append(QLatin1String(" %d"));
+    }
 }
 
 void DownloadManager::show()
@@ -101,7 +105,10 @@ void DownloadManager::keyPressEvent(QKeyEvent* e)
 
 void DownloadManager::startExternalManager(const QUrl &url)
 {
-    QzTools::startExternalProcess(m_externalExecutable, m_externalArguments + url.toEncoded());
+    QString arguments = m_externalArguments;
+    arguments.replace(QLatin1String("%d"), url.toEncoded());
+
+    QzTools::startExternalProcess(m_externalExecutable, arguments);
     m_lastDownloadOption = ExternalManager;
 }
 

src/lib/preferences/preferences.ui

@@ -2014,6 +2014,13 @@
            </item>
           </layout>
          </item>
+         <item row="2" column="1">
+          <widget class="QLabel" name="label_64">
+           <property name="text">
+            <string>&lt;b&gt;%d&lt;/b&gt; will be replaced with URL to be downloaded</string>
+           </property>
+          </widget>
+         </item>
         </layout>
        </item>
        <item row="9" column="0" colspan="4">

translations/empty.ts

@@ -2812,6 +2812,10 @@
         <source>Choose cache path...</source>
         <translation type="unfinished"></translation>
     </message>
+    <message>
+        <source>&lt;b&gt;%d&lt;/b&gt; will be replaced with URL to be downloaded</source>
+        <translation type="unfinished"></translation>
+    </message>
 </context>
 <context>
     <name>QObject</name>