fix bug cause by null opinion

SilenceDut · SilenceDut · commit d4085eceaed1 · 2019-07-02T10:54:45.000+08:00
diff --git a/diffadapter/src/main/java/com/silencedut/diffadapter/DiffAdapter.java b/diffadapter/src/main/java/com/silencedut/diffadapter/DiffAdapter.java
@@ -519,10 +519,10 @@ public void onBindViewHolder(@NonNull BaseDiffViewHolder holder, int position, @
                         }
                     }
                 }
-                if (diffPayloads.isEmpty() && payloadKeys!=null && payloadKeys.isEmpty()) {
+                if (payloadKeys ==null || (diffPayloads.isEmpty() &&  payloadKeys.isEmpty())) {
                     this.onBindViewHolder(holder, position);
                 } else {
-                    if (payloadKeys!=null && payloadKeys.isEmpty()) {
+                    if (payloadKeys.isEmpty()) {
                         holder.updatePartWithPayload(mDatas.get(position), diffPayloads, position);
                     } else {
                         holder.updatePartWithPayload(mDatas.get(position), payloadKeys, position);

Original file line number	Diff line number	Diff line change
`@@ -519,10 +519,10 @@ public void onBindViewHolder(@NonNull BaseDiffViewHolder holder, int position, @`
`519`	`519`	`}`
`520`	`520`	`}`
`521`	`521`	`}`
`522`		`- if (diffPayloads.isEmpty() && payloadKeys!=null && payloadKeys.isEmpty()) {`
	`522`	`+ if (payloadKeys ==null \|\| (diffPayloads.isEmpty() && payloadKeys.isEmpty())) {`
`523`	`523`	`this.onBindViewHolder(holder, position);`
`524`	`524`	`} else {`
`525`		`- if (payloadKeys!=null && payloadKeys.isEmpty()) {`
	`525`	`+ if (payloadKeys.isEmpty()) {`
`526`	`526`	`holder.updatePartWithPayload(mDatas.get(position), diffPayloads, position);`
`527`	`527`	`} else {`
`528`	`528`	`holder.updatePartWithPayload(mDatas.get(position), payloadKeys, position);`