commit 59aee29699f8485b14bba6a4faf52965e6fd2d45
Author: Leszek Koltunski <leszek@koltunski.pl>
Date:   Thu Dec 3 23:34:11 2020 +0100

    Change the way we request for reviews.

diff --git a/src/main/java/org/distorted/main/RubikPreRender.java b/src/main/java/org/distorted/main/RubikPreRender.java
index 28650ce4..1483205f 100644
--- a/src/main/java/org/distorted/main/RubikPreRender.java
+++ b/src/main/java/org/distorted/main/RubikPreRender.java
@@ -380,13 +380,9 @@ public class RubikPreRender implements EffectController
   private void requestReview()
     {
     final RubikScores scores = RubikScores.getInstance();
-    int numRuns   = scores.getNumRuns();
-    int numPlay   = scores.getNumPlays();
-    int numReview = scores.getNumReviews();
+    int numWins = scores.incrementNumWins();
 
-    if( numRuns>=2 && numPlay>=5 ) scores.incrementNumReviews();
-
-    if( numReview==0 )
+    if( numWins==8 || numWins==35 || numWins==100 || numWins==200)
       {
       final long timeBegin = System.currentTimeMillis();
       final RubikActivity act = (RubikActivity)mView.getContext();
@@ -409,7 +405,7 @@ public class RubikPreRender implements EffectController
               @Override
               public void onFailure(Exception e)
                 {
-                analyticsReport(act,"Review Flow Failed", name, timeBegin);
+                analyticsReport(act,"Failed", name, timeBegin);
                 }
               });
 
@@ -418,14 +414,14 @@ public class RubikPreRender implements EffectController
               @Override
               public void onComplete(@NonNull Task<Void> task)
                 {
-                analyticsReport(act,"Review Flow Complete", name, timeBegin);
+                analyticsReport(act,"Complete", name, timeBegin);
                 }
               });
             }
           else
             {
             String name = scores.getName();
-            analyticsReport(act,"Request Review Flow not successful", name, timeBegin);
+            analyticsReport(act,"Not Successful", name, timeBegin);
             }
           }
         });
diff --git a/src/main/java/org/distorted/scores/RubikScores.java b/src/main/java/org/distorted/scores/RubikScores.java
index 8e6014ea..4326d52b 100644
--- a/src/main/java/org/distorted/scores/RubikScores.java
+++ b/src/main/java/org/distorted/scores/RubikScores.java
@@ -49,7 +49,7 @@ public class RubikScores
   private boolean mNameIsVerified;
   private int mNumRuns;
   private int mNumPlays;
-  private int mNumReviews;
+  private int mNumWins;
   private int mDeviceID;
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
@@ -72,10 +72,10 @@ public class RubikScores
 
     mNameIsVerified = false;
 
-    mNumPlays   = -1;
-    mNumRuns    = -1;
-    mDeviceID   = -1;
-    mNumReviews =  0;
+    mNumPlays= -1;
+    mNumRuns = -1;
+    mDeviceID= -1;
+    mNumWins =  0;
     }
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
@@ -243,13 +243,6 @@ public class RubikScores
     return mNumRuns;
     }
 
-///////////////////////////////////////////////////////////////////////////////////////////////////
-
-  public int getNumReviews()
-    {
-    return mNumReviews;
-    }
-
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
   public String getName()
@@ -280,9 +273,10 @@ public class RubikScores
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
-  public void incrementNumReviews()
+  public int incrementNumWins()
     {
-    mNumReviews++;
+    mNumWins++;
+    return mNumWins;
     }
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
@@ -388,9 +382,9 @@ public class RubikScores
     editor.putString("scores_name"  , mName  );
     editor.putBoolean("scores_isVerified", mNameIsVerified);
     editor.putInt("scores_numPlays", mNumPlays);
-    editor.putInt("scores_numRuns" , mNumRuns);
+    editor.putInt("scores_numRuns" , mNumRuns );
     editor.putInt("scores_deviceid", mDeviceID);
-    editor.putInt("scores_review"  , mNumReviews);
+    editor.putInt("scores_review"  , mNumWins );
     }
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
@@ -461,7 +455,7 @@ public class RubikScores
     mNumPlays       = preferences.getInt("scores_numPlays", 0);
     mNumRuns        = preferences.getInt("scores_numRuns" , 0);
     mDeviceID       = preferences.getInt("scores_deviceid",-1);
-    mNumReviews     = preferences.getInt("scores_review"  ,-3);
+    mNumWins        = preferences.getInt("scores_review"  , 0);
 
     if( mDeviceID==-1 ) mDeviceID = privateGetDeviceID();