Changeset 15114 for platform/eclipse/sabotter/trunk/jp.xet.sabotter.core/src/jp/xet/sabotter/core/twitter/TwitterService.java

Timestamp:

07/03/08 19:49:47 (5 months ago)

Author:

yamashiro

Message:

不正なid/パスワードのときに固まる問題が再現したので修正した。また、その際に、userNameなどをuserIdに統一するようにリファクタリングしたり、その他のリファクタを行った

Files:

• platform/eclipse/sabotter/trunk/jp.xet.sabotter.core/src/jp/xet/sabotter/core/twitter/TwitterService.java (modified) (4 diffs)

platform/eclipse/sabotter/trunk/jp.xet.sabotter.core/src/jp/xet/sabotter/core/twitter/TwitterService.java

@@ -28 +28 @@
         public TwitterService() {
                 // FIXME
-//              IProxyService manager = ProxyManager.getProxyManager();
-//              if (manager.isProxiesEnabled()) {
-//                      IProxyData proxyData = manager.getProxyData(IProxyData.HTTP_PROXY_TYPE);
-//                      if (proxyData != null) {
-//                              System.setProperty("http.proxyHost", proxyData.getHost());
-//                              System.setProperty("http.proxyPort", String.valueOf(proxyData.getPort()));
-//                              if (proxyData.isRequiresAuthentication()) {
-//                                      System.setProperty("http.proxyUser", proxyData.getUserId());
-//                                      System.setProperty("http.proxyPassword", proxyData.getPassword());
-//                              }
-//                      }
+// IProxyService manager = ProxyManager.getProxyManager();
+// if (manager.isProxiesEnabled()) {
+// IProxyData proxyData = manager.getProxyData(IProxyData.HTTP_PROXY_TYPE);
+// if (proxyData != null) {
+// System.setProperty("http.proxyHost", proxyData.getHost());
+// System.setProperty("http.proxyPort", String.valueOf(proxyData.getPort()));
+// if (proxyData.isRequiresAuthentication()) {
+// System.setProperty("http.proxyUser", proxyData.getUserId());
+// System.setProperty("http.proxyPassword", proxyData.getPassword());
+// }
+// }
 //                      
-//              }
+// }
                 
                 this.twitter = new PatentAsyncTwitterImpl();
-//              this.twitter.verifyCredentials();
                 this.twitter.setSource(TWITTER_SOURCE);
         }
@@ -62 +61 @@
          * {@inheritDoc}
          */
-        public void setUserId(String userId) {
+        public void setAccount(String userId, String password) {
                 twitter.setUserId(userId);
-        }
-        
-        /**
-         * {@inheritDoc}
-         */
-        public void setPassword(String password) {
                 twitter.setPassword(password);
+                twitter.verifyCredentials();
         }
         
@@ -83 +77 @@
          * {@inheritDoc}
          */
-        public final void createFavorite(String id, MiniblogServiceResponceListener listener) {
-                this.twitter.createFavoriteAsync(Integer.parseInt(id), new TwitterAdapterWrapper(listener));
+        public final void createFavorite(String statusId, MiniblogServiceResponceListener listener) {
+                this.twitter.createFavoriteAsync(Integer.parseInt(statusId), new TwitterAdapterWrapper(listener));
         }
         
@@ -138 +132 @@
          * @see jp.xet.sabotter.core.MiniblogService#changeAccount(java.lang.String, java.lang.String)
          */
-        public void changeAccount(String username, String password) {
-                twitter.setUserId(username);
+        public void changeAccount(String userId, String password) {
+                twitter.setUserId(userId);
                 twitter.setPassword(password);
+                twitter.verifyCredentials();
         }
 }