more reliable return value for Watcher::checkUpdate

Signed-off-by: Robin Appelman <[email protected]>

lib/private/Files/Cache/Watcher.php

@@ -89,7 +89,14 @@ public function checkUpdate($path, $cachedEntry = null) {
 		}
 		if ($cachedEntry === false || $this->needsUpdate($path, $cachedEntry)) {
 			$this->update($path, $cachedEntry);
-			return true;
+
+			if ($cachedEntry === false) {
+				return true;
+			} else {
+				// storage backends can sometimes return false positives, only return true if the scanner actually found a change
+				$newEntry = $this->cache->get($path);
+				return $newEntry->getStorageMTime() > $cachedEntry->getStorageMTime();
+			}
 		} else {
 			return false;
 		}