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

Merge branch 'bugfix/same_json_equatable' into 'develop'

Browse Source 
Bugfix/same json equatable

See merge request BPHV_MIPS/mvm_core_ui!752
This commit is contained in:
Pfeil, Scott Robert 2021-08-12 17:57:36 +00:00
commit ee574adc8c
5 changed files with 20 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
MVMCoreUI/Atomic/Molecules/HorizontalCombinationViews/ImageHeadlineBodyModel.swift
View File

@ -8,7 +8,7 @@
import Foundation
public struct ImageHeadlineBodyModel: MoleculeModelProtocol {
public class ImageHeadlineBodyModel: MoleculeModelProtocol {
//--------------------------------------------------
// MARK: - Properties
//--------------------------------------------------

2
MVMCoreUI/Atomic/Molecules/LeftRightViews/ActionDetailWithImageModel.swift
View File

@ -8,7 +8,7 @@
import Foundation
public struct ActionDetailWithImageModel: MoleculeModelProtocol {
public class ActionDetailWithImageModel: MoleculeModelProtocol {
public static var identifier: String = "actionDetailWithImage"
public var moleculeName: String = ActionDetailWithImageModel.identifier
public var backgroundColor: Color?

2
MVMCoreUI/Atomic/Molecules/LeftRightViews/ToggleMolecules/HeadlineBodyLinkToggleModel.swift
View File

@ -7,7 +7,7 @@
//
import Foundation
public struct HeadlineBodyLinkToggleModel: MoleculeModelProtocol {
public class HeadlineBodyLinkToggleModel: MoleculeModelProtocol {
public static var identifier: String = "headlineBodyLinkToggle"
public var moleculeName: String = HeadlineBodyLinkToggleModel.identifier
public var backgroundColor: Color?

2
MVMCoreUI/Atomic/Molecules/VerticalCombinationViews/HeadlineBodyLinkModel.swift
View File

@ -9,7 +9,7 @@
import Foundation
public struct HeadlineBodyLinkModel: MoleculeModelProtocol {
public class HeadlineBodyLinkModel: MoleculeModelProtocol {
//--------------------------------------------------
// MARK: - Properties
//--------------------------------------------------

24
MVMCoreUI/Atomic/Templates/MoleculeListTemplate.swift
View File

@ -188,10 +188,8 @@ open class MoleculeListTemplate: ThreeLayerTableViewController, TemplateProtocol
open func newData(for molecule: MoleculeModelProtocol) {
//TODO: expand for header, navigation, etc
let json = molecule.toJSON()
guard let index = moleculesInfo?.firstIndex(where: { (moleculeInfo) -> Bool in
//TODO: check for molecule protocol eqaulity
if json == moleculeInfo.molecule.toJSON() {
if equal(moleculeA: molecule, moleculeB: moleculeInfo.molecule) {
return true
} else if let parent = moleculeInfo.molecule as? ParentMoleculeModelProtocol {
// Get all molecules of the same type for faster check.
@ -201,8 +199,8 @@ open class MoleculeListTemplate: ThreeLayerTableViewController, TemplateProtocol
}
return accumulator
}
for molecule in molecules {
if json == molecule.toJSON() {
for moleculeB in molecules {
if equal(moleculeA: molecule, moleculeB: moleculeB) {
return true
}
}
@ -283,6 +281,18 @@ open class MoleculeListTemplate: ThreeLayerTableViewController, TemplateProtocol
return modules
}
/// Checks if the two molecules are equal
private func equal(moleculeA: MoleculeModelProtocol, moleculeB: MoleculeModelProtocol) -> Bool {
// TODO: move this to a better approach, maybe a UUID for each model.
// Do instance check
if let classMoleculeA = moleculeA as? NSObjectProtocol,
let classMoleculeB = moleculeB as? NSObjectProtocol {
return classMoleculeA === classMoleculeB
}
// Do json check
return moleculeA.toJSON() == moleculeB.toJSON()
}
}
extension MoleculeListTemplate: MoleculeListProtocol {
@ -320,9 +330,7 @@ extension MoleculeListTemplate: MoleculeListProtocol {
open func getIndexPath(for molecule: ListItemModelProtocol & MoleculeModelProtocol) -> IndexPath? {
guard let index = moleculesInfo?.firstIndex(where: { (moleculeInfo) -> Bool in
//TODO: check for molecule protocol eqaulity
let json = moleculeInfo.molecule.toJSON()
return json == molecule.toJSON()
return equal(moleculeA: molecule, moleculeB: moleculeInfo.molecule)
}) else { return nil }
return IndexPath(row: index, section: 0)