Skip to content

Commit 40ba1b8

Browse files
committed
fix spotbugs
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
1 parent c80aec1 commit 40ba1b8

1 file changed

Lines changed: 12 additions & 23 deletions

File tree

app/src/main/java/com/owncloud/android/ui/adapter/OCFileListAdapter.java

Lines changed: 12 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -76,14 +76,12 @@
7676
import java.util.ArrayList;
7777
import java.util.Collections;
7878
import java.util.Date;
79-
import java.util.HashMap;
8079
import java.util.List;
8180
import java.util.Locale;
8281
import java.util.Map;
8382
import java.util.Objects;
8483
import java.util.Set;
8584
import java.util.UUID;
86-
import java.util.function.BiConsumer;
8785

8886
import androidx.annotation.NonNull;
8987
import androidx.annotation.Nullable;
@@ -1067,33 +1065,24 @@ public void removeAllFiles() {
10671065
notifyDataSetChanged();
10681066
}
10691067

1070-
public void removeFileIndicator(OCFile file) {
1071-
if (file != null) {
1072-
file.setFileIndicator(null);
1073-
notifyItemChanged(file);
1074-
}
1075-
}
1076-
10771068
public void updateFileIndicators(Map<Long, FileIndicator> indicators) {
10781069
if (indicators == null || indicators.isEmpty()) {
10791070
return;
10801071
}
10811072

1082-
Map<Long, Integer> positionMap = new HashMap<>(mFiles.size());
1083-
for (int i = 0; i < mFiles.size(); i++) {
1084-
positionMap.put(mFiles.get(i).getFileId(), i);
1085-
}
1073+
for (OCFile file: mFiles) {
1074+
final var fileIndicator = indicators.get(file.getFileId());
1075+
if (fileIndicator == null) {
1076+
continue;
1077+
}
10861078

1087-
indicators.forEach((id, fileIndicator) -> {
1088-
Integer position = positionMap.get(id);
1089-
if (position != null) {
1090-
OCFile file = mFiles.get(position);
1091-
Integer newIndicator = fileIndicator.getIconRes();
1092-
if (!Objects.equals(file.getFileIndicator(), newIndicator)) {
1093-
file.setFileIndicator(newIndicator);
1094-
notifyItemChanged(file);
1095-
}
1079+
final var newIndicator = fileIndicator.getIconRes();
1080+
if (Objects.equals(file.getFileIndicator(), newIndicator)) {
1081+
continue;
10961082
}
1097-
});
1083+
1084+
file.setFileIndicator(newIndicator);
1085+
notifyItemChanged(file);
1086+
}
10981087
}
10991088
}

0 commit comments

Comments
 (0)