Version 0.5.2

  * Scopes for Google (#281)
  * Gradle build fix (#272)
  * README colorization for code snippets

Change-Id: Ie35a987a027307f3eb092a241b6ca1ce1f21095d

Author: samtstern

README.md

@@ -25,16 +25,16 @@ those separately in your app.
 In your `app/build.gradle` file add a dependency on one of the FirebaseUI
 libraries.
 
-```
+```groovy
 dependencies {
     // Single target that includes all FirebaseUI libraries
-    compile 'com.firebaseui:firebase-ui:0.5.1'
+    compile 'com.firebaseui:firebase-ui:0.5.2'
 
     // FirebaseUI Database only
-    compile 'com.firebaseui:firebase-ui-database:0.5.1'
+    compile 'com.firebaseui:firebase-ui-database:0.5.2'
 
     // FirebaseUI Auth only
-    compile 'com.firebaseui:firebase-ui-auth:0.5.1'
+    compile 'com.firebaseui:firebase-ui-auth:0.5.2'
 }
 ```
 
@@ -61,7 +61,7 @@ For convenience, here are some examples:
 
 | FirebaseUI Version | Firebase/Play Services Version |
 |--------------------|--------------------------------|
-| 0.5.1              | 9.4.0                          |
+| 0.5.2              | 9.4.0                          |
 | 0.4.4              | 9.4.0                          |
 | 0.4.3              | 9.2.1                          |
 | 0.4.2              | 9.2.0                          |

auth/README.md

@@ -41,20 +41,20 @@ Firebase: see the
 Then, add the FirebaseUI auth library dependency. If your project uses
 Gradle, add:
 
-```
+```groovy
 dependencies {
   // ...
-  compile 'com.firebaseui:firebase-ui-auth:0.5.1'
+  compile 'com.firebaseui:firebase-ui-auth:0.5.2'
 }
 ```
 
 If instead your project uses Maven, add:
 
-```
+```xml
 <dependency>
   <groupId>com.firebaseui</groupId>
   <artifactId>firebase-ui-auth</artifactId>
-  <version>0.5.1</version>
+  <version>0.5.2</version>
 </dependency>
 ```
 
@@ -70,7 +70,7 @@ If support for Facebook Sign-in is also required, define the
 resource string `facebook_application_id` to match the application ID in
 the [Facebook developer dashboard](https://developers.facebook.com):
 
-```
+```xml
 <resources>
   <!-- ... -->
   <string name="facebook_application_id" translatable="false">APPID</string>
@@ -83,7 +83,7 @@ Before invoking the FirebaseUI authentication flow, your app should check
 whether a
 [user is already signed in](https://firebase.google.com/docs/auth/android/manage-users#get_the_currently_signed-in_user) from a previous session:
 
-```
+```java
 FirebaseAuth auth = FirebaseAuth.getInstance();
 if (auth.getCurrentUser() != null) {
   // already signed in
@@ -123,7 +123,7 @@ The builder provides the following customization options for the authentication
 If no customization is required, and only email authentication is required, the sign-in flow
 can be started as follows:
 
-```
+```java
 startActivityForResult(
     // Get an instance of AuthUI based on the default app
     AuthUI.getInstance().createSignInIntentBuilder().build(),
@@ -133,7 +133,7 @@ startActivityForResult(
 You can enable sign-in providers like Google Sign-In or Facebook Log In by calling the
 `setProviders` method:
 
-```
+```java
 startActivityForResult(
     AuthUI.getInstance()
         .createSignInIntentBuilder()
@@ -147,7 +147,7 @@ startActivityForResult(
 
 If a terms of service URL and a custom theme are required:
 
-```
+```java
 startActivityForResult(
     AuthUI.getInstance()
         .createSignInIntentBuilder()
@@ -164,7 +164,7 @@ Using SmartLock is recommended to provide the best user experience, but in some
 to disable SmartLock for testing or development.  To disable SmartLock, you can use the 
 `setIsSmartLockEnabled` method when building your sign-in Intent:
 
-```
+```java
 startActivityForResult(
     AuthUI.getInstance()
         .createSignInIntentBuilder()
@@ -176,7 +176,7 @@ startActivityForResult(
 It is often desirable to disable SmartLock in development but enable it in production. To achieve
 this, you can use the `BuildConfig.DEBUG` flag to control SmartLock:
 
-```
+```java
 startActivityForResult(
     AuthUI.getInstance()
         .createSignInIntentBuilder()
@@ -194,7 +194,7 @@ typically useful; the only recourse for most apps if sign in fails is to ask
 the user to sign in again later, or proceed with an anonymous account if
 supported.
 
-```
+```java
 protected void onActivityResult(int requestCode, int resultCode, Intent data) {
    super.onActivityResult(requestCode, resultCode, data);
    if (requestCode == RC_SIGN_IN) {
@@ -234,7 +234,7 @@ In order to make this process easier, AuthUI provides a simple `signOut` method
 to encapsulate this behavior. The method returns a `Task` which is marked
 completed once all necessary sign-out operations are completed:
 
-```
+```java
 public void onClick(View v) {
   if (v.getId() == R.id.sign_out) {
       AuthUI.getInstance()
@@ -265,7 +265,7 @@ the flow, a new theme can be declared. Standard material design color
 and typography properties will take effect as expected. For example, to define
 a green theme:
 
-```
+```xml
 <style name="GreenTheme" parent="FirebaseUI">
     <item name="colorPrimary">@color/material_green_500</item>
     <item name="colorPrimaryDark">@color/material_green_700</item>
@@ -279,7 +279,7 @@ a green theme:
 
 With associated colors:
 
-```
+```xml
 <color name="material_green_50">#E8F5E9</color>
 <color name="material_green_500">#4CAF50</color>
 <color name="material_green_700">#388E3C</color>
@@ -290,7 +290,7 @@ With associated colors:
 
 This would then be used in the construction of the sign-in intent:
 
-```
+```java
 startActivityForResult(
     AuthUI.getInstance(this).createSignInIntentBuilder()
         // ...
@@ -306,7 +306,7 @@ easily overridden by name in your application. See
 [the built-in strings.xml](src/main/res/values/strings.xml) and simply
 redefine a string to change it, for example:
 
-```
+```java
 <resources>
   <!-- was "Signing up..." -->
   <string name="progress_dialog_signing_up">Creating your shiny new account...</string>
@@ -315,24 +315,37 @@ redefine a string to change it, for example:
 
 ### OAuth Scope Customization
 
+#### Google
+By default, FirebaseUI requests the `email` and `profile` scopes when using Google Sign In. If you
+would like to request additional scopes from the user, add a string array resource to your
+application like this:
+
+```xml
+<!--
+    For a list of all scopes, see:
+    https://developers.google.com/identity/protocols/googlescopes
+-->
+<array name="google_permissions">
+    <!-- Request permission to read the user's Google Drive files -->
+    <item>https://www.googleapis.com/auth/drive.readonly</item>
+</array>
+```
+
+
 #### Facebook
 
 By default, FirebaseUI requests the `email` and `public_profile` permissions when initiating
 Facebook Login.  If you would like to override these scopes, add a string array resource
 to your application like this:
 
-```
+```xml
 <!--
     See:
     https://developers.facebook.com/docs/facebook-login/android
     https://developers.facebook.com/docs/facebook-login/permissions
 -->
 <array name="facebook_permissions">
-    <item>public_profile</item>
-    <item>email</item>
-    <!-- ... -->
+    <!-- Request permission to know the user's birthday -->
+    <item>user_birthday</item>
 </array>
 ```
-
-Note that if you do not include at least the `email` and `public_profile` scopes, FirebaseUI
-will not work properly.

auth/src/main/java/com/firebase/ui/auth/provider/FacebookProvider.java

@@ -37,6 +37,7 @@
 import org.json.JSONObject;
 
 import java.util.Arrays;
+import java.util.List;
 
 public class FacebookProvider implements IDPProvider, FacebookCallback<LoginResult> {
     public static final String ACCESS_TOKEN = "facebook_access_token";
@@ -46,6 +47,9 @@ public class FacebookProvider implements IDPProvider, FacebookCallback<LoginResu
 
     private static final String TAG = "FacebookProvider";
     private static final String APPLICATION_ID = "application_id";
+    private static final String EMAIL = "email";
+    private static final String PUBLIC_PROFILE = "public_profile";
+
     private CallbackManager mCallbackManager;
     private IDPCallback mCallbackObject;
 
@@ -79,9 +83,19 @@ public void startLogin(Activity activity) {
         loginManager.registerCallback(mCallbackManager, this);
 
         String[] permissions = activity.getResources().getStringArray(R.array.facebook_permissions);
+        List<String> permissionsList = Arrays.asList(permissions);
+
+        // Ensure we have email and public_profile scopes
+        if (!permissionsList.contains(EMAIL)) {
+            permissionsList.add(EMAIL);
+        }
+
+        if (!permissionsList.contains(PUBLIC_PROFILE)) {
+            permissionsList.add(PUBLIC_PROFILE);
+        }
 
-        loginManager.logInWithReadPermissions(
-                activity, Arrays.asList(permissions));
+        // Log in with permissions
+        loginManager.logInWithReadPermissions(activity, permissionsList);
     }
 
     @Override

auth/src/main/java/com/firebase/ui/auth/provider/GoogleProvider.java

@@ -33,6 +33,7 @@
 import com.google.android.gms.auth.api.signin.GoogleSignInResult;
 import com.google.android.gms.common.ConnectionResult;
 import com.google.android.gms.common.api.GoogleApiClient;
+import com.google.android.gms.common.api.Scope;
 import com.google.firebase.auth.AuthCredential;
 import com.google.firebase.auth.GoogleAuthProvider;
 
@@ -60,6 +61,12 @@ public GoogleProvider(FragmentActivity activity, IDPProviderParcel parcel, @Null
                 .requestEmail()
                 .requestIdToken(mClientId);
 
+        // Add additional scopes
+        String[] scopeStrings = activity.getResources().getStringArray(R.array.google_permissions);
+        for (String scopeString : scopeStrings) {
+            builder.requestScopes(new Scope(scopeString));
+        }
+
         if (!TextUtils.isEmpty(email)) {
             builder.setAccountName(email);
         }

auth/src/main/res/values/config.xml

@@ -30,8 +30,20 @@
     https://developers.facebook.com/docs/facebook-login/permissions
     -->
     <array name="facebook_permissions">
-        <item>public_profile</item>
-        <item>email</item>
+    </array>
+
+    <!--
+    The additional Google scopes that this application will request from the user. Users of
+    FirebaseUI can override this array to customize. Note that the 'email' and 'profile' scopes are
+    always requested.
+
+    For example to add the permission to read the user's files in Google Drive:
+    <item>https://www.googleapis.com/auth/drive.readonly</item>
+
+    For a list of all scopes, see:
+    https://developers.google.com/identity/protocols/googlescopes
+    -->
+    <array name="google_permissions">
     </array>
 
 

build.gradle

@@ -6,7 +6,7 @@ buildscript {
         mavenLocal()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:2.1.2'
+        classpath 'com.android.tools.build:gradle:2.1.3'
         classpath 'com.google.gms:google-services:3.0.0'
 
         classpath 'com.neenbedankt.gradle.plugins:android-apt:1.8'
@@ -44,9 +44,6 @@ allprojects { project ->
     def isLibrary = 'library'.equals(project.name)
     def isSubmodule = project.ext.submodules.contains(project.name)
 
-    // Only applies to submodules, not the library module
-    def pomTask = "generatePomFileFor${project.name.capitalize()}LibraryPublication";
-
     if (isSubmodule || isLibrary) {
 
         // So that we can resolve 'android' variable
@@ -163,9 +160,6 @@ allprojects { project ->
     }
 
     if (isSubmodule) {
-        // Convenience task to prepare everything we need for releases
-        task prepareArtifacts(dependsOn: [javadocJar, sourcesJar, assembleRelease, pomTask]) {}
-
         publishing {
 
             // Define a publication for each submodule
@@ -276,3 +270,18 @@ allprojects { project ->
         }
     }
 }
+
+allprojects {
+    afterEvaluate { project ->
+        def isSubmodule = project.ext.submodules.contains(project.name)
+
+        if (isSubmodule) {
+            // Only applies to submodules, not the library module
+            def pomTask = "generatePomFileFor${project.name.capitalize()}LibraryPublication";
+
+            // Convenience task to prepare everything we need for releases
+            task prepareArtifacts(dependsOn: [javadocJar, sourcesJar, assembleRelease, pomTask]) {}
+        }
+
+    }
+}

common/constants.gradle

@@ -3,7 +3,7 @@ project.ext.support_library_version = '23.4.0'
 
 project.ext.submodules = ['database', 'auth']
 project.ext.group = "com.firebaseui"
-project.ext.version = '0.5.1'
+project.ext.version = '0.5.2'
 project.ext.pomdesc = 'Firebase UI Android'
 project.ext.buildtools = '23.0.3'
 project.ext.compileSdk = 23

library/build.gradle

@@ -65,17 +65,6 @@ def testTasks() {
     }.toArray()
 }
 
-/**
- * Prepare artifacts for this an all sub-projects.
- */
-task prepareArtifacts(dependsOn: [
-        javadocJar,
-        sourcesJar,
-        assembleRelease,
-        prepareArtifactsTasks(),
-        "generatePomFileForMonolithLibraryPublication"]) {
-}
-
 /**
  * Publish all artifacts to the maven local repository.
  */
@@ -108,3 +97,16 @@ task bintrayUploadAll(dependsOn: [
  */
 task testAll(dependsOn: [testTasks()]) {
 }
+
+afterEvaluate {
+    /**
+     * Prepare artifacts for this an all sub-projects.
+     */
+    task prepareArtifacts(dependsOn: [
+            javadocJar,
+            sourcesJar,
+            assembleRelease,
+            prepareArtifactsTasks(),
+            "generatePomFileForMonolithLibraryPublication"]) {
+    }
+}