From b2c4f51ae8fd39ee1db99a3ee52ef3c3acc02d69 Mon Sep 17 00:00:00 2001
From: Andrei <dan.andrei91@gmail.com>
Date: Wed, 17 Jun 2020 09:45:49 -0700
Subject: [PATCH] #1161 (android widget) RemoteViews empty state

---
 clients/android/NewsBlur/res/layout/view_app_widget.xml    | 2 +-
 clients/android/NewsBlur/res/values/strings.xml            | 1 +
 .../NewsBlur/src/com/newsblur/activity/WidgetConfig.java   | 4 ++--
 .../NewsBlur/src/com/newsblur/widget/WidgetProvider.java   | 7 +++++++
 .../NewsBlur/src/com/newsblur/widget/WidgetUtils.java      | 6 ------
 5 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/clients/android/NewsBlur/res/layout/view_app_widget.xml b/clients/android/NewsBlur/res/layout/view_app_widget.xml
index 60ac65100..85ab736d1 100644
--- a/clients/android/NewsBlur/res/layout/view_app_widget.xml
+++ b/clients/android/NewsBlur/res/layout/view_app_widget.xml
@@ -45,7 +45,7 @@
         android:layout_width="match_parent"
         android:layout_height="match_parent"
         android:gravity="center"
-        android:text="@string/title_widget_setup"
+        android:text="@string/title_widget_loading"
         android:textColor="@color/widget_feed_title" />
 
     <!--    Note that empty views must be siblings of the collection view
diff --git a/clients/android/NewsBlur/res/values/strings.xml b/clients/android/NewsBlur/res/values/strings.xml
index 2a8f6f374..c5ea7f311 100644
--- a/clients/android/NewsBlur/res/values/strings.xml
+++ b/clients/android/NewsBlur/res/values/strings.xml
@@ -243,6 +243,7 @@
     <string name="widget">Widget</string>
     <string name="title_widget_setup">Tap to setup in NewsBlur</string>
     <string name="title_no_subscriptions">No active subscriptions detected</string>
+    <string name="title_widget_loading">Loading...</string>
 
     <string name="settings_cat_offline">Offline</string>
     <string name="settings_enable_offline">Download Stories</string>
diff --git a/clients/android/NewsBlur/src/com/newsblur/activity/WidgetConfig.java b/clients/android/NewsBlur/src/com/newsblur/activity/WidgetConfig.java
index d0482cda5..60f01a0c1 100644
--- a/clients/android/NewsBlur/src/com/newsblur/activity/WidgetConfig.java
+++ b/clients/android/NewsBlur/src/com/newsblur/activity/WidgetConfig.java
@@ -62,8 +62,8 @@ public class WidgetConfig extends NbActivity {
     @Override
     protected void onPause() {
         super.onPause();
-        // notify widget to refresh next time it's viewed
-        WidgetUtils.notifyViewDataChanged(this);
+        // notify widget to update next time it's viewed
+        WidgetUtils.updateWidget(this);
     }
 
     @Override
diff --git a/clients/android/NewsBlur/src/com/newsblur/widget/WidgetProvider.java b/clients/android/NewsBlur/src/com/newsblur/widget/WidgetProvider.java
index 17ee7c60b..c3c69ec12 100644
--- a/clients/android/NewsBlur/src/com/newsblur/widget/WidgetProvider.java
+++ b/clients/android/NewsBlur/src/com/newsblur/widget/WidgetProvider.java
@@ -18,6 +18,8 @@ import com.newsblur.util.FeedSet;
 import com.newsblur.util.PrefsUtils;
 import com.newsblur.util.WidgetBackground;
 
+import java.util.Set;
+
 public class WidgetProvider extends AppWidgetProvider {
 
     private static String TAG = "WidgetProvider";
@@ -51,6 +53,7 @@ public class WidgetProvider extends AppWidgetProvider {
         Log.d(TAG, "onUpdate");
         WidgetUtils.checkWidgetUpdateAlarm(context);
         WidgetBackground widgetBackground = PrefsUtils.getWidgetBackground(context);
+        Set<String> feedIds = PrefsUtils.getWidgetFeedIds(context);
         for (int appWidgetId : appWidgetIds) {
 
             // Set up the intent that starts the WidgetRemoteViewService, which will
@@ -79,6 +82,10 @@ public class WidgetProvider extends AppWidgetProvider {
             // object above.
             rv.setEmptyView(R.id.widget_list, R.id.widget_empty_view);
 
+            if (feedIds != null && feedIds.isEmpty()) {
+                rv.setTextViewText(R.id.widget_empty_view, context.getString(R.string.title_widget_setup));
+            }
+
             Intent configIntent = new Intent(context, WidgetProvider.class);
             configIntent.setAction(WidgetUtils.ACTION_OPEN_CONFIG);
             PendingIntent configIntentTemplate = PendingIntent.getBroadcast(context, WidgetUtils.RC_WIDGET_CONFIG, configIntent,
diff --git a/clients/android/NewsBlur/src/com/newsblur/widget/WidgetUtils.java b/clients/android/NewsBlur/src/com/newsblur/widget/WidgetUtils.java
index b5c8b2a72..d1a1a70ca 100644
--- a/clients/android/NewsBlur/src/com/newsblur/widget/WidgetUtils.java
+++ b/clients/android/NewsBlur/src/com/newsblur/widget/WidgetUtils.java
@@ -55,12 +55,6 @@ public class WidgetUtils {
         return appWidgetIds.length > 0;
     }
 
-    public static void notifyViewDataChanged(Context context) {
-        AppWidgetManager appWidgetManager = AppWidgetManager.getInstance(context);
-        int[] appWidgetIds = appWidgetManager.getAppWidgetIds(new ComponentName(context, WidgetProvider.class));
-        appWidgetManager.notifyAppWidgetViewDataChanged(appWidgetIds, R.id.widget_list);
-    }
-
     public static boolean isLoggedIn(Context context) {
         return PrefsUtils.getUniqueLoginKey(context) != null;
     }