Check if data prop has changed when new content is received (#56)

* Fix diff issue

* Changed yarn.lock

Author: Javier Morant

examples/yarn.lock

@@ -392,6 +392,16 @@
     lodash "^4.2.0"
     to-fast-properties "^2.0.0"
 
+"@types/lodash.isequal@^4.5.2":
+  version "4.5.2"
+  resolved "https://registry.yarnpkg.com/@types/lodash.isequal/-/lodash.isequal-4.5.2.tgz#adbdff67f7c956ed703009e5466a34eeddb0b712"
+  dependencies:
+    "@types/lodash" "*"
+
+"@types/lodash@*":
+  version "4.14.106"
+  resolved "https://registry.yarnpkg.com/@types/lodash/-/lodash-4.14.106.tgz#6093e9a02aa567ddecfe9afadca89e53e5dce4dd"
+
 abbrev@1:
   version "1.1.0"
   resolved "https://registry.yarnpkg.com/abbrev/-/abbrev-1.1.0.tgz#d0554c2256636e2f56e7c2e5ad183f859428d81f"
@@ -2872,6 +2882,10 @@ lodash.isarray@^3.0.0:
   version "3.0.4"
   resolved "https://registry.yarnpkg.com/lodash.isarray/-/lodash.isarray-3.0.4.tgz#79e4eb88c36a8122af86f844aa9bcd851b5fbb55"
 
+lodash.isequal@^4.5.0:
+  version "4.5.0"
+  resolved "https://registry.yarnpkg.com/lodash.isequal/-/lodash.isequal-4.5.0.tgz#415c4478f2bcc30120c22ce10ed3226f7d3e18e0"
+
 lodash.keys@^3.0.0:
   version "3.1.2"
   resolved "https://registry.yarnpkg.com/lodash.keys/-/lodash.keys-3.1.2.tgz#4dbc0472b156be50a0b286855d1bd0b0c656098a"
@@ -3700,6 +3714,8 @@ react-native-drawer-layout@1.3.2:
 "react-native-nested-listview@file:..":
   version "0.3.0"
   dependencies:
+    "@types/lodash.isequal" "^4.5.2"
+    lodash.isequal "^4.5.0"
     shortid "^2.2.8"
 
 react-native-safe-area-view@^0.7.0:

package.json

@@ -16,6 +16,8 @@
     "url": "git+https://github.com/fjmorant/react-native-nested-listview.git"
   },
   "dependencies": {
+    "@types/lodash.isequal": "^4.5.2",
+    "lodash.isequal": "^4.5.0",
     "shortid": "^2.2.8"
   },
   "peerDependencies": {
@@ -36,18 +38,23 @@
     "eslint-plugin-react": "^7.4.0",
     "eslint-plugin-react-native": "^3.1.0",
     "flow-bin": "^0.68.0",
-    "jest": "^22.4.2",
     "istanbul": "^0.4.5",
     "istanbul-api": "1.2.2",
     "istanbul-reports": "1.1.4",
+    "jest": "^22.4.2",
     "prettier": "^1.7.0",
     "prettier-eslint": "^8.2.1",
     "react-test-renderer": "16.2.0",
     "ts-jest": "^22.4.1",
     "tslint": "^5.9.1",
     "typescript": "^2.7.2"
   },
