- Allow to change UPnP port while it is enabled

- Allow to change qBT style! (GTK people should like Cleanlooks style)

Changelog

@@ -2,6 +2,7 @@
     - FEATURE: Added UPnP port forwarding support
     - FEATURE: Display more infos about the torrent in its properties
     - FEATURE: Allow the user to edit torrents' trackers
+    - FEATURE: Allow user to change qBT's style (Plastique, Cleanlooks, Motif, CDE, MacOSX, WinXP)
     - COSMETIC: Redesigned torrent properties a little
     - COSMETIC: Redesigned options a little
 

TODO

@@ -28,19 +28,19 @@
 - Option to shutdown computer when downloads are finished
 - Add a torrent scheduler
 
-// in v0.10 (partial)
-- Download from RSS feeds (WIP by gtsoul in RSS_SUPPORT branch)
-- Move finished torrent to another tab and keep on seeding them even after restart
+// in v0.10 (partial) - WIP
+- Download from RSS feeds (WIP by gtsoul in RSS_SUPPORT branch, clean & finish rss.h, merge with trunk, add a tab in mainWindow, debug)
+- Move finished torrent to another tab and keep on seeding them even after restart (better for sharing)
 - Improve torrent creation dialog (look & features) :
     - Add Private combobox (allow to share on DHT or not)
     - Use a QListWidget to allow multiple input paths
     - Possibility to add url seeds
-- Add IPv6 support (at least start working on it)
-- UPnP support (debug, sync with aMule CVS)
-- Allow user to change application style? (WinXP, MacOS, CleanLooks...) : app.setStyle(new QCleanlooksStyle());
+- Add IPv6 support (at least start working on it, libtorrent seems to support it, we should limit our code to IPv4 anymore)
+- UPnP support (debug, sync with next aMule release (or CVS)) : seems to be working!
 - Update v0.9.0 changelog after its release
 - Display Url seeds in torrent properties and allow to edit them
-- Improve Ipfilter.dat parser (move to a thread ?)
+- Improve Ipfilter.dat parser (move to a thread ? - too slow to load qBT and more importantly options)
 - Use tooltips to explain options
-- Exit confirmation only if there are active downloads
-- Support UPnP port edition
+- Exit confirmation only if there are active downloads (display number of downloads) - SMARTER
+- Make use of QNetworkInterface (could be useful ?)
+- Display more info in log (PeX, UPnP, DHT w/ ports...)

src/GUI.cpp

@@ -988,6 +988,7 @@ void GUI::configureSession(bool deleteOptions){
   // Upnp
   if(options->isDHTEnabled()){
     BTSession.enableUPnP(options->getUPnPPort());
+    BTSession.setUPnPPort(options->getUPnPPort());
   }else{
     BTSession.disableUPnP();
   }

src/UPnP.cpp

@@ -979,7 +979,8 @@ m_RootDeviceMap(),
 m_ServiceMap(),
 m_ActivePortMappingsMap(),
 m_IGWDeviceDetected(false),
-m_WanService(NULL)
+m_WanService(NULL),
+m_UPnPPort(udpPort)
 {
 	// Pointer to self
 	s_CtrlPoint = this;

src/UPnP.h

@@ -640,6 +640,7 @@ class CUPnPControlPoint
 	bool m_IGWDeviceDetected;
 	CUPnPService *m_WanService;
 	QMutex m_WaitForSearchTimeout;
+        unsigned short m_UPnPPort;
 
 public:
 	CUPnPControlPoint(unsigned short udpPort);
@@ -650,6 +651,9 @@ class CUPnPControlPoint
 		std::vector<CUPnPPortMapping> &upnpPortMapping);
 	bool DeletePortMappings(
 		std::vector<CUPnPPortMapping> &upnpPortMapping);
+        unsigned short getUPnPPort() const{
+          return m_UPnPPort;
+        }
 
 	UpnpClient_Handle GetUPnPClientHandle()	const
 		{ return m_UPnPClientHandle; }

src/bittorrent.cpp

@@ -90,6 +90,22 @@ void bittorrent::enableUPnP(int port){
   }
 }
 
+// Set UPnP port (>= 1000)
+void bittorrent::setUPnPPort(int upnp_port){
+  if(!UPnPEnabled){
+    qDebug("Cannot set UPnP port because it is disabled");
+    return;
+  }
+  if(m_upnp->getUPnPPort() != upnp_port){
+    qDebug("Changing UPnP port to %d", upnp_port);
+    delete m_upnp;
+    m_upnp = new CUPnPControlPoint(upnp_port);
+    m_upnp->AddPortMappings(m_upnpMappings);
+  }else{
+    qDebug("UPnP: No need to set the port, it is already listening on this port");
+  }
+}
+
 void bittorrent::disableUPnP(){
   if(UPnPEnabled){
     qDebug("Disabling UPnP");

src/bittorrent.h

@@ -121,6 +121,7 @@ class bittorrent : public QObject{
 #ifndef NO_UPNP
     void enableUPnP(int port=50000);
     void disableUPnP();
+    void setUPnPPort(int upnp_port);
 #endif
 
   protected slots: