mbrucedogs/mvm_core_ui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'bugfix/CXTDT-579049' into 'release/20_0_3'

Browse Source 
Digital PCT265 defect CXTDT-579049: Index out of bounds crash fixes.

### Summary
Array indexing safety for mid property updates.

### JIRA Ticket
https://onejira.verizon.com/browse/CXTDT-579049

See merge request https://gitlab.verizon.com/BPHV_MIPS/mvm_core_ui/-/merge_requests/1138
This commit is contained in:
Hedden, Kyle Matthew 2024-06-28 19:58:59 +00:00
commit 3092804f7c
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
MVMCoreUI/Atomic/Molecules/Items/TabsListItemModel.swift
View File

@ -108,8 +108,7 @@ extension TabsListItemModel: AddMolecules {
public func moleculesToAdd() -> AddMolecules.AddParameters? {
guard addedMolecules == nil else { return nil }
let index = tabs.selectedIndex
guard molecules.count >= index else { return nil }
let addedMolecules = molecules[index]
guard let addedMolecules = molecules[safe: index] else { return nil }
self.addedMolecules = addedMolecules
return (addedMolecules, .below)
}

2
MVMCoreUI/Atomic/Organisms/Carousel/Carousel.swift
View File

@ -361,7 +361,7 @@ open class Carousel: View {
}
func trackSwipeActionAnalyticsforIndex(_ index : Int){
guard let itemModel = molecules?[index],
guard let itemModel = molecules?[safe:index],
let viewControllerObject = delegateObject?.moleculeDelegate as? MVMCoreViewControllerProtocol else { return }
MVMCoreUILoggingHandler.shared()?.defaultLogAction(forController: viewControllerObject, actionInformation: itemModel.toJSON(), additionalData: nil)
}