From dcdfe59a6ebf929edb1d1b284f96ddc02b5db96f Mon Sep 17 00:00:00 2001
From: Jade_VAN_BRABANDT <Jade.VAN_BRABANDT@etu.uca.fr>
Date: Mon, 5 Jun 2023 13:54:32 +0200
Subject: [PATCH] =?UTF-8?q?Fix=20:=20Bug=20sur=20la=20mise=20=C3=A0=20jour?=
 =?UTF-8?q?=20apr=C3=A8s=20ajout=20d'un=20jeu=20(Merci=20Samuel=20pour=20l?=
 =?UTF-8?q?'aide)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Sources/Stim.Model/Game.cs       |  2 +-
 Sources/Stim.Model/Manager.cs    | 22 ++++++++++
 Sources/Stim.Model/User.cs       |  5 ++-
 Sources/Stim/AddGamePage.xaml.cs |  4 +-
 Sources/Stim/Create.xaml         | 12 +++---
 Sources/Stim/Create.xaml.cs      | 72 +++++++++++++++++---------------
 Sources/Stim/EntryPopup.xaml     |  7 ++--
 Sources/Stim/EntryPopup.xaml.cs  |  5 +++
 Sources/Stim/LoginPage.xaml      | 41 ++++++++++++------
 Sources/Stim/LoginPage.xaml.cs   | 20 ++++++---
 Sources/Stim/MainPage.xaml       |  2 +-
 Sources/Stim/MainPage.xaml.cs    | 22 +++-------
 Sources/Stim/UserInfo.xaml.cs    |  5 +--
 13 files changed, 133 insertions(+), 86 deletions(-)

diff --git a/Sources/Stim.Model/Game.cs b/Sources/Stim.Model/Game.cs
index 93c1d04..b78cdd5 100644
--- a/Sources/Stim.Model/Game.cs
+++ b/Sources/Stim.Model/Game.cs
@@ -10,7 +10,7 @@ namespace Model
     public class Game : INotifyPropertyChanged, IEquatable<Game>
     {
         [DataMember]
-        public string? Name
+        public string Name
         {
             get => name;
             private set
diff --git a/Sources/Stim.Model/Manager.cs b/Sources/Stim.Model/Manager.cs
index 75c1608..da4e746 100644
--- a/Sources/Stim.Model/Manager.cs
+++ b/Sources/Stim.Model/Manager.cs
@@ -12,6 +12,12 @@ namespace Model
         }
         private IPersistance mgrpersistance;
         public ObservableCollection<Game> GameList { get;}
+
+        // dégage ça
+        // fait une méthode qui te renvoie une liste filtrer avec les params (genre ton nom :D)
+        // Quand t'ajoute peut être que dasn un certains monde
+        // il y aura pas le pb car c'est le retour d'une collection Obs
+        // donc potentiellement si les astres sont alignés ça devrait la mettre à jour
         public ObservableCollection<Game> ResearchedGame { get; set; }
         public User CurrentUser { get; set; }
         public HashSet<User> Users { get; set; }
@@ -29,6 +35,22 @@ namespace Model
             }
         }
 
+        public IEnumerable<Game> FilterGames(string? filterName, string? filterTag1, string? filterTag2)
+        {
+            IEnumerable<Game> retList;
+            retList = GameList;
+            if (filterName != null) retList = retList
+                .Where(game => game.Name.IndexOf(filterName, StringComparison.OrdinalIgnoreCase) >= 0
+                );
+            if (filterTag1 != null) retList = retList
+                .Where(game => game.Tags.Any(tag => tag.IndexOf(filterTag1, StringComparison.OrdinalIgnoreCase) >= 0)
+                );
+            if (filterTag2 != null) retList = retList
+                .Where(game => game.Tags.Any(tag => tag.IndexOf(filterTag2, StringComparison.OrdinalIgnoreCase) >= 0)
+                );
+            return retList;
+        }
+
         public void AddGametoGamesList(Game game)
         {
             GameList.Add(game);
diff --git a/Sources/Stim.Model/User.cs b/Sources/Stim.Model/User.cs
index 3fc12a7..25296b0 100644
--- a/Sources/Stim.Model/User.cs
+++ b/Sources/Stim.Model/User.cs
@@ -122,8 +122,9 @@ namespace Model
             return other != null && Username.Equals(other.Username);
         }
         public override int GetHashCode()
