Refactor recursiveDelete

The recursiveDelete method causes problems in certain environments with the RecursiveDirectoryIterator not iterating through all elements of a tree when deleting nodes while iterating.
This is solved by first completing the iteration and only then deleting all elements.

Apply refactoring of recursiveDelete also to lib/Updater.php

Remove newline to make integration test happy

Now really make the functions the same in index.php and Updater.php

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

index.php

@@ -828,10 +828,23 @@ private function recursiveDelete($folder) {
 			\RecursiveIteratorIterator::CHILD_FIRST
 		);
 
+		$directories = array();
+		$files = array();
 		foreach ($iterator as $fileInfo) {
-			$action = $fileInfo->isDir() ? 'rmdir' : 'unlink';
-			$action($fileInfo->getRealPath());
+			if ($fileInfo->isDir()) {
+				$directories[] = $fileInfo->getRealPath();
+			} else {
+				$files[] = $fileInfo->getRealPath();
+			}
+		}
+
+		foreach ($files as $file) {
+			unlink($file);
+		}
+		foreach ($directories as $dir) {
+			rmdir($dir);	
 		}
+
 		$state = rmdir($folder);
 		if($state === false) {
 			throw new \Exception('Could not rmdir ' . $folder);

lib/Updater.php

@@ -713,10 +713,23 @@ private function recursiveDelete($folder) {
 			\RecursiveIteratorIterator::CHILD_FIRST
 		);
 
+		$directories = array();
+		$files = array();
 		foreach ($iterator as $fileInfo) {
-			$action = $fileInfo->isDir() ? 'rmdir' : 'unlink';
-			$action($fileInfo->getRealPath());
+			if ($fileInfo->isDir()) {
+				$directories[] = $fileInfo->getRealPath();
+			} else {
+				$files[] = $fileInfo->getRealPath();
+			}
+		}
+
+		foreach ($files as $file) {
+			unlink($file);
+		}
+		foreach ($directories as $dir) {
+			rmdir($dir);	
 		}
+
 		$state = rmdir($folder);
 		if($state === false) {
 			throw new \Exception('Could not rmdir ' . $folder);