Permalink
Browse files

BUGFIX "Sync files" button doesn't work when syncing the root of the …

…assets folder
  • Loading branch information...
1 parent 04e5ff0 commit 72fd0128c441a6733ddb2cee932eb46a3d082acc @halkyon halkyon committed Jun 14, 2012
Showing with 9 additions and 5 deletions.
  1. +9 −5 filesystem/Folder.php
View
@@ -128,13 +128,17 @@ function syncChildren() {
}
}
$unwantedDbChildren = $hasDbChild;
-
-
+
+ // if we're syncing a folder with no ID, we assume we're syncing the root assets folder
+ // however the Filename field is populated with "NewFolder", so we need to set this to empty
+ // to satisfy the baseDir variable below, which is the root folder to scan for new files in
+ if(!$parentID) $this->Filename = '';
+
// Iterate through the actual children, correcting the database as necessary
$baseDir = $this->FullPath;
-
- if(!$this->Filename) die($this->ID . " - " . $this->FullPath);
-
+
+ // @todo this shouldn't call die() but log instead
+ if($parentID && !$this->Filename) die($this->ID . " - " . $this->FullPath);
if(file_exists($baseDir)) {
$actualChildren = scandir($baseDir);

0 comments on commit 72fd012

Please sign in to comment.