-        {
-            return Username.GetHashCode();
+        { 
+            if (Username!=null) return Username.GetHashCode();
+            return 0;
         }
 
         public void AddReview(Game game, float rate, string text)
diff --git a/Sources/Stim/AddGamePage.xaml.cs b/Sources/Stim/AddGamePage.xaml.cs
index db7d2c8..328e8e6 100644
--- a/Sources/Stim/AddGamePage.xaml.cs
+++ b/Sources/Stim/AddGamePage.xaml.cs
@@ -29,8 +29,10 @@ public partial class AddGamePage : ContentPage
         {
             message = "Jeu ajouté !";
             ((App)App.Current).Manager.AddGametoGamesList(new Game(NameEntry.Text, DescriptionEntry.Text, year, new List<string> { TagEntry1.Text, TagEntry2.Text, TagEntry3.Text }, imgName, LinkEntry.Text));
-            Navigation.RemovePage(this);
+            await this.ShowPopupAsync(new MessagePopup(message));
+            await Navigation.PopModalAsync();
             ((App)App.Current).Manager.SaveGames();
+
         }
         //      //if (_ImgPath!=null) NameEntry.Text + ".png";
         //      //System.IO.File.Copy(_ImgPath, /**/, true);
diff --git a/Sources/Stim/Create.xaml b/Sources/Stim/Create.xaml
index a267e1b..bc60331 100644
--- a/Sources/Stim/Create.xaml
+++ b/Sources/Stim/Create.xaml
@@ -22,11 +22,11 @@
         <VerticalStackLayout BackgroundColor="Black" Grid.Column="2" Grid.Row="1"/>
         <Grid Grid.Column="1">
             <Grid.RowDefinitions>
-                <RowDefinition Height="*"/>
-                <RowDefinition Height="*"/>
-                <RowDefinition Height="4*"/>
+                <RowDefinition Height="500"/>
+                <RowDefinition Height="auto"/>
+                <RowDefinition Height="auto"/>
             </Grid.RowDefinitions>
-            <Image Source="no_cover.png"/>
+            <Image Source="no_cover.png" HeightRequest="490" Margin="0,10,0,0"/>
             <Grid Grid.Row="1">
                 <Grid.RowDefinitions>
                     <RowDefinition Height="auto"/>
@@ -40,13 +40,13 @@
                 <Grid Grid.Row="3" Margin="10,0,0,10">
                     <Grid.ColumnDefinitions>
                         <ColumnDefinition Width="*"/>
-                        <ColumnDefinition Width="*"/>
+                        <ColumnDefinition Width="3*"/>
                     </Grid.ColumnDefinitions>
                     <Button Text="Se connecter"    Clicked="Se_connecter"    HeightRequest="50" BackgroundColor="{StaticResource Gray500}"/>
                     <Button Text="Créer un compte" Clicked="Creer_un_compte" HeightRequest="50" BackgroundColor="{StaticResource Gray500}" Grid.Column="1"/>
                 </Grid>
             </Grid>
-            <HorizontalStackLayout x:Name="Error" Grid.Row="2">
+            <HorizontalStackLayout x:Name="Error" Grid.Row="2" HorizontalOptions="Center">
             </HorizontalStackLayout>
         </Grid>
     </Grid>
diff --git a/Sources/Stim/Create.xaml.cs b/Sources/Stim/Create.xaml.cs
index 093aa43..952808c 100644
--- a/Sources/Stim/Create.xaml.cs
+++ b/Sources/Stim/Create.xaml.cs
@@ -1,38 +1,42 @@
 using Microsoft.Maui.Graphics;
-using System.Text.RegularExpressions;
-
-namespace Stim;
-
-public partial class Create : ContentPage
-{
-	public Create()
-	{
-		InitializeComponent();
-	}
-    private async void Creer_un_compte(object sender, EventArgs e)
-    {
-        Error.Clear();
-        if (!string.IsNullOrWhiteSpace(Username.Text) && !string.IsNullOrWhiteSpace(Pswd.Text) && !string.IsNullOrWhiteSpace(Email.Text))
-        {
-            if (((App)App.Current).Manager.SearchUser(Username.Text) == null)
-            {
-                Regex rg = new Regex("^(?=.*[A-Za-z])(?=.*[0-9@$!%*#?&])[A-Za-z-0-9@$!%*#?&]{8,}$");
-                if (rg.IsMatch(Pswd.Text))
-                {
-                    ((App)App.Current).Manager.AddUsertoUserList(new("", Username.Text, "", Email.Text, Pswd.Text));
-                    ((App)App.Current).Manager.CurrentUser = ((App)App.Current).Manager.SearchUser(Username.Text);
-                    Application.Current.MainPage = new AppShell();
-                    await Shell.Current.GoToAsync("//MainPage");
-                }
-                else Error.Children.Add(new Label { Text = "Mot de passe invalide, votre mot de passe doit contenir une Majuscule, une minuscule, un chiffre et faire au moins 8 caractères", TextColor = Colors.Red, VerticalTextAlignment = TextAlignment.Start });
+using System.Text.RegularExpressions;
+
+namespace Stim;
+
+public partial class Create : ContentPage
+{
+	public Create()
+	{
+		InitializeComponent();
+	}
+    private async void Creer_un_compte(object sender, EventArgs e)
+    {
+        Error.Clear();
+        if (!string.IsNullOrWhiteSpace(Username.Text) || !string.IsNullOrWhiteSpace(Pswd.Text) || !string.IsNullOrWhiteSpace(Email.Text))
+        {
+            if (((App)App.Current).Manager.SearchUser(Username.Text) == null)
+            {
+                Regex rg = new Regex("^(?=.*[A-Za-z])(?=.*[0-9@$!%*#?&])[A-Za-z-0-9@$!%*#?&]{8,}$");
+                if (rg.IsMatch(Pswd.Text))
+                {
+                    ((App)App.Current).Manager.AddUsertoUserList(new("", Username.Text, "", Email.Text, Pswd.Text));
+                    ((App)App.Current).Manager.CurrentUser = ((App)App.Current).Manager.SearchUser(Username.Text);
+                    Application.Current.MainPage = new AppShell();
+                    await Shell.Current.GoToAsync("//MainPage");
+                }
+                else Error.Children.Add(new Label { Text = "Mot de passe invalide, votre mot de passe doit contenir une Majuscule, une minuscule, un chiffre et faire au moins 8 caractères", TextColor = Colors.Red, VerticalTextAlignment = TextAlignment.Start });
+                rg = new Regex("^[\\w-\\.]+@([\\w-]+\\.)+[\\w-]{2,4}$");
+                /*if ( rg.IsMatch )
+                {
+
+                }*/
             }
-            else Error.Children.Add(new Label { Text = "Ce nom d'utilisateur est déjà pris", TextColor = Colors.Red, VerticalTextAlignment = TextAlignment.Start });
-        }
-        else Error.Children.Add(new Label { Text = "Champs vides", TextColor = Colors.Red, VerticalTextAlignment = TextAlignment.Start });
-    }
-    private void Se_connecter(object sender, EventArgs e)
+            else Error.Children.Add(new Label { Text = "Ce nom d'utilisateur est déjà pris", TextColor = Colors.Red, VerticalTextAlignment = TextAlignment.Start });
+        }
+        else Error.Children.Add(new Label { Text = "Champs vides", TextColor = Colors.Red, VerticalTextAlignment = TextAlignment.Start });
+    }
+    private void Se_connecter(object sender, EventArgs e)
     {
-        //Ca ça marche pas faut une autre commande, Marc svp aide moi
-        Application.Current.MainPage = new LoginPage();
-    }
+        Application.Current.MainPage = new LoginPage();
+    }
 }
\ No newline at end of file
diff --git a/Sources/Stim/EntryPopup.xaml b/Sources/Stim/EntryPopup.xaml
index 2a0b30b..7c9137b 100644
--- a/Sources/Stim/EntryPopup.xaml
+++ b/Sources/Stim/EntryPopup.xaml
@@ -5,10 +5,11 @@
              x:Class="Stim.EntryPopup"
                CanBeDismissedByTappingOutsideOfPopup="False">
     <VerticalStackLayout>
-        <Grid ColumnDefinitions="*" RowDefinitions="auto, auto, auto">
+        <Grid ColumnDefinitions="*" RowDefinitions="auto, auto, auto,auto">
             <Label x:Name="LabelTxt"/>
-            <Entry Grid.Row="1"/>
-            <Button Grid.Row="2" x:Name="Entrytxt" Text="Valider" HorizontalOptions="Center" VerticalOptions="End" Background="{StaticResource Transparent}" Clicked="CloseButton"/>
+            <Entry Grid.Row="1" x:Name="Entrytxt"/>
+            <Button Grid.Row="2" Text="Valider" HorizontalOptions="Center" VerticalOptions="End" Background="{StaticResource Transparent}" Clicked="Valider"/>
+            <Button Grid.Row="3" Text="Annuler" HorizontalOptions="Center" VerticalOptions="End" Background="{StaticResource Transparent}" Clicked="CloseButton"/>
         </Grid>
     </VerticalStackLayout>
 </toolkit:Popup>
diff --git a/Sources/Stim/EntryPopup.xaml.cs b/Sources/Stim/EntryPopup.xaml.cs
index 05fed6b..41b0c6e 100644
--- a/Sources/Stim/EntryPopup.xaml.cs
+++ b/Sources/Stim/EntryPopup.xaml.cs
@@ -14,4 +14,9 @@ public partial class EntryPopup : Popup
     {
         if (!string.IsNullOrWhiteSpace(Entrytxt.Text)) Close(Entrytxt.Text);
     }
+
+    private void Valider(object sender, EventArgs e)
+    {
+
+    }
 }
\ No newline at end of file
diff --git a/Sources/Stim/LoginPage.xaml b/Sources/Stim/LoginPage.xaml
index 7d6b4ab..c3be651 100644
--- a/Sources/Stim/LoginPage.xaml
+++ b/Sources/Stim/LoginPage.xaml
@@ -22,30 +22,47 @@
         <VerticalStackLayout BackgroundColor="Black" Grid.Column="2" Grid.Row="1"/>
         <Grid Grid.Column="1">
             <Grid.RowDefinitions>
-                <RowDefinition Height="*"/>
-                <RowDefinition Height="*"/>
-                <RowDefinition Height="4*"/>
+                <RowDefinition Height="500"/>
+                <RowDefinition Height="auto"/>
+                <RowDefinition Height="auto"/>
             </Grid.RowDefinitions>
-            <Image Source="no_cover.png"/>
+            <Image Source="no_cover.png" HeightRequest="490" Margin="0,10,0,0"/>
             <Grid Grid.Row="1">
                 <Grid.RowDefinitions>
                     <RowDefinition Height="auto"/>
                     <RowDefinition Height="auto"/>
                     <RowDefinition Height="auto"/>
                 </Grid.RowDefinitions>
-                <Entry Placeholder="Username" PlaceholderColor="{StaticResource Primary}" IsPassword="False" x:Name="Username" HeightRequest="50" ClearButtonVisibility="WhileEditing"/>
-                <Entry Placeholder="Mot de passe" PlaceholderColor="{StaticResource Primary}" IsPassword="True" x:Name="Pswd" HeightRequest="50" Grid.Row="1" ClearButtonVisibility="WhileEditing"/>
-                <Grid Grid.Row="2" Margin="10,0,0,10">
+                <Entry Placeholder="Username" PlaceholderColor="{StaticResource Primary}"
+                       IsPassword="False"
+                       x:Name="Username"
+                       HeightRequest="50"
+                       ClearButtonVisibility="WhileEditing"/>
+                <Entry Placeholder="Mot de passe"
+                       PlaceholderColor="{StaticResource Primary}"
+                       IsPassword="True" x:Name="Pswd"
+                       HeightRequest="50"
+                       Grid.Row="1"
+                       ClearButtonVisibility="WhileEditing"/>
+
+                <Grid Grid.Row="2"
+                      Margin="10,0,0,10">
                     <Grid.ColumnDefinitions>
-                        <ColumnDefinition Width="*"/>
+                        <ColumnDefinition Width="3*"/>
                         <ColumnDefinition Width="*"/>
                     </Grid.ColumnDefinitions>
-                    <Button Text="Se connecter" Clicked="Se_connecter" HeightRequest="50" BackgroundColor="{StaticResource Gray500}"/>
-                    <Button Text="Créer un compte" Clicked="Creer_un_compte" HeightRequest="50" BackgroundColor="{StaticResource Gray500}" Grid.Column="1"/>
+                    <Button Text="Se connecter"
+                            Clicked="Se_connecter"
+                            HeightRequest="50"
+                            BackgroundColor="{StaticResource Gray500}"/>
+                    <Button Text="Créer un compte"
+                            Clicked="Creer_un_compte"
+                            HeightRequest="50"
+                            BackgroundColor="{StaticResource Gray500}"
+                            Grid.Column="1"/>
                 </Grid>
             </Grid>
-            <HorizontalStackLayout x:Name="Error" Grid.Row="2">
-                
+            <HorizontalStackLayout x:Name="Error" Grid.Row="2" HorizontalOptions="Center">
             </HorizontalStackLayout>
         </Grid>
     </Grid>
diff --git a/Sources/Stim/LoginPage.xaml.cs b/Sources/Stim/LoginPage.xaml.cs
index 84557dd..9a67433 100644
--- a/Sources/Stim/LoginPage.xaml.cs
+++ b/Sources/Stim/LoginPage.xaml.cs
@@ -12,7 +12,8 @@ public partial class LoginPage : ContentPage
 
     private async void Se_connecter(object sender, EventArgs e)
     {
-		if (!string.IsNullOrWhiteSpace(Username.Text) && !string.IsNullOrWhiteSpace(Pswd.Text))
+        Error.Clear();
+        if (!string.IsNullOrWhiteSpace(Username.Text) || !string.IsNullOrWhiteSpace(Pswd.Text))
 		{
 			User user = ((App)App.Current).Manager.SearchUser(Username.Text);
             if (user != null)
@@ -23,13 +24,20 @@ public partial class LoginPage : ContentPage
                     Application.Current.MainPage = new AppShell();
                     await Shell.Current.GoToAsync("//MainPage");
                 }
-
-                else Error.Children.Add(new Label { Text = "Mot de passe incorrect", TextColor = Colors.Red, VerticalTextAlignment = TextAlignment.Start });
+                else Error.Children.Add(new Label { Text = "Mot de passe incorrect",
+                    TextColor = Colors.Red,
+                    VerticalTextAlignment = TextAlignment.Start,
+                    HorizontalTextAlignment = TextAlignment.Start });
             }
-            else Error.Children.Add(new Label { Text = "Information incorrecte", TextColor = Colors.Red, VerticalTextAlignment = TextAlignment.Start });
+            else Error.Children.Add(new Label { Text = "Information incorrecte",
+                TextColor = Colors.Red,
+                VerticalTextAlignment = TextAlignment.Start,
+                HorizontalTextAlignment = TextAlignment.Start});
 		}
