Fix naming conventions and remove redundant awaits in iTunesController

Co-authored-by: samsmithnz <8389039+samsmithnz@users.noreply.github.com>

Author: Copilot

src/SamSmithNZ.Web/Controllers/iTunesController.cs

@@ -11,9 +11,9 @@ public class ITunesController : Controller
     {
         private readonly IITunesServiceApiClient _ServiceApiClient;
 
-        public ITunesController(IITunesServiceApiClient ServiceApiClient  )
+        public ITunesController(IITunesServiceApiClient serviceApiClient)
         {
-            _ServiceApiClient = ServiceApiClient;
+            _ServiceApiClient = serviceApiClient;
         }
 
         public async Task<IActionResult> Index()
@@ -25,8 +25,8 @@ public async Task<IActionResult> Index()
 
             return View(new IndexViewModel
             {
-                TopArtists = await topArtistsTask,
-                Playlists = await playlistsTask
+                TopArtists = topArtistsTask.Result,
+                Playlists = playlistsTask.Result
             });
         }
 
@@ -40,10 +40,10 @@ public async Task<IActionResult> PlayList(int playlistCode, bool showJustSummary
 
             return View(new PlaylistViewModel
             {
-                Playlist = await playlistTask,
-                TopArtists = await topArtistsTask,
-                Movements = await movementsTask,
-                Tracks = await tracksTask
+                Playlist = playlistTask.Result,
+                TopArtists = topArtistsTask.Result,
+                Movements = movementsTask.Result,
+                Tracks = tracksTask.Result
             });
         }