Added ProviderStateChanged event to ProviderManager (#112)

CommunityToolkit.Authentication/CommunityToolkit.Authentication.csproj

@@ -7,12 +7,10 @@
         This package includes .NET Standard authentication helpers such as:
         - BaseProvider:
         - IProvider:
-        - MockPRovider:
+        - MockProvider:
         - ProviderManager:
-        - ProviderManagerChangedState:
         - ProviderState:
         - ProviderStateChangedEventArgs:
-        - ProviderUpdatedEventArgs:
     </Description>
     <PackageTags>Community Toolkit Provider Authentication Auth</PackageTags>
   </PropertyGroup>

CommunityToolkit.Authentication/ProviderManager.cs

@@ -15,7 +15,7 @@ namespace CommunityToolkit.Authentication
     /// ProviderManager.Instance.GlobalProvider = await MsalProvider.CreateAsync(...);
     /// </code>
     /// </example>
-    public partial class ProviderManager : INotifyPropertyChanged
+    public partial class ProviderManager
     {
         /// <summary>
         /// Gets the name of the toolkit client to identify self in Graph calls.
@@ -28,12 +28,14 @@ public partial class ProviderManager : INotifyPropertyChanged
         public static ProviderManager Instance { get; } = new ProviderManager();
 
         /// <summary>
-        /// Event called when the <see cref="IProvider"/> changes.
+        /// Event called when the <see cref="IProvider"/> instance changes.
         /// </summary>
-        public event EventHandler<ProviderUpdatedEventArgs> ProviderUpdated;
+        public event EventHandler<IProvider> ProviderUpdated;
 
-        /// <inheritdoc/>
-        public event PropertyChangedEventHandler PropertyChanged;
+        /// <summary>
+        /// Event called when the <see cref="IProvider"/> state changes.
+        /// </summary>
+        public event EventHandler<ProviderStateChangedEventArgs> ProviderStateChanged;
 
         private IProvider _provider;
 
@@ -49,32 +51,28 @@ public IProvider GlobalProvider
 
             set
             {
+                var oldState = _provider?.State;
                 if (_provider != null)
                 {
                     _provider.StateChanged -= ProviderStateChanged;
                 }
 
                 _provider = value;
 
+                var newState = _provider?.State;
                 if (_provider != null)
                 {
                     _provider.StateChanged += ProviderStateChanged;
                 }
 
-                PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(nameof(GlobalProvider)));
-                ProviderUpdated?.Invoke(this, new ProviderUpdatedEventArgs(ProviderManagerChangedState.ProviderChanged));
-                ProviderUpdated?.Invoke(this, new ProviderUpdatedEventArgs(ProviderManagerChangedState.ProviderStateChanged));
+                ProviderUpdated?.Invoke(this, _provider);
+                ProviderStateChanged?.Invoke(this, new ProviderStateChangedEventArgs(oldState, newState));
             }
         }
 
         private ProviderManager()
         {
             // Use Instance
         }
-
-        private void ProviderStateChanged(object sender, ProviderStateChangedEventArgs e)
-        {
-            ProviderUpdated?.Invoke(this, new ProviderUpdatedEventArgs(ProviderManagerChangedState.ProviderStateChanged));
-        }
     }
 }

CommunityToolkit.Authentication/ProviderStateChangedEventArgs.cs

@@ -16,7 +16,7 @@ public class ProviderStateChangedEventArgs : EventArgs
         /// </summary>
         /// <param name="oldState">Previous <see cref="ProviderState"/>.</param>
         /// <param name="newState">Current <see cref="ProviderState"/>.</param>
-        public ProviderStateChangedEventArgs(ProviderState oldState, ProviderState newState)
+        public ProviderStateChangedEventArgs(ProviderState? oldState, ProviderState? newState)
         {
             OldState = oldState;
             NewState = newState;
@@ -25,11 +25,11 @@ public ProviderStateChangedEventArgs(ProviderState oldState, ProviderState newSt
         /// <summary>
         /// Gets the previous state of the <see cref="IProvider"/>.
         /// </summary>
-        public ProviderState OldState { get; private set; }
+        public ProviderState? OldState { get; private set; }
 
         /// <summary>
         /// Gets the new state of the <see cref="IProvider"/>.
         /// </summary>
-        public ProviderState NewState { get; private set; }
+        public ProviderState? NewState { get; private set; }
     }
 }

CommunityToolkit.Graph.Uwp/Controls/LoginButton/LoginButton.cs

