[Applications.UI] Modify UICoreBackend for time zone event (#5542)

This patch registers and unregister the time zone changed event. Signed-off-by: Hwankyu Jhun <[email protected]>
Samsung · Sep 11, 2023 · 7148102 · 7148102
1 parent 706b47d
commit 7148102
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/src/Tizen.Applications.UI/Tizen.Applications.CoreBackend/UICoreBackend.cs b/src/Tizen.Applications.UI/Tizen.Applications.CoreBackend/UICoreBackend.cs
@@ -28,12 +28,14 @@ internal class UICoreBackend : DefaultCoreBackend
         private IntPtr _localeChangedEventHandle = IntPtr.Zero;
         private IntPtr _regionChangedEventHandle = IntPtr.Zero;
         private IntPtr _deviceOrientationChangedEventHandle = IntPtr.Zero;
+        private IntPtr _timeZoneChangedEventHandle = IntPtr.Zero;
         private bool _disposedValue = false;
         private Interop.Application.AppEventCallback _onLowMemoryNative;
         private Interop.Application.AppEventCallback _onLowBatteryNative;
         private Interop.Application.AppEventCallback _onLocaleChangedNative;
         private Interop.Application.AppEventCallback _onRegionChangedNative;
         private Interop.Application.AppEventCallback _onDeviceOrientationChangedNative;
+        private Interop.Application.AppEventCallback _onTimeZoneChangedNative;
 
         public UICoreBackend()
         {
@@ -48,6 +50,7 @@ public UICoreBackend()
             _onLocaleChangedNative = new Interop.Application.AppEventCallback(OnLocaleChangedNative);
             _onRegionChangedNative = new Interop.Application.AppEventCallback(OnRegionChangedNative);
             _onDeviceOrientationChangedNative = new Interop.Application.AppEventCallback(OnDeviceOrientationChangedNative);
+            _onTimeZoneChangedNative = new Interop.Application.AppEventCallback(OnTimeZoneChangedNative);
         }
 
         public override void Exit()
@@ -89,6 +92,12 @@ public override void Run(string[] args)
                 Log.Error(LogTag, "Failed to add event handler for DeviceOrientationChanged event. Err = " + err);
             }
 
+            err = Interop.Application.AddEventHandler(out _timeZoneChangedEventHandle, AppEventType.TimeZoneChanged, _onTimeZoneChangedNative, IntPtr.Zero);
+            if (err != ErrorCode.None)
+            {
+                Log.Error(LogTag, "Failed to add event handler for TimeZoneChanged event. Err = " + err);
+            }
+
             err = Interop.Application.Main(args.Length, args, ref _callbacks, IntPtr.Zero);
             if (err != ErrorCode.None)
             {
@@ -125,6 +134,10 @@ protected override void Dispose(bool disposing)
                 {
                     Interop.Application.RemoveEventHandler(_deviceOrientationChangedEventHandle);
                 }
+                if (_timeZoneChangedEventHandle != IntPtr.Zero)
+                {
+                    Interop.Application.RemoveEventHandler(_timeZoneChangedEventHandle);
+                }
 
                 _disposedValue = true;
             }
@@ -161,8 +174,7 @@ private void OnAppControlNative(IntPtr appControlHandle, IntPtr data)
             {
                 // Create a SafeAppControlHandle but the ownsHandle is false,
                 // because the appControlHandle will be closed by native appfw after this method automatically.
-                SafeAppControlHandle safeHandle = new SafeAppControlHandle(appControlHandle, false);
-
+                using SafeAppControlHandle safeHandle = new SafeAppControlHandle(appControlHandle, false);
                 var handler = Handlers[EventType.AppControlReceived] as Action<AppControlReceivedEventArgs>;
                 handler?.Invoke(new AppControlReceivedEventArgs(new ReceivedAppControl(safeHandle)));
             }
@@ -210,5 +222,10 @@ protected override void OnDeviceOrientationChangedNative(IntPtr infoHandle, IntP
         {
             base.OnDeviceOrientationChangedNative(infoHandle, data);
         }
+
+        protected override void OnTimeZoneChangedNative(IntPtr infoHandle, IntPtr data)
+        {
+            base.OnTimeZoneChangedNative(infoHandle, data);
+        }
     }
 }
diff --git a/src/Tizen.Applications.UI/Tizen.Applications/CoreUIApplication.cs b/src/Tizen.Applications.UI/Tizen.Applications/CoreUIApplication.cs
@@ -35,7 +35,9 @@ public class CoreUIApplication : CoreApplication
         /// The default backend for the UI application will be used.
         /// </remarks>
         /// <since_tizen> 3 </since_tizen>
+#pragma warning disable CA2000
         public CoreUIApplication() : base(new UICoreBackend())
+#pragma warning restore CA2000
         {
         }