Updated cssdoc::save() to return the compiled code instead of true if saved, as it returns the code if no file is specified.

Updated tests.

Author: hexydec

src/cssdoc.php

@@ -456,9 +456,9 @@ public function minify(array $minify = []) : void {
 	}
 
 	/**
-	 * Compile the property to a string
+	 * Compile the document to a string
 	 *
-	 * @param array $options An array of compilation options
+	 * @param array $options An array indicating output options, this is merged with cssdoc::$output
 	 * @return void
 	 */
 	public function compile(array $options = []) : string {
@@ -467,25 +467,23 @@ public function compile(array $options = []) : string {
 	}
 
 	/**
-	 * Compile the document as an HTML string and save it to the specified location
+	 * Compile the document and save it to the specified location
 	 *
-	 * @param array $options An array indicating output options, this is merged with htmldoc::$output
-	 * @return string The compiled HTML
+	 * @param string|null $file The file location to save the document to, or null to just return the compiled code
+	 * @param array $options An array indicating output options, this is merged with cssdoc::$output
+	 * @return string|bool The compiled CSS, or false if the file could not be saved
 	 */
 	public function save(string $file = null, array $options = []) {
 		$css = $this->compile($options);
 
-		// send back as string
-		if (!$file) {
-			return $css;
-
 		// save file
-		} elseif (\file_put_contents($file, $css) === false) {
+		if ($file && \file_put_contents($file, $css) === false) {
 			\trigger_error('File could not be written', E_USER_WARNING);
-		} else {
-			return true;
+			return false;
 		}
-		return false;
+
+		// send back as string
+		return $css;
 	}
 
 	public function collection(array $rules) {

tests/cssdocTest.php

@@ -154,7 +154,7 @@ public function testCanMinifyCss() {
 			// test save method
 			$this->assertEquals($minified, $obj->save());
 			$file = __DIR__.'/test.css';
-			$this->assertEquals(true, $obj->save($file, ['style' => 'beautify']));
+			$this->assertEquals($obj->compile(['style' => 'beautify']), $obj->save($file, ['style' => 'beautify']));
 			$this->assertEquals(true, file_exists($file));
 			unlink($file);
 		}