@@ -0,0 +1,229 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+// See the LICENSE file in the project root for more information.
+
+using System;
+using System.ComponentModel;
+using System.Diagnostics;
+using System.Threading.Tasks;
+using CommunityToolkit.Authentication;
+using CommunityToolkit.Graph.Extensions;
+using Windows.UI.Xaml;
+using Windows.UI.Xaml.Controls;
+using Windows.UI.Xaml.Controls.Primitives;
+
+namespace CommunityToolkit.Graph.Uwp.Controls
+{
+    /// <summary>
+    /// The <see cref="LoginButton"/> control is a button which can be used to sign the user in or show them profile details.
+    /// </summary>
+    [TemplatePart(Name = LoginButtonPart, Type = typeof(Button))]
+    [TemplatePart(Name = SignOutButtonPart, Type = typeof(ButtonBase))]
+    public partial class LoginButton : Control
+    {
+        private const string LoginButtonPart = "PART_LoginButton";
+        private const string SignOutButtonPart = "PART_SignOutButton";
+
+        private Button _loginButton;
+        private ButtonBase _signOutButton;
+
+        /// <summary>
+        /// Initializes a new instance of the <see cref="LoginButton"/> class.
+        /// </summary>
+        public LoginButton()
+        {
+            this.DefaultStyleKey = typeof(LoginButton);
+
+            ProviderManager.Instance.ProviderStateChanged += (sender, args) => LoadData();
+        }
+
+        /// <inheritdoc/>
+        protected override void OnApplyTemplate()
+        {
+            base.OnApplyTemplate();
+
+            IsLoading = true;
+            LoadData();
+
+            if (_loginButton != null)
+            {
+                _loginButton.Click -= LoginButton_Click;
+            }
+
+            _loginButton = GetTemplateChild(LoginButtonPart) as Button;
+
+            if (_loginButton != null)
+            {
+                _loginButton.Click += LoginButton_Click;
+            }
+
+            if (_signOutButton != null)
+            {
+                _signOutButton.Click -= LoginButton_Click;
+            }
+
+            _signOutButton = GetTemplateChild(SignOutButtonPart) as ButtonBase;
+
+            if (_signOutButton != null)
+            {
+                _signOutButton.Click += SignOutButton_Click;
+            }
+        }
+
+        private async void LoginButton_Click(object sender, RoutedEventArgs e)
+        {
+            if (this.UserDetails != null)
+            {
+                if (FlyoutBase.GetAttachedFlyout(_loginButton) is FlyoutBase flyout)
+                {
+                    flyout.ShowAt(_loginButton);
+                }
+            }
+            else
+            {
+                var cargs = new CancelEventArgs();
+                LoginInitiated?.Invoke(this, cargs);
+
+                if (!cargs.Cancel)
+                {
+                    await SignInAsync();
+                }
+            }
+        }
+
+        private async void SignOutButton_Click(object sender, RoutedEventArgs e)
+        {
+            await SignOutAsync();
+        }
+
+        private async void LoadData()
+        {
+            var provider = ProviderManager.Instance.GlobalProvider;
+
+            if (provider == null)
+            {
+                return;
+            }
+
+            if (provider.State == ProviderState.Loading)
+            {
+                IsLoading = true;
+            }
+            else if (provider.State == ProviderState.SignedIn)
+            {
+                try
+                {
+                    // https://github.com/microsoftgraph/microsoft-graph-toolkit/blob/master/src/components/mgt-login/mgt-login.ts#L139
+                    // TODO: Batch with photo request later? https://github.com/microsoftgraph/msgraph-sdk-dotnet-core/issues/29
+                    UserDetails = await provider.GetClient().GetMeAsync();
+                }
+                catch (Exception e)
+                {
+                    LoginFailed?.Invoke(this, new LoginFailedEventArgs(e));
+                }
+
+                IsLoading = false;
+            }
+            else if (provider.State == ProviderState.SignedOut)
+            {
+                UserDetails = null; // What if this was user provided? Should we not hook into these events then?
+
+                IsLoading = false;
+            }
+            else
+            {
+                // Provider in Loading state
+                Debug.Fail("unsupported state");
+            }
+        }
+
+        /// <summary>
+        /// Initiates logging in with the current <see cref="IProvider"/> registered in the <see cref="ProviderManager"/>.
+        /// </summary>
+        /// <returns>A <see cref="Task"/> representing the asynchronous operation.</returns>
+        public async Task SignInAsync()
+        {
+            if (UserDetails != null || IsLoading)
+            {
+                return;
+            }
+
+            var provider = ProviderManager.Instance.GlobalProvider;
+
+            if (provider != null)
+            {
+                try
+                {
+                    IsLoading = true;
+                    await provider.SignInAsync();
+
+                    if (provider.State == ProviderState.SignedIn)
+                    {
+                        // TODO: include user details?
+                        LoginCompleted?.Invoke(this, new EventArgs());
+
+                        LoadData();
+                    }
+                    else
+                    {
+                        LoginFailed?.Invoke(this, new LoginFailedEventArgs(new TimeoutException("Login did not complete.")));
+                    }
+                }
+                catch (Exception e)
+                {
+                    LoginFailed?.Invoke(this, new LoginFailedEventArgs(e));
+                }
+                finally
+                {
+                    IsLoading = false;
+                }
+            }
+        }
+
+        /// <summary>
+        /// Log a signed-in user out.
+        /// </summary>
+        /// <returns>A <see cref="Task"/> representing the asynchronous operation.</returns>
+        public async Task SignOutAsync()
+        {
+            // Close Menu
+            if (FlyoutBase.GetAttachedFlyout(_loginButton) is FlyoutBase flyout)
+            {
+                flyout.Hide();
+            }
+
+            if (IsLoading)
+            {
+                return;
+            }
+
+            var cargs = new CancelEventArgs();
+            LogoutInitiated?.Invoke(this, cargs);
+
+            if (cargs.Cancel)
+            {
+                return;
+            }
+
+            if (UserDetails != null)
+            {
+                UserDetails = null;
+            }
+            else
+            {
+                return; // No-op
+            }
+
+            var provider = ProviderManager.Instance.GlobalProvider;
+
+            if (provider != null)
+            {
+                IsLoading = true;
+                await provider.SignOutAsync();
+                IsLoading = false;
+
+                LogoutCompleted?.Invoke(this, new EventArgs());
+            }
+        }
+    }
+}