Skip to content

Commit 3c1f7e3

Browse files
authored
Merge pull request #1146 from SamSmithNZ-dotcom/copilot/fix-camelcase-parameter-name
Fix parameter naming and remove redundant awaits in iTunesController
2 parents 09e7fa0 + 298874c commit 3c1f7e3

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/SamSmithNZ.Web/Controllers/iTunesController.cs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ public class ITunesController : Controller
1111
{
1212
private readonly IITunesServiceApiClient _ServiceApiClient;
1313

14-
public ITunesController(IITunesServiceApiClient ServiceApiClient )
14+
public ITunesController(IITunesServiceApiClient serviceApiClient)
1515
{
16-
_ServiceApiClient = ServiceApiClient;
16+
_ServiceApiClient = serviceApiClient;
1717
}
1818

1919
public async Task<IActionResult> Index()
@@ -25,8 +25,8 @@ public async Task<IActionResult> Index()
2525

2626
return View(new IndexViewModel
2727
{
28-
TopArtists = await topArtistsTask,
29-
Playlists = await playlistsTask
28+
TopArtists = topArtistsTask.Result,
29+
Playlists = playlistsTask.Result
3030
});
3131
}
3232

@@ -40,10 +40,10 @@ public async Task<IActionResult> PlayList(int playlistCode, bool showJustSummary
4040

4141
return View(new PlaylistViewModel
4242
{
43-
Playlist = await playlistTask,
44-
TopArtists = await topArtistsTask,
45-
Movements = await movementsTask,
46-
Tracks = await tracksTask
43+
Playlist = playlistTask.Result,
44+
TopArtists = topArtistsTask.Result,
45+
Movements = movementsTask.Result,
46+
Tracks = tracksTask.Result
4747
});
4848
}
4949

0 commit comments

Comments
 (0)