From 516164e78e2434634c410faba35976f378d0ee51 Mon Sep 17 00:00:00 2001
From: Mitchell Livingston <livings124@transmissionbt.com>
Date: Sat, 27 Aug 2011 23:54:10 +0000
Subject: [PATCH] add an RPC session callback for when queue order changes

---
 libtransmission/rpcimpl.c      | 11 ++++++-----
 libtransmission/transmission.h |  1 +
 macosx/Controller.h            |  1 +
 macosx/Controller.m            | 20 ++++++++++++++++++++
 macosx/Torrent.m               |  1 -
 5 files changed, 28 insertions(+), 6 deletions(-)

diff --git a/libtransmission/rpcimpl.c b/libtransmission/rpcimpl.c
index 6f302f652..16ef3f7d9 100644
--- a/libtransmission/rpcimpl.c
+++ b/libtransmission/rpcimpl.c
@@ -187,11 +187,12 @@ getTorrents( tr_session * session,
 }
 
 static void
-notifyBatchChange( tr_session * session, tr_torrent ** torrents, int n )
+notifyBatchQueueChange( tr_session * session, tr_torrent ** torrents, int n )
 {
     int i;
     for( i=0; i<n; ++i )
         notify( session, TR_RPC_TORRENT_CHANGED, torrents[i] );
+    notify( session, TR_RPC_SESSION_QUEUE_POSITIONS_CHANGED, NULL );
 }
 
 static const char*
@@ -203,7 +204,7 @@ queueMoveTop( tr_session               * session,
     int n;
     tr_torrent ** torrents = getTorrents( session, args_in, &n );
     tr_torrentsQueueMoveTop( torrents, n );
-    notifyBatchChange( session, torrents, n );
+    notifyBatchQueueChange( session, torrents, n );
     tr_free( torrents );
     return NULL;
 }
@@ -217,7 +218,7 @@ queueMoveUp( tr_session               * session,
     int n;
     tr_torrent ** torrents = getTorrents( session, args_in, &n );
     tr_torrentsQueueMoveUp( torrents, n );
-    notifyBatchChange( session, torrents, n );
+    notifyBatchQueueChange( session, torrents, n );
     tr_free( torrents );
     return NULL;
 }
@@ -231,7 +232,7 @@ queueMoveDown( tr_session               * session,
     int n;
     tr_torrent ** torrents = getTorrents( session, args_in, &n );
     tr_torrentsQueueMoveDown( torrents, n );
-    notifyBatchChange( session, torrents, n );
+    notifyBatchQueueChange( session, torrents, n );
     tr_free( torrents );
     return NULL;
 }
@@ -245,7 +246,7 @@ queueMoveBottom( tr_session               * session,
     int n;
     tr_torrent ** torrents = getTorrents( session, args_in, &n );
     tr_torrentsQueueMoveBottom( torrents, n );
-    notifyBatchChange( session, torrents, n );
+    notifyBatchQueueChange( session, torrents, n );
     tr_free( torrents );
     return NULL;
 }
diff --git a/libtransmission/transmission.h b/libtransmission/transmission.h
index 6e764d928..4ccfcc177 100644
--- a/libtransmission/transmission.h
+++ b/libtransmission/transmission.h
@@ -517,6 +517,7 @@ typedef enum
     TR_RPC_TORRENT_CHANGED, /* catch-all for the "torrent-set" rpc method */
     TR_RPC_TORRENT_MOVED,
     TR_RPC_SESSION_CHANGED,
+    TR_RPC_SESSION_QUEUE_POSITIONS_CHANGED, /* catch potentially multiple torrents being moved in the queue */
     TR_RPC_SESSION_CLOSE
 }
 tr_rpc_callback_type;
diff --git a/macosx/Controller.h b/macosx/Controller.h
index 5a7770b9a..b9f762a35 100644
--- a/macosx/Controller.h
+++ b/macosx/Controller.h
@@ -261,5 +261,6 @@ typedef enum
 - (void) rpcStartedStoppedTorrent: (Torrent *) torrent;
 - (void) rpcChangedTorrent: (Torrent *) torrent;
 - (void) rpcMovedTorrent: (Torrent *) torrent;
+- (void) rpcUpdateQueue;
 
 @end
diff --git a/macosx/Controller.m b/macosx/Controller.m
index e054e6e4a..bafb81eb3 100644
--- a/macosx/Controller.m
+++ b/macosx/Controller.m
@@ -4143,6 +4143,10 @@ static void sleepCallback(void * controller, io_service_t y, natural_t messageTy
             [self performSelectorOnMainThread: @selector(rpcMovedTorrent:) withObject: torrent waitUntilDone: NO];
             break;
         
+        case TR_RPC_SESSION_QUEUE_POSITIONS_CHANGED:
+            [self performSelectorOnMainThread: @selector(rpcUpdateQueue) withObject: nil waitUntilDone: NO];
+            break;
+        
         case TR_RPC_SESSION_CHANGED:
             [fPrefsController performSelectorOnMainThread: @selector(rpcUpdatePrefs) withObject: nil waitUntilDone: NO];
             break;
@@ -4224,4 +4228,20 @@ static void sleepCallback(void * controller, io_service_t y, natural_t messageTy
     [torrent release];
 }
 
+- (void) rpcUpdateQueue
+{
+    NSLog(@"rpcUpdateQueue");
+    
+    for (Torrent * torrent in fTorrents)
+        [torrent update];
+    
+    NSSortDescriptor * descriptor = [[NSSortDescriptor alloc] initWithKey: @"queuePosition" ascending: YES];
+    NSArray * descriptors = [NSArray arrayWithObject: descriptor];
+    [descriptor release];
+    
+    [fTorrents sortUsingDescriptors: descriptors];
+    
+    [self fullUpdateUI];
+}
+
 @end
diff --git a/macosx/Torrent.m b/macosx/Torrent.m
index 5eb339238..dbad0a161 100644
--- a/macosx/Torrent.m
+++ b/macosx/Torrent.m
@@ -346,7 +346,6 @@ int trashDataFile(const char * filename)
     }
 }
 
-#warning remove
 - (NSInteger) queuePosition
 {
     return fStat->queuePosition;