From: Akash Kumar <akash1.kumar@samsung.com>
Date: Sat, 9 Sep 2023 11:16:14 +0000 (+0530)
Subject: Resolve correct MoreMenu items not showing based on wifi state
X-Git-Tag: accepted/tizen/unified/20230920.161437~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1a2f6e4d1bffe740a89831c47bd974b33e0fe44e;p=profile%2Fiot%2Fapps%2Fdotnet%2Fsetting-wifi.git

Resolve correct MoreMenu items not showing based on wifi state

This patch resolves
- 'Find Hidden network' and 'WifiDirect' was also showing along with 'Advanced' items when wifi is off.
- Jira Issue: https://code.sec.samsung.net/jira/browse/TEIGHT-5154

Change-Id: I45b237ce37fcf82335e0a2d369ecee9a8976f03c
Signed-off-by: Akash Kumar <akash1.kumar@samsung.com>
---

diff --git a/SettingWiFi/SettingWiFi/view/MainPage.cs b/SettingWiFi/SettingWiFi/view/MainPage.cs
index 04a6b30..cf086fe 100755
--- a/SettingWiFi/SettingWiFi/view/MainPage.cs
+++ b/SettingWiFi/SettingWiFi/view/MainPage.cs
@@ -5,7 +5,6 @@ using Tizen.NUI.BaseComponents;
 using Tizen.NUI.Binding;
 using Tizen.NUI.Components;
 using SettingCore;
-//using SettingWiFi.res.locale;
 using static SettingWiFi.Logger;
 using System;
 using System.Linq;
@@ -17,6 +16,8 @@ namespace SettingWiFi
         private WiFi mWifi;
         private APSource mApSource;
         private Button mScanButton, mAddNetwork;
+        private List<MoreMenuItem> mMenus;
+        private MoreMenuItem mHiddenMenuItem, mWifiDirectMenuItem;
 
         public override string ProvideIconPath() => GetResourcePath("main-menu-wifi-icon.svg");
         public override Color ProvideIconColor() => new Color(ThemeManager.PlatformThemeId == "org.tizen.default-light-theme" ? "#FF6200" : "#FF8A00");
@@ -57,38 +58,43 @@ namespace SettingWiFi
 
         public override IEnumerable<MoreMenuItem> ProvideMoreMenu()
         {
-            var menus = new List<MoreMenuItem>();
+            mMenus = new List<MoreMenuItem>();
 
             bool? active = GetWifiState();
             if (active.HasValue && active.Value)
             {
-                menus.Add(new MoreMenuItem
+                mHiddenMenuItem = new MoreMenuItem
                 {
                     Text = Resources.IDS_WIFI_BUTTON_FIND_HIDDEN_NETWORK,
-                    Action = () => {
+                    Action = () =>
+                    {
                         OnHiddenNetworkClicked();
                     },
-                });
-                menus.Add(new MoreMenuItem
+                };
+                mMenus.Add(mHiddenMenuItem);
+
+                mMenus.Add(new MoreMenuItem
                 {
                     Text = Resources.IDS_ST_BODY_ADVANCED,
                     Action = () => { },
                 });
-                menus.Add(new MoreMenuItem
+
+                mWifiDirectMenuItem = new MoreMenuItem
                 {
                     Text = Resources.IDS_WIFI_BODY_WI_FI_DIRECT_ABB,
                     Action = () => { },
-                });
+                };
+                mMenus.Add(mWifiDirectMenuItem);
             }
             else
             {
-                menus.Add(new MoreMenuItem
+                mMenus.Add(new MoreMenuItem
                 {
                     Text = Resources.IDS_ST_BODY_ADVANCED,
                     Action = () => { },
                 });
             }
-            return menus;
+            return mMenus;
         }
 
         public override IEnumerable<View> ProvideMoreActions()
@@ -390,6 +396,7 @@ namespace SettingWiFi
                 UpdateMainPageButtonState(true);
                 mApSource.UpdateGroupTitle(Resources.IDS_ST_HEADER_AVAILABLE_NETWORKS);
                 ScanAP();
+                UpdateMoreMenuItems(true);
             }
             else
             {
@@ -400,6 +407,7 @@ namespace SettingWiFi
                 mApSource.RemoveScanList();
                 mApSource.UpdateGroupTitle("");
                 UpdateMainPageButtonState(false);
+                UpdateMoreMenuItems(false);
             }
         }
 
@@ -409,6 +417,20 @@ namespace SettingWiFi
             mAddNetwork.IsEnabled = isEnabled;
         }
 
+        private void UpdateMoreMenuItems(bool isWiFiEnabled)
+        {
+            if (isWiFiEnabled)
+            {
+                mMenus.Insert(0, mHiddenMenuItem);
+                mMenus.Insert(2, mWifiDirectMenuItem);
+            }
+            else
+            {
+                mMenus.RemoveAt(2);
+                mMenus.RemoveAt(0);
+            }
+        }
+
         private void OnScanClicked(object sender, ClickedEventArgs e)
         {
             Debug("OnScanClicked");
diff --git a/packaging/org.tizen.cssetting-wifi-1.1.4.rpk b/packaging/org.tizen.cssetting-wifi-1.1.4.rpk
index 6dbe761..b0cbfff 100644
Binary files a/packaging/org.tizen.cssetting-wifi-1.1.4.rpk and b/packaging/org.tizen.cssetting-wifi-1.1.4.rpk differ