From: gu.martinm@gmail.com Date: Wed, 3 Sep 2014 12:45:28 +0000 (+0200) Subject: WeatherInformaion WP8: TODO/doubts X-Git-Url: https://git.gumartinm.name/?p=CSharpForFun%2F.git;a=commitdiff_plain;h=12710e729d2d4cbdbe6e4425d81b8234742ff603 WeatherInformaion WP8: TODO/doubts --- diff --git a/WindowsPhone/WP8/WeatherInformation/WeatherInformation/MapPage.xaml.cs b/WindowsPhone/WP8/WeatherInformation/WeatherInformation/MapPage.xaml.cs index d7a6800..2f77793 100644 --- a/WindowsPhone/WP8/WeatherInformation/WeatherInformation/MapPage.xaml.cs +++ b/WindowsPhone/WP8/WeatherInformation/WeatherInformation/MapPage.xaml.cs @@ -86,6 +86,7 @@ namespace WeatherInformation Exception errorException = eventData.Error; if (errorException != null) { + // TODO: if user changed the page, where is this going to appear? MessageBox.Show( AppResources.NoticeErrorLocationAutodetection, AppResources.UnavailableAutomaticCurrentLocationMessageBox, @@ -95,6 +96,10 @@ namespace WeatherInformation { if (eventData.Result.Count > 0) { + // TODO: Should I call UpdateMap even if there are not results? + // I could use country and city default values in that case... + // Problem this method: requires GeoCoordinate and I wouldn't have it here... + // Somehow this method should take them... MapAddress address = eventData.Result[0].Information.Address; GeoCoordinate currentGeoCoordinate = eventData.Result[0].GeoCoordinate; @@ -242,6 +247,7 @@ namespace WeatherInformation catch (Exception ex) { // TODO: make sure when exception in GetCurrentLocationAndUpdateMap we catch it here. + // TODO: if user changed the page, where is this going to appear? MessageBox.Show( AppResources.NoticeErrorLocationAutodetection, AppResources.UnavailableAutomaticCurrentLocationMessageBox,