Disobedience checks server version and expects rights-changed

notifications if server is version 4.1 or better.

diff --git a/disobedience/disobedience.c b/disobedience/disobedience.c
index afcd921d83bf68c8d339860bd4abd131ca18a74f..53aea5683021110bff1af0b3a5d3185122b5eacf 100644 (file)
--- a/disobedience/disobedience.c
+++ b/disobedience/disobedience.c
@@ -96,6 +96,12 @@ int rtp_supported;
 /** @brief True if RTP play is enabled */
 int rtp_is_running;
 
+/** @brief Server version */
+const char *server_version;
+
+/** @brief Parsed server version */
+long server_version_bytes;
+
 static void check_rtp_address(const char *event,
                               void *eventdata,
                               void *callbackdata);
@@ -218,6 +224,16 @@ static void userinfo_rights_completed(void attribute((unused)) *v,
   rights_lookup_in_flight = 0;
 }
 
+static void check_rights(void) {
+  if(!rights_lookup_in_flight) {
+    rights_lookup_in_flight = 1;
+    disorder_eclient_userinfo(client,
+                              userinfo_rights_completed,
+                              config->username, "rights",
+                              0);
+  }
+}
+
 /** @brief Called occasionally */
 static gboolean periodic_slow(gpointer attribute((unused)) data) {
   D(("periodic_slow"));
@@ -259,13 +275,15 @@ static gboolean periodic_fast(gpointer attribute((unused)) data) {
     }
   }
   /* Periodically check what our rights are */
-  if(!rights_lookup_in_flight) {
-    rights_lookup_in_flight = 1;
-    disorder_eclient_userinfo(client,
-                              userinfo_rights_completed,
-                              config->username, "rights",
-                              0);
-  }
+  int recheck_rights = 1;
+  if(server_version_bytes >= 0x04010000)
+    /* Server versions after 4.1 will send updates */
+    recheck_rights = 0;
+  if((server_version_bytes & 0xFF) == 0x01)
+    /* Development servers might do regardless of their version number */
+    recheck_rights = 0;
+  if(recheck_rights)
+    check_rights();
   return TRUE;
 }
 
@@ -359,6 +377,40 @@ static void help(void) {
   exit(0);
 }
 
+static void version_completed(void attribute((unused)) *v,
+                              const char attribute((unused)) *err,
+                              const char *version) {
+  long major, minor, patch, dev;
+
+  if(!version) {
+    server_version = 0;
+    server_version_bytes = 0;
+    return;
+  }
+  server_version = version;
+  server_version_bytes = 0;
+  major = strtol(version, (char **)&version, 10);
+  if(*version != '.')
+    return;
+  ++version;
+  minor = strtol(version, (char **)&version, 10);
+  if(*version == '.') {
+    ++version;
+    patch = strtol(version, (char **)&version, 10);
+  } else
+    patch = 0;
+  if(*version) {
+    if(*version == '+') {
+      dev = 1;
+      ++version;
+    }
+    if(*version)
+      dev = 2;
+  } else
+    dev = 0;
+  server_version_bytes = (major << 24) + (minor << 16) + (patch << 8) + dev;
+}
+
 void logged_in(void) {
   /* reset the clients */
   disorder_eclient_close(client);
@@ -368,6 +420,8 @@ void logged_in(void) {
   /* Force the periodic checks */
   periodic_slow(0);
   periodic_fast(0);
+  /* Recheck server version */
+  disorder_eclient_version(client, version_completed, 0);
 }
 
 int main(int argc, char **argv) {
@@ -422,6 +476,7 @@ int main(int argc, char **argv) {
   disorder_eclient_log(logclient, &log_callbacks, 0);
   /* Initiate all the checks */
   periodic_fast(0);
+  disorder_eclient_version(client, version_completed, 0);
   event_register("log-connected", check_rtp_address, 0);
   suppress_actions = 0;
   /* If no password is set yet pop up a login box */

diff --git a/disobedience/help.c b/disobedience/help.c
index f1822b9b6b623ceb58721cad7a4d7e6b2568b3ca..a1829408d749b01ec1a33d31fc9d4a5c1c185c2c 100644 (file)
--- a/disobedience/help.c
+++ b/disobedience/help.c
@@ -1,3 +1,4 @@
+
 /*
  * This file is part of DisOrder
  * Copyright (C) 2007, 2008 Richard Kettlewell

diff --git a/disobedience/log.c b/disobedience/log.c
index 4dadae73aa8b1f5fa177f95524eb8cfae7457987..a4db0cf73eb743894998b791a20113f297fe94fd 100644 (file)
--- a/disobedience/log.c
+++ b/disobedience/log.c
@@ -42,6 +42,7 @@ static void log_scratched(void *v, const char *track, const char *user);
 static void log_state(void *v, unsigned long state);
 static void log_volume(void *v, int l, int r);
 static void log_rescanned(void *v);
+static void log_rights_changed(void *v, rights_type r);
 
 /** @brief Callbacks for server state monitoring */
 const disorder_eclient_log_callbacks log_callbacks = {
@@ -57,7 +58,8 @@ const disorder_eclient_log_callbacks log_callbacks = {
   .scratched = log_scratched,
   .state = log_state,
   .volume = log_volume,
-  .rescanned = log_rescanned
+  .rescanned = log_rescanned,
+  .rights_changed = log_rights_changed
 };
 
 /** @brief Update everything */
@@ -192,6 +194,15 @@ static void log_rescanned(void attribute((unused)) *v) {
   event_raise("rescan-complete", 0);
 }
 
+/** @brief Called when our rights change */
+static void log_rights_changed(void attribute((unused)) *v,
+                               rights_type new_rights) {
+  ++suppress_actions;
+  last_rights = new_rights;
+  event_raise("rights-changed", 0);
+  --suppress_actions;
+}
+
 /*
 Local Variables:
 c-basic-offset:2