Reorder Channel::new args

src/padfoot/channel.rs

@@ -31,9 +31,10 @@ pub const HANDLE_TYPE_GROUP: HandleType = 4; // Deprecated
 // FIXME: I'm assuming that all channels will be of type text and 1-1 for now.
 #[derive(Debug)]
 pub struct Channel {
-    ctx: Arc<RwLock<dc::context::Context>>,
     actq: mpsc::Sender<DbusAction>,
+    ctx: Arc<RwLock<dc::context::Context>>,
     path: dbus::Path<'static>,
+    requested: bool,
     target_handle: u32, // Who we're talking to
 }
 
@@ -44,16 +45,18 @@ pub fn channel_interfaces() -> Vec<String> {
 
 impl Channel {
     pub fn new(
+        actq: mpsc::Sender<DbusAction>,
         ctx: Arc<RwLock<dc::context::Context>>,
         path: dbus::Path<'static>,
+        requested: bool,
         target_handle: u32,
-        actq: mpsc::Sender<DbusAction>,
     ) -> Self {
         Channel {
+            actq,
             ctx,
             path,
+            requested,
             target_handle,
-            actq,
         }
     }
 

src/padfoot/channel/channel.rs

@@ -77,7 +77,7 @@ impl telepathy::Channel for Channel {
     fn requested(&self) -> Result<bool> {
         println!("Channel::requested()");
 
-        Ok(false) // FIXME: channels initiated by ourselves *will* be requested
+        Ok(true) // FIXME: channels initiated by ourselves *will* be requested
     }
 
     fn initiator_handle(&self) -> Result<u32> {

src/padfoot/connection.rs

@@ -403,7 +403,13 @@ impl Connection {
                             }
 
                             let handle = contacts.first().unwrap();
-                            let chan = Channel::new(ctx.clone(), chan_path, *handle, actq.clone());
+                            let chan = Channel::new(
+                                actq.clone(),
+                                ctx.clone(),
+                                chan_path,
+                                false, // FIXME: this needs to handle requested channels
+                                *handle,
+                            );
                             actq.send(DbusAction::NewChannel(chan)).unwrap();
                             actq.send(act).unwrap();