-        else Error.Children.Add(new Label { Text = "Champs vides", TextColor = Colors.Red, VerticalTextAlignment = TextAlignment.Start });
-
+        else Error.Children.Add(new Label { Text = "Champs vides",
+            TextColor = Colors.Red, 
+            VerticalTextAlignment = TextAlignment.Start,
+            HorizontalTextAlignment=TextAlignment.Start});
     }
 
     private void Creer_un_compte(object sender, EventArgs e)
diff --git a/Sources/Stim/MainPage.xaml b/Sources/Stim/MainPage.xaml
index 65ae940..638e98f 100644
--- a/Sources/Stim/MainPage.xaml
+++ b/Sources/Stim/MainPage.xaml
@@ -39,7 +39,7 @@
                     <SearchBar x:Name="Tag2" TextChanged="SearchBar_GameChanged" Grid.Column="1" Grid.Row="1" Placeholder="Tag 2" WidthRequest="200" HorizontalOptions="Start" Margin="5"/>
                 </Grid>
 
-                <CollectionView ItemsSource="{Binding ResearchedGame}" SelectionMode="Single" SelectionChanged="OnClickGameList" ItemsLayout="VerticalGrid, 5"  Grid.Column="1" Grid.Row="1">
+                <CollectionView ItemsSource="{Binding}" SelectionMode="Single" SelectionChanged="OnClickGameList" ItemsLayout="VerticalGrid, 5"  Grid.Column="1" Grid.Row="1">
                     <CollectionView.ItemTemplate>
                         <DataTemplate>
                             <Border MinimumWidthRequest="200" Margin="10, 10, 10, 10">
