Thepatrick CheetahWatch

Comments and changes to this ticket

• 

  Nick Hibma February 18th, 2010 @ 07:53 PM

  Not pretending that I understand the code, but would this do? ... Hm. More testing shows that calling haveModem (instead of haveNoModem; leaving out the haveNoModem function completely) is sufficient. To be honest I am pretty clueless (still) when it comes to UI code on the Mac.

  I guess that it is sufficient to trigger an update of the UI when leaving mainController.

  diff --git a/Classes/CWMain.m b/Classes/CWMain.m
  index d365378..63dd6d5 100644
  --- a/Classes/CWMain.m
  +++ b/Classes/CWMain.m
  @@ -302,6 +302,18 @@ void calloutProc (SCNetworkConnectionRef connection, SCNetworkConnectionStatus s
          [connectedInStatus setStringValue:@"Not connected"];
   }
   
  +// called by the monitor thread to say "Um... No."
  +-(void)haveNoModem
  +{
  +//     NSImage *imageFromBundle = [NSImage imageNamed:@"have-modem.png"];
  +       [statusItem setTitle:@"?"]; 
  +       //[status setImage: imageFromBundle];
  +       [self clearAllUI];
  +       [self performSelectorOnMainThread:@selector(haveModemMain:) withObject:nil waitUntilDone:YES];
  +       [statusItemConectedFor setTitle:@"No modem"];
  +       [connectedInStatus setStringValue:@"No modem"];
  +}
  +
   -(BOOL)storeUsageHistory
   {
          return [[NSUserDefaults standardUserDefaults] boolForKey:@"CWStoreHistory"];
  @@ -686,6 +706,9 @@ void calloutProc (SCNetworkConnectionRef connection, SCNetworkConnectionStatus s
                          }
                  }       
          }
  +
  +       [mainController haveNoModem];
  +
          [mainController performSelectorOnMainThread:@selector(noModem:) withObject:nil waitUntilDone:YES];
       [pool release];
          free(buf_stream);
  

• 

  Nick Hibma March 28th, 2011 @ 01:18 AM

  • State changed from “new” to “resolved”
  • Milestone order changed from “0” to “0”

  This seems to have been resolved in the 1.4 version. I've not seen a wrong indicator. And the code has changed significantly for this patch to be useful.