@dazn/kopytko-framework 1.2.0 → 1.3.2

package/CHANGELOG.md

@@ -1,3 +1,24 @@
+## [1.3.2](https://github.com/getndazn/kopytko-framework/compare/v1.3.1...v1.3.2) (2022-09-01)
+
+
+### Bug Fixes
+
+* Ignoring Invalid elements when calculating elements' index in children array ([#31](https://github.com/getndazn/kopytko-framework/issues/31)) ([73aed6e](https://github.com/getndazn/kopytko-framework/commit/73aed6e2e7ba38b946c81fe18ceecf414f7bbcf2))
+
+## [1.3.1](https://github.com/getndazn/kopytko-framework/compare/v1.3.0...v1.3.1) (2022-08-05)
+
+
+### Bug Fixes
+
+* update dependencies ([#35](https://github.com/getndazn/kopytko-framework/issues/35)) ([7e317e0](https://github.com/getndazn/kopytko-framework/commit/7e317e0850cec411832318bf1491c8375c219cad))
+
+# [1.3.0](https://github.com/getndazn/kopytko-framework/compare/v1.2.0...v1.3.0) (2022-08-01)
+
+
+### Features
+
+* enhanced destroyKopytkoRoot to call destroyKopytko ([#34](https://github.com/getndazn/kopytko-framework/issues/34)) ([c21c2bb](https://github.com/getndazn/kopytko-framework/commit/c21c2bb61bae0f165d2a68341aa1cfb75d13a6b8))
+
 # [1.2.0](https://github.com/getndazn/kopytko-framework/compare/v1.1.2...v1.2.0) (2022-08-01)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dazn/kopytko-framework",
-  "version": "1.2.0",
+  "version": "1.3.2",
   "description": "A modern Roku's Brightscript framework",
   "keywords": [
     "brightscript",
@@ -25,13 +25,13 @@
     "test": "node node_modules/@dazn/kopytko-unit-testing-framework/scripts/test.js"
   },
   "dependencies": {
-    "@dazn/kopytko-utils": "^2.0.1"
+    "@dazn/kopytko-utils": "^2.2.2"
   },
   "devDependencies": {
     "@dazn/eslint-plugin-kopytko": "^2.1.0",
-    "@dazn/kopytko-packager": "^1.2.0",
-    "@dazn/kopytko-unit-testing-framework": "^1.0.3",
-    "eslint": "^8.10.0"
+    "@dazn/kopytko-packager": "^1.2.3",
+    "@dazn/kopytko-unit-testing-framework": "^2.0.1",
+    "eslint": "^8.21.0"
   },
   "license": "MIT",
   "kopytkoModuleDir": "src/",

package/src/components/renderer/KopytkoDiffUtility.brs

@@ -69,21 +69,6 @@ function KopytkoDiffUtility() as Object
     m._diffElementChildren(currentElement.children, newElement.children, newElement.props.id)
   end sub
 
-  ' @private
-  prototype._markElementToBeRemoved = sub (element as Object)
-    m._diffResult.elementsToRemove.push(element.props.id)
-
-    if (element.children = Invalid OR element.children.count() = 0)
-      return
-    end if
-
-    for each child in element.children
-      if (child <> Invalid)
-        m._markElementToBeRemoved(child)
-      end if
-    end for
-  end sub
-
   ' @private
   prototype._diffElementProps = sub (elementId as String, currentProps as Object, newProps as Object)
     for each newProp in newProps
@@ -98,34 +83,49 @@ function KopytkoDiffUtility() as Object
     end for
   end sub
 
-  ' @todo Make it not rerender the whole children tree if they were only reordered
+  ' @todo Support elements reordering
   ' @private
   prototype._diffElementChildren = sub (currentChildren as Object, newChildren as Object, parentElementId = Invalid as Dynamic)
-    if (currentChildren = Invalid)
-      currentChildren = []
-    end if
+    if (newChildren = Invalid) then newChildren = []
 
-    if (newChildren = Invalid)
-      newChildren = []
+    currentChildrenMapped = {}
+    if (currentChildren <> Invalid)
+      for each currentChild in currentChildren
+        if (currentChild <> Invalid)
+          currentChildrenMapped[currentChild.props.id] = currentChild
+        end if
+      end for
     end if
 
-    if (currentChildren.count() > newChildren.count())
-      biggestArray = currentChildren
-    else
-      biggestArray = newChildren
-    end if
+    nonInvalidNewChildIndex = 0
+    for each newChild in newChildren
+      if (newChild <> Invalid)
+        newChild.index = nonInvalidNewChildIndex
+        nonInvalidNewChildIndex++
+        newChild.parentId = parentElementId
 
-    for i = 0 to biggestArray.count() - 1
-      if (newChildren[i] <> Invalid)
-        newChildren[i].index = i
-        newChildren[i].parentId = parentElementId
+        m._diffElement(currentChildrenMapped[newChild.props.id], newChild)
+        currentChildrenMapped.delete(newChild.props.id)
       end if
+    end for
 
-      if (currentChildren[i] <> Invalid)
-        currentChildren[i].index = i
-      end if
+    for each currentChildIdToRemove in currentChildrenMapped
+      m._markElementToBeRemoved(currentChildrenMapped[currentChildIdToRemove])
+    end for
+  end sub
+
+  ' @private
+  prototype._markElementToBeRemoved = sub (element as Object)
+    m._diffResult.elementsToRemove.push(element.props.id)
 
-      m._diffElement(currentChildren[i], newChildren[i])
+    if (element.children = Invalid OR element.children.count() = 0)
+      return
+    end if
+
+    for each child in element.children
+      if (child <> Invalid)
+        m._markElementToBeRemoved(child)
+      end if
     end for
   end sub
 

package/src/components/renderer/kopytkoRoot.brs

@@ -19,6 +19,7 @@ sub destroyKopytkoRoot()
   end for
 
   m._dynamicProps = Invalid
+  destroyKopytko()
 end sub
 
 sub kopytkoRoot_dynamicPropChanged(event as Object)