Make SocialProvider names match the case the Firebase provider use

Author: puf

app/src/main/java/com/firebase/uidemo/RecyclerViewDemoActivity.java

@@ -17,6 +17,7 @@
 import com.firebase.client.AuthData;
 import com.firebase.client.Firebase;
 import com.firebase.client.FirebaseError;
+import com.firebase.client.Query;
 import com.firebase.ui.FirebaseLoginBaseActivity;
 import com.firebase.ui.FirebaseRecyclerViewAdapter;
 import com.firebase.ui.com.firebasei.ui.authimpl.SocialProvider;
@@ -58,7 +59,8 @@ public void onComplete(FirebaseError firebaseError, Firebase firebase) {
             }
         });
 
-        FirebaseRecyclerViewAdapter<Chat, ChatHolder> adapter = new FirebaseRecyclerViewAdapter<Chat, ChatHolder>(Chat.class, android.R.layout.two_line_list_item, ChatHolder.class, mRef) {
+        Query recentMessages = mRef.limitToLast(50);
+        FirebaseRecyclerViewAdapter<Chat, ChatHolder> adapter = new FirebaseRecyclerViewAdapter<Chat, ChatHolder>(Chat.class, android.R.layout.two_line_list_item, ChatHolder.class, recentMessages) {
             @Override
             public void populateViewHolder(ChatHolder chatView, Chat chat) {
                 chatView.textView.setText(chat.getText());
@@ -102,7 +104,7 @@ public boolean onPrepareOptionsMenu(Menu menu) {
     public boolean onOptionsItemSelected(MenuItem item) {
         switch (item.getItemId()) {
             case LOGIN:
-                this.loginWithProvider(SocialProvider.Google);
+                this.loginWithProvider(SocialProvider.google);
                 return true;
             case LOGOUT:
                 this.logout();

library/src/main/java/com/firebase/ui/FirebaseLoginBaseActivity.java

@@ -41,11 +41,11 @@ public abstract class FirebaseLoginBaseActivity extends AppCompatActivity {
 
     public void loginWithProvider(SocialProvider provider) {
         switch (provider) {
-            case Google:
+            case google:
                 mGoogleAuthHelper.login();
                 break;
-            case Facebook:
-            case Twitter:
+            case facebook:
+            case twitter:
                 throw new UnsupportedOperationException();
         }
 
@@ -54,11 +54,11 @@ public void loginWithProvider(SocialProvider provider) {
 
     public void logout() {
         switch (mChosenProvider) {
-            case Google:
+            case google:
                 mGoogleAuthHelper.logout();
                 break;
-            case Facebook:
-            case Twitter:
+            case facebook:
+            case twitter:
                 throw new UnsupportedOperationException();
         }
         mRef.unauth();
@@ -97,18 +97,7 @@ protected void onStart() {
             @Override
             public void onAuthStateChanged(AuthData authData) {
                 if (authData != null) {
-                    if (authData.getProvider().equals("google")) {
-                        mChosenProvider = SocialProvider.Google;
-                    }
-                    else if (authData.getProvider().equals("facebook")) {
-                        mChosenProvider = SocialProvider.Facebook;
-                    }
-                    else if (authData.getProvider().equals("twitter")) {
-                        mChosenProvider = SocialProvider.Twitter;
-                    }
-                    else {
-                        throw new UnsupportedOperationException("Provider '"+authData.getProvider()+"' is not handled");
-                    }
+                    mChosenProvider = SocialProvider.valueOf(authData.getProvider());
                     onFirebaseLogin(authData);
                 } else {
                     onFirebaseLogout();

library/src/main/java/com/firebase/ui/com/firebasei/ui/authimpl/SocialProvider.java

@@ -1,7 +1,8 @@
 package com.firebase.ui.com.firebasei.ui.authimpl;
 
+// Note: these enum values must be lowercase, to ensure they match the casing used by the Firebase Authentication providers
 public enum SocialProvider {
-    Google,
-    Facebook,
-    Twitter
+    google,
+    facebook,
+    twitter
 }