From ee3fc8ba03ddd8451cac60547af72ea2cef7dc6a Mon Sep 17 00:00:00 2001
From: Zach Copley <zach@status.net>
Date: Tue, 10 Nov 2009 15:23:33 -0800
Subject: [PATCH] Added some notes in the comments

---
 actions/apiaccountupdatedeliverydevice.php | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/actions/apiaccountupdatedeliverydevice.php b/actions/apiaccountupdatedeliverydevice.php
index 5b7176bbbe..684906fe90 100644
--- 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);