Fix nested conditional

iMattPro · iMattPro · commit aef1a709436c · 2024-11-24T18:40:33.000-08:00
Signed-off-by: Matt Friedman &lt;maf675@gmail.com&gt;
diff --git a/includes/qi_file.php b/includes/qi_file.php
@@ -51,12 +51,9 @@ public static function copy_dir($src_dir, $dst_dir)
 		self::append_slash($src_dir);
 		self::append_slash($dst_dir);
 
-		if (!is_dir($dst_dir))
+		if (!is_dir($dst_dir) && !mkdir($dst_dir) && !is_dir($dst_dir))
 		{
-			if (!mkdir($dst_dir) && !is_dir($dst_dir))
-			{
-				throw new \RuntimeException('COPY_DIR_ERROR');
-			}
+			throw new RuntimeException('COPY_DIR_ERROR');
 		}
 
 		foreach (scandir($src_dir) as $file)

Original file line number	Diff line number	Diff line change
`@@ -51,12 +51,9 @@ public static function copy_dir($src_dir, $dst_dir)`
`51`	`51`	`self::append_slash($src_dir);`
`52`	`52`	`self::append_slash($dst_dir);`
`53`	`53`
`54`		`- if (!is_dir($dst_dir))`
	`54`	`+ if (!is_dir($dst_dir) && !mkdir($dst_dir) && !is_dir($dst_dir))`
`55`	`55`	`{`
`56`		`- if (!mkdir($dst_dir) && !is_dir($dst_dir))`
`57`		`- {`
`58`		`- throw new \RuntimeException('COPY_DIR_ERROR');`
`59`		`- }`
	`56`	`+ throw new RuntimeException('COPY_DIR_ERROR');`
`60`	`57`	`}`
`61`	`58`
`62`	`59`	`foreach (scandir($src_dir) as $file)`