Ask all subscribers for rating

app/src/main/java/org/tasks/dialogs/WhatsNewDialog.kt

@@ -46,25 +46,33 @@ class WhatsNewDialog : InjectingDialogFragment() {
         val text = "$entries\n\nVisit https://tasks.org/changelog for more info"
         changelog.text = text
 
-        @Suppress("ConstantConditionIf")
+        val begForRating = (inventory.hasPro() || firebase.noChurn())
-        if (BuildConfig.FLAVOR == "generic") {
+                && !preferences.getBoolean(R.string.p_clicked_rate, false)
-            actionText.text = getString(R.string.upgrade_blurb_4)
+        val begForSubscription = firebase.noChurn() && !inventory.hasPro()
-            actionButton.text = getString(R.string.TLA_menu_donate)
+
-            actionButton.setOnClickListener { onDonateClick() }
+        when {
-        } else if (firebase.noChurn() && !preferences.getBoolean(R.string.p_clicked_rate, false)) {
+            BuildConfig.FLAVOR == "generic" -> {
-            displayedRate = true
+                actionText.text = getString(R.string.upgrade_blurb_4)
-            actionButton.text = getString(R.string.rate_tasks)
+                actionButton.text = getString(R.string.TLA_menu_donate)
-            actionButton.setOnClickListener { onRateClick() }
+                actionButton.setOnClickListener { onDonateClick() }
-        } else if (firebase.noChurn() && !inventory.hasPro()) {
+            }
-            displayedSubscribe = true
+            begForRating -> {
-            actionText.text = getString(R.string.support_development_subscribe)
+                displayedRate = true
-            actionButton.text = getString(R.string.button_subscribe)
+                actionButton.text = getString(R.string.rate_tasks)
-            actionButton.setOnClickListener { onSubscribeClick() }
+                actionButton.setOnClickListener { onRateClick() }
-        } else {
+            }
-            actionQuestion.visibility = View.GONE
+            begForSubscription -> {
-            actionText.visibility = View.GONE
+                displayedSubscribe = true
-            actionButton.visibility = View.GONE
+                actionText.text = getString(R.string.support_development_subscribe)
-            dismissButton.text = getString(R.string.got_it)
+                actionButton.text = getString(R.string.button_subscribe)
+                actionButton.setOnClickListener { onSubscribeClick() }
+            }
+            else -> {
+                actionQuestion.visibility = View.GONE
+                actionText.visibility = View.GONE
+                actionButton.visibility = View.GONE
+                dismissButton.text = getString(R.string.got_it)
+            }
         }
 
         if (!resources.getBoolean(R.bool.whats_new_action)) {