Add option to auto select the highest Ex level of units in unit builder

[kristoffer-henriksson]

No description provided.

[kristoffer-henriksson]

Screenshot of option to toggle auto selecting highest Ex level:

[sailorsben]

// See if they own the unit. We need to look for both the current unit as well as the braveshift to handle the case where they start with the braveshifted unit. const ownedUnit = actuallyOwnedUnits[builds[currentUnitIndex].unit.id] ?? actuallyOwnedUnits[builds[currentUnitIndex].getBraveshift().unit.id]; const exRank = $("#autoSelectHighestExLevel").prop('checked') ? ownedUnit?.exRank ?? 1 : 1; $("#unitExAwakeningLevel select").val(exRank.toString()); builds[currentUnitIndex].setExAwakeningLevel(exRank);

This code doesn't check if actuallyOwnedUnits is defined. If someone doesn't log in, actuallyOwnedUnits never is populated to anything. It also causes the default EX level to be -1, since the code fails before the exRank is set.

[kristoffer-henriksson]

// See if they own the unit. We need to look for both the current unit as well as the braveshift to handle the case where they start with the braveshifted unit. const ownedUnit = actuallyOwnedUnits[builds[currentUnitIndex].unit.id] ?? actuallyOwnedUnits[builds[currentUnitIndex].getBraveshift().unit.id]; const exRank = $("#autoSelectHighestExLevel").prop('checked') ? ownedUnit?.exRank ?? 1 : 1; $("#unitExAwakeningLevel select").val(exRank.toString()); builds[currentUnitIndex].setExAwakeningLevel(exRank);

This code doesn't check if actuallyOwnedUnits is defined. If someone doesn't log in, actuallyOwnedUnits never is populated to anything. It also causes the default EX level to be -1, since the code fails before the exRank is set.

Fixed by checking for assigned actuallyOwnedUnits before attempting to index into it.

[kristoffer-henriksson]

@sailorsben the fix for the problem you pointed you is available and this is ready for another review.