From 12710e729d2d4cbdbe6e4425d81b8234742ff603 Mon Sep 17 00:00:00 2001 From: "gu.martinm@gmail.com" Date: Wed, 3 Sep 2014 14:45:28 +0200 Subject: [PATCH] WeatherInformaion WP8: TODO/doubts --- .../WP8/WeatherInformation/WeatherInformation/MapPage.xaml.cs | 6 ++++++ 1 file changed, 6 insertions(+) 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, -- 2.1.4