Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 60d10b1f authored by Jonathan Klee's avatar Jonathan Klee
Browse files

Merge branch '762-fix-possible-NPEs' into 'v1-oreo'

Protect code about possible NPEs

See merge request !174
parents 985451c2 b753ad2a
Loading
Loading
Loading
Loading
Loading
+5 −3
Original line number Diff line number Diff line
@@ -146,7 +146,8 @@ public class ListFileRemoteOperation extends RemoteOperation<ArrayList<RemoteFil
        syncedFolder.setToSync(true);

        final File localFolder = new File(syncedFolder.getLocalFolder());
        if (localFolder.listFiles().length == 0) {
        final File[] files = localFolder.listFiles();
        if (files != null && files.length == 0) {
            localFolder.delete();
        }

@@ -166,7 +167,7 @@ public class ListFileRemoteOperation extends RemoteOperation<ArrayList<RemoteFil
     */
    private boolean isHiddenFile(RemoteFile file) {
        final String fileName = CommonUtils.getFileNameFromPath(file.getRemotePath());
        return fileName.startsWith(".");
        return fileName != null && fileName.startsWith(".");
    }

    /**
@@ -175,8 +176,9 @@ public class ListFileRemoteOperation extends RemoteOperation<ArrayList<RemoteFil
     * @return
     */
    private boolean shouldSkipSyncedFolder(SyncedFolder syncedFolder) {
        final String fileName = CommonUtils.getFileNameFromPath(syncedFolder.getRemoteFolder());
        return (syncedFolder.isMediaType()
                && CommonUtils.getFileNameFromPath(syncedFolder.getRemoteFolder()).startsWith("."))
                && fileName != null && fileName.startsWith("."))
                || !syncedFolder.isScanRemote();
    }