Added some notes in the comments

diff --git a/actions/apiaccountupdatedeliverydevice.php b/actions/apiaccountupdatedeliverydevice.php
index 5b7176bbbecb723839f4c0cf7e76d5a0d65b4065..684906fe9019eb75604c733a3e16ff7e4d842218 100644 (file)
--- a/actions/apiaccountupdatedeliverydevice.php
+++ b/actions/apiaccountupdatedeliverydevice.php
@@ -97,6 +97,8 @@ class ApiAccountUpdateDeliveryDeviceAction extends ApiAuthAction
             return;
         }
 
+        // Note: Twitter no longer supports IM
+
         if (!in_array(strtolower($this->device), array('sms', 'im', 'none'))) {
             $this->clientError(
                 _(
@@ -135,6 +137,12 @@ class ApiAccountUpdateDeliveryDeviceAction extends ApiAuthAction
 
         $twitter_user = $this->twitterUserArray($profile, true);
 
+        // Note: this Twitter API method is retarded because it doesn't give
+        // any success/failure information. Twitter's docs claim that the
+        // notification field will change to reflect notification choice,
+        // but that's not true; notification> is used to indicate
+        // whether the auth user is following the user in question.
+
         if ($this->format == 'xml') {
             $this->initDocument('xml');
             $this->showTwitterXmlUser($twitter_user);