Changeset 254

Timestamp:

07/19/06 02:47:48 (18 years ago)

Author:

Nicholas Riley

Message:

Register for time changed events so we actually get them in the alarm set dialog.

File:

• trunk/hiptop/pester/net/sabi/pester/AlarmSetDialog.java (modified) (4 diffs)

trunk/hiptop/pester/net/sabi/pester/AlarmSetDialog.java

@@ -127 +127 @@
         }
         validate();
+        Application.registerForEvent(this, Event.EVENT_TIME_CHANGED));
         show();
         if (!mHaveShownWindow) {
@@ -157 +158 @@
             date.addSeconds(editingInterval());
         } else {
+            // XXX is it necessary to do this, or do the two share the
+            // XXX same physical Date object?
             date = mDateEditor.getDate();
             date.setTime(mTimeEditor.getDate().getTime());
@@ -203 +206 @@
         mEditingAlarm = null;
         mValidateAlarm.deactivate();
+        Application.unregisterForEvent(this, Event.EVENT_TIME_CHANGED);
     }
 
@@ -231 +235 @@
             stopEditing();
             return true;
-        case Event.EVENT_TIME_CHANGED:
-            // XXX this isn't being delivered on the simulator, why?
-            DEBUG.p("time changed");
-            validate();
-            return true;
-        case Event.EVENT_ALARM:
+        case Event.EVENT_TIME_CHANGED: // time on device changed
+        case Event.EVENT_ALARM: // computed time invalid
         case EVENT_VALIDATE: // something changed
             validate();