diff --git a/Sources/Stim/MainPage.xaml.cs b/Sources/Stim/MainPage.xaml.cs
index 29cfc33..b5223e7 100644
--- a/Sources/Stim/MainPage.xaml.cs
+++ b/Sources/Stim/MainPage.xaml.cs
@@ -9,10 +9,12 @@ using System.Collections.Generic;
 
 public partial class MainPage : ContentPage
 {
+    public IEnumerable<Game> filterdGame { get;  private set; }
+
     public MainPage()
     {
         InitializeComponent();
-        BindingContext = ((App)App.Current).Manager;
+        BindingContext = ((App)App.Current).Manager.FilterGames(null,null,null);
     }
 
     private async void OnClickGameList(object sender, EventArgs e)
@@ -22,7 +24,7 @@ public partial class MainPage : ContentPage
 
     private async void GoToAddGamePage(object sender, EventArgs e)
     {
-        await Navigation.PushAsync(new AddGamePage());
+        await Navigation.PushModalAsync(new AddGamePage());
     }
 
     private void SearchBar_GameChanged(object sender, TextChangedEventArgs e)
@@ -30,21 +32,7 @@ public partial class MainPage : ContentPage
         string GameText = Game.Text;
         string Tag1Text = Tag1.Text;
         string Tag2Text = Tag2.Text;
-        ((App)App.Current).Manager.ResearchedGame.Clear();
-        IEnumerable<Game> filteredGames = ((App)App.Current).Manager.GameList;
-        if (GameText!=null) filteredGames = filteredGames
-            .Where(game => game.Name.IndexOf(GameText, StringComparison.OrdinalIgnoreCase) >= 0
-            );
-        if (Tag1Text != null) filteredGames = filteredGames
-            .Where(game => game.Tags.Any(tag => tag.IndexOf(Tag1Text, StringComparison.OrdinalIgnoreCase) >= 0)
-            );
-        if (Tag2Text != null) filteredGames = filteredGames
-            .Where(game => game.Tags.Any(tag => tag.IndexOf(Tag2Text, StringComparison.OrdinalIgnoreCase) >= 0)
-            );
 
-        foreach (var game in filteredGames)
-        {
-            ((App)App.Current).Manager.ResearchedGame.Add(game);
-        }
+        BindingContext=((App)App.Current).Manager.FilterGames(GameText, Tag1Text, Tag2Text);
     }
 }
diff --git a/Sources/Stim/UserInfo.xaml.cs b/Sources/Stim/UserInfo.xaml.cs
index 128c7f8..2f7655e 100644
--- a/Sources/Stim/UserInfo.xaml.cs
+++ b/Sources/Stim/UserInfo.xaml.cs
@@ -13,7 +13,6 @@ public partial class UserInfo : ContentView
     private static void OnBindChanged(BindableObject bindable, object oldValue, object newValue)
     {
         var contentView = (UserInfo)bindable;
-        contentView.OnBindChanged((string)oldValue, (string)newValue);
     }
 
     public UserInfo()
@@ -22,8 +21,8 @@ public partial class UserInfo : ContentView
         BindingContext = ((App)App.Current).Manager;
     }
 
-    private void OnBindChanged(string oldValue, string newValue)
+    private void Button_Clicked(object sender, EventArgs e)
     {
-        // Réagissez aux changements de la propriété de liaison ici
+
     }
 }