From: Paul DiPietro <paul.dipietro@microsoft.com>
Date: Wed, 1 Feb 2017 16:52:40 +0000 (-0600)
Subject: [C] Update SelectedItem prior to event invoke
X-Git-Tag: beta-2.3.4-pre4~6
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f1505ab5cad20f45358415afe868906a5d59e584;p=platform%2Fupstream%2Fxamarin-forms.git

[C] Update SelectedItem prior to event invoke
---

diff --git a/Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Bugzilla51642.xaml b/Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Bugzilla51642.xaml
index e7318b6..8526144 100644
--- a/Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Bugzilla51642.xaml
+++ b/Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Bugzilla51642.xaml
@@ -2,16 +2,15 @@
 <ContentPage xmlns="http://xamarin.com/schemas/2014/forms"
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              x:Class="Xamarin.Forms.Controls.Issues.Bugzilla51642">
-  <StackLayout>
-    
-  <Picker>
-    <Picker.Items>
-      <x:String>1</x:String>
-      <x:String>2</x:String>
-      <x:String>3</x:String>
-    </Picker.Items>
-  </Picker>
-    <Picker ItemsSource="{Binding Items}"/>
-
-  </StackLayout>
+	<StackLayout>
+		<Picker>
+			<Picker.Items>
+				<x:String>1</x:String>
+				<x:String>2</x:String>
+				<x:String>3</x:String>
+			</Picker.Items>
+		</Picker>
+		<Picker x:Name="BoundPicker" ItemsSource="{Binding Items}"/>
+		<Label x:Name="SelectedItemLabel"/>
+	</StackLayout>
 </ContentPage>
\ No newline at end of file
diff --git a/Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Bugzilla51642.xaml.cs b/Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Bugzilla51642.xaml.cs
index ab16428..9bf210d 100644
--- a/Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Bugzilla51642.xaml.cs
+++ b/Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Bugzilla51642.xaml.cs
@@ -18,6 +18,10 @@ namespace Xamarin.Forms.Controls.Issues
 		{
 			InitializeComponent ();
             LoadDelayedVM();
+			BoundPicker.SelectedIndexChanged += (s, e) =>
+			{
+				SelectedItemLabel.Text = BoundPicker.SelectedItem.ToString();
+			};
 		}
 
         public async void LoadDelayedVM()
diff --git a/Xamarin.Forms.Core/Picker.cs b/Xamarin.Forms.Core/Picker.cs
index ae17969..ad85f48 100644
--- a/Xamarin.Forms.Core/Picker.cs
+++ b/Xamarin.Forms.Core/Picker.cs
@@ -181,8 +181,8 @@ namespace Xamarin.Forms
 		static void OnSelectedIndexChanged(object bindable, object oldValue, object newValue)
 		{
 			var picker = (Picker)bindable;
-			picker.SelectedIndexChanged?.Invoke(bindable, EventArgs.Empty);
 			picker.UpdateSelectedItem();
+			picker.SelectedIndexChanged?.Invoke(bindable, EventArgs.Empty);
 		}
 
 		static void OnSelectedItemChanged(BindableObject bindable, object oldValue, object newValue)