Skip to content

Commit 15f58eb

Browse files
committed
fixed out of range issue
1 parent 2c50dbe commit 15f58eb

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

Dfe.Academies.Academisation.Service/Mappers/CompleteProjects/CompleteConversionProjectServiceModelMapper.cs

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,10 @@ internal static CreateConversionMatProjectCommand FormAMatFromDomain(IProject pr
3434
{
3535
var assignedUser = project.Details.AssignedUser;
3636

37-
string? fullName = assignedUser?.FullName;
38-
37+
string? fullName = assignedUser?.FullName;
38+
string[] nameParts = fullName?
39+
.Split(' ', StringSplitOptions.RemoveEmptyEntries)
40+
?? [];
3941
return new CreateConversionMatProjectCommand
4042
{
4143
Urn = project.Details.Urn,
@@ -44,8 +46,8 @@ internal static CreateConversionMatProjectCommand FormAMatFromDomain(IProject pr
4446
ProvisionalConversionDate = project.Details.ProposedConversionDate,
4547
DirectiveAcademyOrder = project.Details.AcademyTypeAndRoute?.Equals("Sponsored") ?? false,
4648
CreatedByEmail = assignedUser?.EmailAddress,
47-
CreatedByFirstName = fullName?.Split(' ')[0],
48-
CreatedByLastName = fullName?.Split(' ')[1],
49+
CreatedByFirstName = nameParts.Length > 0 ? nameParts[0] : null,
50+
CreatedByLastName = nameParts.Length > 1 ? nameParts[1] : null,
4951
PrepareId = project.Id,
5052
NewTrustName = project.Details.NameOfTrust,
5153
NewTrustReferenceNumber = project.Details.TrustReferenceNumber,

0 commit comments

Comments
 (0)