android_sentiment_pr_inline_comments_joined.csv

comment_id,polarity,text,created_at_gold,author_login,owner,repo,review_id,html_url,created_at_kaiaulu,updated_at,comment_user_login,author_association,file_path,start_line,line,original_start_line,original_line,position,diff_hunk,body,commit_id
1664046,0,"`GitHubAccountAuthenticator` sounds reasonable.""",2012-09-21 06:10:53,kevinsawicki,github,android,,https://github.com/pockethub/PocketHub/pull/224#discussion_r1664046,2012-09-21T18:10:53Z,2012-09-21T18:50:50Z,kevinsawicki,CONTRIBUTOR,app/src/main/java/com/github/mobile/accounts/AccountAuthenticator.java,,,,,1,"@@ -75,7 +85,58 @@ public Bundle editProperties(AccountAuthenticatorResponse response,
     public Bundle getAuthToken(AccountAuthenticatorResponse response,
             Account account, String authTokenType, Bundle options)
             throws NetworkErrorException {
-        return null;
+
+        final Bundle bundle = new Bundle();
+
+        if(!authTokenType.equals(ACCOUNT_TYPE)) return bundle;
+
+        AccountManager am = AccountManager.get(context);
+        String username = account.name;
+        String password = am.getPassword(account);
+
+        String authToken = null;
+        DefaultClient client = new DefaultClient();
+        client.setCredentials(username, password);
+
+        OAuthService oAuthService = new OAuthService(client);
+
+        // Get authorizations for app if they exist
+        try {
+          try {
+            List<Authorization> auths = oAuthService.getAuthorizations();
+            for(Authorization auth : auths)
+              if(auth.getApp().getName().equals(ACCOUNT_NAME))
+                authToken = auth.getToken();
+          } 
+          catch ( NullPointerException npe ) { }
+
+          // Setup authorization for app if others didn't exist.
+          if(TextUtils.isEmpty(authToken)) {
+            Authorization auth = oAuthService.createAuthorization(
+                new Authorization().setNote(ACCOUNT_NAME).setUrl(APP_URL)
+            );
+            if(auth != null) authToken = auth.getToken();
+          }
+
+          // If couldn't get authToken 
+          if(TextUtils.isEmpty(authToken)) {
+            final Intent intent = new Intent(context, LoginActivity.class);
+            intent.putExtra(PARAM_AUTHTOKEN_TYPE, ACCOUNT_TYPE);
+            intent.putExtra(KEY_ACCOUNT_AUTHENTICATOR_RESPONSE, response);
+            bundle.putParcelable(KEY_INTENT, intent);
+            return bundle;
+          }
+
+          // Assemble and return bundle
+          bundle.putString(AccountManager.KEY_ACCOUNT_NAME, account.name);
+          bundle.putString(AccountManager.KEY_ACCOUNT_TYPE, ACCOUNT_TYPE);
+          bundle.putString(AccountManager.KEY_AUTHTOKEN, authToken);
+          
+          // Clear password from account
+          am.clearPassword(account);
+          return bundle;
+        } catch ( Exception e ) { e.printStackTrace(); }","`GitHubAccountAuthenticator` sounds reasonable.
",e70b701b4b7bd5e2dc5cc00ba82605689c3723fb
1663939,1,"Yes! I'll do that.  On Fri, Sep 21, 2012 at 12:57 PM, Kevin Sawicki <notifications@github.com>wrote:  > In app/src/main/java/com/github/mobile/AuthorizationClient.java: > > > @@ -0,0 +1,68 @@ > > +/* > > This class can be removed now right? > >""",2012-09-21 06:00:32,TrevorBasinger,github,android,,https://github.com/pockethub/PocketHub/pull/224#discussion_r1663939,2012-09-21T18:00:32Z,2012-09-21T18:50:50Z,TrevorBasinger,CONTRIBUTOR,app/src/main/java/com/github/mobile/AuthorizationClient.java,,,,,1,"@@ -0,0 +1,68 @@
+/*","Yes! I'll do that.

On Fri, Sep 21, 2012 at 12:57 PM, Kevin Sawicki notifications@github.comwrote:

> In app/src/main/java/com/github/mobile/AuthorizationClient.java:
> 
> > @@ -0,0 +1,68 @@
> > +/*
> 
> This class can be removed now right?
> 
> —
> Reply to this email directly or view it on GitHubhttps://github.com/github/android/pull/224/files#r1663907.
",e70b701b4b7bd5e2dc5cc00ba82605689c3723fb
1222306,2,"Shame on me. You are absolutely right. I reworked the commit according to your comments.""",2012-07-23 17:37:37,Bananeweizen,github,android,,https://github.com/pockethub/PocketHub/pull/145#discussion_r1222306,2012-07-24T05:37:37Z,2012-07-24T05:37:37Z,Bananeweizen,CONTRIBUTOR,app/src/main/java/com/github/mobile/ui/commit/DiffStyler.java,,,,,1,"@@ -72,7 +72,10 @@ public DiffStyler setFiles(final Collection<CommitFile> files) {
             StyledText styled = new StyledText();
             while (end != -1) {
                 String line = patch.substring(start, end + 1);
-                switch (patch.charAt(start)) {
+                if (line.length() == 0) {
+                    continue;","Shame on me. You are absolutely right. I reworked the commit according to your comments.
",d1e578a144299d4a7e1cfcc1705f6d4502152f91