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

Merge branch 'bugfix/CXTDT-574791' into 'release/20_0_0'

Browse Source 
Digital PCT265 defect CXTDT-574791: Restore table reload to ensure table is synced to model.

### Summary
Table data not synced properly causing table crashes in some scenarios.

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

Co-authored-by: Hedden, Kyle Matthew <kyle.hedden@verizonwireless.com>

See merge request https://gitlab.verizon.com/BPHV_MIPS/mvm_core_ui/-/merge_requests/1133
This commit is contained in:
Pfeil, Scott Robert 2024-06-19 19:31:07 +00:00
commit 18aca32b43
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
MVMCoreUI/BaseControllers/ThreeLayerTableViewController.swift
View File

@ -52,18 +52,20 @@ open class ThreeLayerTableViewController: ProgrammaticTableViewController, Rotor
bottomView.updateView(width) bottomView.updateView(width)
showFooter(width) showFooter(width)
} }
tableView.reloadData() tableView.visibleCells.forEach { cell in
(cell as? MVMCoreViewProtocol)?.updateView(width)
}
} }
open override func updateUI(for molecules: [MoleculeModelProtocol]? = nil) { open override func updateUI(for molecules: [MoleculeModelProtocol]? = nil) {
let isFirstRender = self.isFirstRender
super.updateUI(for: molecules) super.updateUI(for: molecules)
guard molecules == nil else { return } guard molecules == nil else { return }
createViewForTableHeader() createViewForTableHeader()
createViewForTableFooter() createViewForTableFooter()
// Reloading the table is handled in updateViews. // Reloading the table is handled in updateViews, however, update views is on a separate rendering task than the current thread. The table render needs to be bound and settled to the new model before others put in additional update requests.
tableView.reloadData()
} }
override open func viewDidLoad() { override open func viewDidLoad() {