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

Merge branch 'bugfix/develop-atomic-label-range' into 'develop'

Browse Source 
Digital PCT265 defect CXTDT-536188 - Validation of ranges.

### Summary
Adding NSRange validation for Actions to ensure there isn't an out of bounds crash.

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

Co-authored-by: Matt Bruce <matt.bruce@verizon.com>

See merge request https://gitlab.verizon.com/BPHV_MIPS/mvm_core_ui/-/merge_requests/1084
This commit is contained in:
Hedden, Kyle Matthew 2024-03-20 22:27:30 +00:00
commit 9c36bbf38f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
MVMCoreUI/Atomic/Atoms/Views/Label/Label.swift
View File

@ -421,8 +421,8 @@ extension Label {
/// Underlines the tappable region and stores the tap logic for interation.
private func setTextLinkState(range: NSRange, actionBlock: @escaping ActionBlock) {
guard range.length > 0 else { return }
guard let text, text.isValid(range: range) else { return }
var textLink = ActionLabelAttribute(location: range.location, length: range.length)
textLink.subscriber = textLink
.action