Better cache loading support

Author: thedillonb

CodeHub.Core/ViewModels/ViewModelExtensions.cs

@@ -12,7 +12,7 @@ namespace CodeHub.Core.ViewModels
 {
     public static class ViewModelExtensions
     {
-        public static Task RequestModel<TRequest>(this MvxViewModel viewModel, GitHubRequest<TRequest> request, bool forceDataRefresh, Action<GitHubResponse<TRequest>> update) where TRequest : new()
+        public static async Task RequestModel<TRequest>(this MvxViewModel viewModel, GitHubRequest<TRequest> request, bool forceDataRefresh, Action<GitHubResponse<TRequest>> update) where TRequest : new()
         {
             if (forceDataRefresh)
             {
@@ -23,29 +23,16 @@ public static class ViewModelExtensions
             var application = Mvx.Resolve<IApplicationService>();
             var uiThrad = Mvx.Resolve<IUIThreadService>();
 
-			return Task.Run(async () =>
-            {
-				var result = await application.Client.ExecuteAsync(request).ConfigureAwait(false);
-                uiThrad.MarshalOnUIThread(() => update(result));
-
-                if (result.WasCached)
-                {
-                    request.RequestFromCache = false;
+			var result = await application.Client.ExecuteAsync(request).ConfigureAwait(false);
+            uiThrad.MarshalOnUIThread(() => update(result));
 
-					Task.Run(async () =>
-                    {
-                        try
-                        {
-							var r = await application.Client.ExecuteAsync(request).ConfigureAwait(false);
-                            uiThrad.MarshalOnUIThread(() => update(r));
-                        }
-						catch (NotModifiedException)
-						{
-							System.Diagnostics.Debug.WriteLine("Not modified: " + request.Url);
-						}
-                    }).FireAndForget();
-                }
-            });
+            if (result.WasCached)
+            {
+                request.RequestFromCache = false;
+                var uncachedTask = application.Client.ExecuteAsync(request);
+                uncachedTask.FireAndForget();
+                uncachedTask.ContinueWith(t => uiThrad.MarshalOnUIThread(() => update(t.Result)), TaskContinuationOptions.OnlyOnRanToCompletion);
+            }
 		}
 
         public static void CreateMore<T>(this MvxViewModel viewModel, GitHubResponse<T> response, 

CodeHub.iOS/CodeHub.iOS.csproj

@@ -71,12 +71,13 @@
     <WarningLevel>4</WarningLevel>
     <ConsolePause>false</ConsolePause>
     <CodesignKey>iPhone Developer</CodesignKey>
-    <MtouchUseLlvm>true</MtouchUseLlvm>
     <MtouchUseThumb>true</MtouchUseThumb>
     <MtouchI18n>
     </MtouchI18n>
     <MtouchArch>ARMv7</MtouchArch>
     <CodesignEntitlements>Entitlements.plist</CodesignEntitlements>
+    <IpaPackageName>
+    </IpaPackageName>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Ad-Hoc|iPhone' ">
     <DebugType>none</DebugType>
@@ -96,7 +97,6 @@
     <CodesignProvision>Automatic:AdHoc</CodesignProvision>
     <CodesignEntitlements>Entitlements.plist</CodesignEntitlements>
     <DefineConstants>DEBUG</DefineConstants>
-    <MtouchExtraArgs>--linkskip=CodeFramework.Core.iOS --linkskip=CodeFramework.iOS --linkskip=CodeHub.Core.iOS --linkskip=CodeHub.iOS</MtouchExtraArgs>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'AppStore|iPhone' ">
     <DebugType>none</DebugType>
@@ -107,7 +107,6 @@
     <ConsolePause>False</ConsolePause>
     <CodesignKey>iPhone Distribution</CodesignKey>
     <CodesignProvision>Automatic:AppStore</CodesignProvision>
-    <MtouchUseLlvm>true</MtouchUseLlvm>
     <MtouchUseThumb>true</MtouchUseThumb>
     <IpaPackageName>
     </IpaPackageName>