Updated PlayerExtractor method GetCurrentClubName() - do not assume that

the current club is first in the list, use the leftDate to identify instead.
MikeHook · May 2, 2011 · 4c50fe9 · 4c50fe9
1 parent 0c2296a
commit 4c50fe9
Show file tree

Hide file tree

Showing 2 changed files with 739 additions and 6 deletions.
diff --git a/CSharpApp/FootyLinks.Processes/PlayerExtractor.cs b/CSharpApp/FootyLinks.Processes/PlayerExtractor.cs
@@ -84,13 +84,10 @@ public string GetCurrentClubName()
 			if (_playerClubs.Any() == false)
 				return null;
 
-			PlayerClubDto mostRecentClub = _playerClubs.First();
+			PlayerClubDto mostRecentClub = _playerClubs
+											.SingleOrDefault(c => c.PlayerLeftDate.HasValue == false);
 
-			//There is a leaving date for their most recent club so they must be retired
-			if (mostRecentClub.PlayerLeftDate != null)
-				return null;
-
-			return mostRecentClub.ClubName;
+			return mostRecentClub != null ? mostRecentClub.ClubName : null;			
 		}
 
 		public IList<string> GetFormerClubs()