Widgets: workaround showNext action for forecasts

* For some reason, if we send a partial update with the 'showNext' action it doesn't work
* The action only works with a 'full' update (updateAppWidget)
* 'updateAppWidget' shouldn't be used here since this is not a 'full' update
* Workaround this by explicitly requesting to show some child view in the ViewFlipper using a persisted index which flips between the numbers 0-1

Author: thewizrd

Diff for: app/src/main/java/com/thewizrd/simpleweather/widgets/WeatherWidgetProvider4x1.kt

@@ -7,16 +7,26 @@ import android.widget.RemoteViews
 import com.thewizrd.simpleweather.R
 
 class WeatherWidgetProvider4x1 : WeatherWidgetProvider() {
+    companion object {
+        private fun getNextIndex(index: Int): Int {
+            return (index + 1) % 2
+        }
+    }
+
     override val info: WidgetProviderInfo by lazy { Info.getInstance() }
 
     override fun onReceive(context: Context, intent: Intent?) {
         val action = intent?.action
         if (ACTION_SHOWNEXTFORECAST == action) {
             val appWidgetManager = AppWidgetManager.getInstance(context)
-            val views = RemoteViews(context.packageName, info.widgetLayoutId)
-            views.showNext(R.id.forecast_layout)
             val appWidgetId =
                 intent.getIntExtra(EXTRA_WIDGET_ID, AppWidgetManager.INVALID_APPWIDGET_ID)
+
+            // Note: workaround; possibly temporary?
+            val views = RemoteViews(context.packageName, info.widgetLayoutId)
+            val index = getNextIndex(WidgetUtils.getDisplayedChild(appWidgetId))
+            views.setInt(R.id.forecast_layout, "setDisplayedChild", index)
+            WidgetUtils.setDisplayedChild(appWidgetId, index)
             appWidgetManager.partiallyUpdateAppWidget(appWidgetId, views)
         } else {
             super.onReceive(context, intent)

Diff for: app/src/main/java/com/thewizrd/simpleweather/widgets/WeatherWidgetProvider4x2.kt

@@ -7,25 +7,38 @@ import android.widget.RemoteViews
 import com.thewizrd.simpleweather.R
 
 class WeatherWidgetProvider4x2 : WeatherWidgetProvider() {
+    companion object {
+        private fun getNextIndex(index: Int): Int {
+            return (index + 1) % 2
+        }
+    }
+
     override val info: WidgetProviderInfo by lazy { Info.getInstance() }
 
     override fun onReceive(context: Context, intent: Intent?) {
         val action = intent?.action
         if (Intent.ACTION_TIME_CHANGED == action || Intent.ACTION_TIMEZONE_CHANGED == action) {
-            WeatherWidgetService.enqueueWork(context, Intent(context, WeatherWidgetService::class.java)
+            WeatherWidgetService.enqueueWork(
+                context, Intent(context, WeatherWidgetService::class.java)
                     .setAction(WeatherWidgetService.ACTION_UPDATECLOCK)
                     .putExtra(EXTRA_WIDGET_IDS, info.appWidgetIds)
-                    .putExtra(EXTRA_WIDGET_TYPE, info.widgetType.value))
+                    .putExtra(EXTRA_WIDGET_TYPE, info.widgetType.value)
+            )
 
             WeatherWidgetService.enqueueWork(context, Intent(context, WeatherWidgetService::class.java)
                     .setAction(WeatherWidgetService.ACTION_UPDATEDATE)
                     .putExtra(EXTRA_WIDGET_IDS, info.appWidgetIds)
                     .putExtra(EXTRA_WIDGET_TYPE, info.widgetType.value))
         } else if (ACTION_SHOWNEXTFORECAST == action) {
             val appWidgetManager = AppWidgetManager.getInstance(context)
+            val appWidgetId =
+                intent.getIntExtra(EXTRA_WIDGET_ID, AppWidgetManager.INVALID_APPWIDGET_ID)
+
+            // Note: workaround; possibly temporary?
             val views = RemoteViews(context.packageName, info.widgetLayoutId)
-            views.showNext(R.id.forecast_layout)
-            val appWidgetId = intent.getIntExtra(EXTRA_WIDGET_ID, AppWidgetManager.INVALID_APPWIDGET_ID)
+            val index = getNextIndex(WidgetUtils.getDisplayedChild(appWidgetId))
+            views.setInt(R.id.forecast_layout, "setDisplayedChild", index)
+            WidgetUtils.setDisplayedChild(appWidgetId, index)
             appWidgetManager.partiallyUpdateAppWidget(appWidgetId, views)
         } else {
             super.onReceive(context, intent)

Diff for: app/src/main/java/com/thewizrd/simpleweather/widgets/WidgetUtils.kt

@@ -947,4 +947,15 @@ object WidgetUtils {
             putString(KEY_LOCATIONS, locations?.joinToString(separator = ";separator;") { it })
         }
     }
+
+    fun getDisplayedChild(appWidgetId: Int): Int {
+        val prefs = getPreferences(appWidgetId)
+        return prefs.getInt("setDisplayedChild", 0)
+    }
+
+    fun setDisplayedChild(appWidgetId: Int, index: Int) {
+        getPreferences(appWidgetId).edit(false) {
+            putInt("setDisplayedChild", index)
+        }
+    }
 }

Diff for: app/src/main/res/anim/widget_fade_in.xml

@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="utf-8"?><!--
+/*
+** Copyright 2010, The Android Open Source Project
+**
+** Licensed under the Apache License, Version 2.0 (the "License");
+** you may not use this file except in compliance with the License.
+** You may obtain a copy of the License at
+**
+**     http://www.apache.org/licenses/LICENSE-2.0
+**
+** Unless required by applicable law or agreed to in writing, software
+** distributed under the License is distributed on an "AS IS" BASIS,
+** WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+** See the License for the specific language governing permissions and
+** limitations under the License.
+*/
+-->
+
+<alpha xmlns:android="http://schemas.android.com/apk/res/android"
+    android:fromAlpha="0.0"
+    android:toAlpha="1.0"
+    android:duration="@android:integer/config_shortAnimTime" />

Diff for: app/src/main/res/anim/widget_fade_out.xml

@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="utf-8"?><!--
+/*
+** Copyright 2010, The Android Open Source Project
+**
+** Licensed under the Apache License, Version 2.0 (the "License");
+** you may not use this file except in compliance with the License.
+** You may obtain a copy of the License at
+**
+**     http://www.apache.org/licenses/LICENSE-2.0
+**
+** Unless required by applicable law or agreed to in writing, software
+** distributed under the License is distributed on an "AS IS" BASIS,
+** WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+** See the License for the specific language governing permissions and
+** limitations under the License.
+*/
+-->
+
+<alpha xmlns:android="http://schemas.android.com/apk/res/android"
+    android:fromAlpha="1.0"
+    android:toAlpha="0.0"
+    android:duration="@android:integer/config_shortAnimTime" />

Diff for: app/src/main/res/layout/app_widget_4x1.xml

@@ -24,7 +24,8 @@
             android:layout_gravity="center_vertical"
             android:autoStart="true"
             android:flipInterval="10000"
-            android:loopViews="true" />
+            android:inAnimation="@anim/widget_fade_in"
+            android:outAnimation="@anim/widget_fade_out" />
 
     </LinearLayout>
 

Diff for: app/src/main/res/layout/app_widget_4x2.xml

@@ -185,6 +185,7 @@
         android:layout_alignParentBottom="true"
         android:autoStart="true"
         android:flipInterval="10000"
-        android:loopViews="true" />
+        android:inAnimation="@anim/widget_fade_in"
+        android:outAnimation="@anim/widget_fade_out" />
 
 </RelativeLayout>