-  "keywords": ["react", "native", "list", "nested"],
+  "keywords": [
+    "react",
+    "native",
+    "list",
+    "nested"
+  ],
   "author": "Javier Morant",
   "license": "MIT",
   "bugs": {
@@ -56,7 +63,11 @@
   "homepage": "https://github.com/fjmorant/react-native-nested-listview#readme",
   "jest": {
     "preset": "react-native",
-    "moduleFileExtensions": ["ts", "tsx", "js"],
+    "moduleFileExtensions": [
+      "ts",
+      "tsx",
+      "js"
+    ],
     "transform": {
       "^.+\\.(js)$": "<rootDir>/node_modules/babel-jest",
       "\\.(ts|tsx)$": "<rootDir>/node_modules/ts-jest/preprocessor.js"
@@ -65,7 +76,9 @@
       "node_modules/(?!(jest-)|react-native|react-navigation|react-clone-referenced-element|mobx-react)"
     ],
     "testRegex": "(/__tests__/.*|(\\.|/)(test|spec))\\.(jsx?|tsx?)$",
-    "modulePathIgnorePatterns": ["<rootDir>/examples/"],
+    "modulePathIgnorePatterns": [
+      "<rootDir>/examples/"
+    ],
     "testPathIgnorePatterns": [
       "\\.snap$",
       "<rootDir>/node_modules/",

src/NestedListView.tsx

@@ -1,5 +1,6 @@
 /* @flow */
 
+import isEqual from 'lodash.isequal'
 import * as React from 'react'
 import {StyleSheet, Text, View} from 'react-native'
 import * as shortid from 'shortid'
@@ -42,7 +43,9 @@ export default class NestedListView extends React.PureComponent<
   }
 
   public componentWillReceiveProps(nextProps: any) {
-    this.setState({root: this.generateRootNode(nextProps)})
+    if (!isEqual(this.props.data, nextProps.data)) {
+      this.setState({root: this.generateRootNode(nextProps)})
+    }
   }
 
   public generateIds = (node?: INode) => {

src/NodeView.tsx

@@ -1,5 +1,6 @@
 /* @flow */
 
+import isEqual from 'lodash.isequal'
 import * as React from 'react'
 import {FlatList, TouchableWithoutFeedback, View} from 'react-native'
 
@@ -36,12 +37,14 @@ export default class NodeView extends React.PureComponent<IProps, IState> {
   }
 
   public componentWillReceiveProps(nextProps: IProps) {
-    this.setState({
-      node: {
-        opened: false,
-        ...nextProps.node,
-      },
-    })
+    if (!isEqual(this.props.node, nextProps.node)) {
+      this.setState({
+        node: {
+          opened: false,
+          ...nextProps.node,
+        },
+      })
+    }
   }
 
   public onNodePressed = () => {

yarn.lock

@@ -82,6 +82,16 @@
   version "22.2.0"
   resolved "https://registry.yarnpkg.com/@types/jest/-/jest-22.2.0.tgz#55ce83139f7ad1b48b414c3927746614c6963c0f"
 
+"@types/lodash.isequal@^4.5.2":
+  version "4.5.2"
+  resolved "https://registry.yarnpkg.com/@types/lodash.isequal/-/lodash.isequal-4.5.2.tgz#adbdff67f7c956ed703009e5466a34eeddb0b712"
+  dependencies:
+    "@types/lodash" "*"
+
+"@types/lodash@*":
+  version "4.14.106"
+  resolved "https://registry.yarnpkg.com/@types/lodash/-/lodash-4.14.106.tgz#6093e9a02aa567ddecfe9afadca89e53e5dce4dd"
+
 "@types/react-native@^0.52.17":
   version "0.52.18"
   resolved "https://registry.yarnpkg.com/@types/react-native/-/react-native-0.52.18.tgz#082fada6fdddc6c745a4c1a6234e4aa3b104176b"
@@ -2887,6 +2897,10 @@ locate-path@^2.0.0:
     p-locate "^2.0.0"
     path-exists "^3.0.0"
 
+lodash.isequal@^4.5.0:
+  version "4.5.0"
+  resolved "https://registry.yarnpkg.com/lodash.isequal/-/lodash.isequal-4.5.0.tgz#415c4478f2bcc30120c22ce10ed3226f7d3e18e0"
+
 lodash.merge@^4.6.0:
   version "4.6.1"
   resolved "https://registry.yarnpkg.com/lodash.merge/-/lodash.merge-4.6.1.tgz#adc25d9cb99b9391c59624f379fbba60d7111d54"