Fix defects detected by static analysis tool 09/318609/1
authorCORP\mobaswirul.i <mobaswirul.i@samsung.com>
Thu, 3 Oct 2024 09:15:33 +0000 (15:15 +0600)
committerCORP\mobaswirul.i <mobaswirul.i@samsung.com>
Thu, 3 Oct 2024 09:15:33 +0000 (15:15 +0600)
[Problem] Issues detected by Static Analysis Tool
[Cause & Measure]
 Cause   : TabButton cannot be disposed properly
 Measure : Used inline style instead

Change-Id: Ideaeab035803a61dca43f363fee20184b917830a
Signed-off-by: CORP\mobaswirul.i <mobaswirul.i@samsung.com>
SettingMainGadget/SettingMainGadget/About/AboutUserCertificatesGadget.cs
packaging/org.tizen.settings.main-1.2.4.rpk

index 85552904bbf86cb626a114528393aeee3c941f5b..bff070e38ff05197eb7caf9db306db7f7cd7859e 100644 (file)
@@ -21,7 +21,6 @@ namespace Setting.Menu.About
         private ScrollableBase vpnTabContent, wifiTabContent, emailTabContent;
 
         private MoreMenuItem installMenuItem, uninstallMenuItem;
-        private TabButtonStyle tabButtonStyle;
         public override string ProvideTitle() => "User certificates";
 
         public override IEnumerable<MoreMenuItem> ProvideMoreMenu() => Moremenu();
@@ -83,31 +82,30 @@ namespace Setting.Menu.About
                 HeightSpecification = LayoutParamPolicies.MatchParent,
             };
 
-            tabButtonStyle = ThemeManager.GetStyle("Tizen.NUI.Components.TabButton") as TabButtonStyle;
-            tabButtonStyle.Padding = new Extents(2, 2, 16, 16).SpToPx();
-            tabButtonStyle.Icon.Size = new Size(2, -1).SpToPx();
-
             // VPN tab
-            var vpnTabButton = new TabButton(tabButtonStyle)
+            var vpnTabButton = new TabButton("Tizen.NUI.Components.TabButton")
             {
                 // TODO: Add string resouce
                 Text = "VPN",
+                Padding = new Extents(2, 2, 16, 16).SpToPx()
             };
             vpnTabContent = TabView();
 
             // Wi-Fi tab
-            var wifiTabButton = new TabButton(tabButtonStyle)
+            var wifiTabButton = new TabButton("Tizen.NUI.Components.TabButton")
             {
                 // TODO: Add string resouce
                 Text = "Wi-Fi",
+                Padding = new Extents(2, 2, 16, 16).SpToPx()
             };
             wifiTabContent = TabView();
 
             // Email tab
-            var emailTabButton = new TabButton(tabButtonStyle)
+            var emailTabButton = new TabButton("Tizen.NUI.Components.TabButton")
             {
                 // TODO: Add string resouce
                 Text = "Email",
+                Padding = new Extents(2, 2, 16, 16).SpToPx()
             };
             emailTabContent = TabView();
 
@@ -231,7 +229,6 @@ namespace Setting.Menu.About
             Logger.Debug("OnDestroy()");
 
             base.OnDestroy();
-            tabButtonStyle.Dispose();
         }
     }
 }
index 574cfbf27cc0409f88b0de28939ba4f8de5aa1da..3ffbf8a84c9d8f05aec5396f90c87804b0af7e0a 100644 (file)
Binary files a/packaging/org.tizen.settings.main-1.2.4.rpk and b/packaging/org.tizen.settings.main-1.2.4.rpk differ