Apply suggestions by Sabberworm

Author: raxbg

lib/Sabberworm/CSS/Parser.php

@@ -485,19 +485,20 @@ private function parseNumericValue($bForColor = false) {
 
 	private function parseLineNameValue() {
 		$this->consume('[');
-		$sName = '';
-		try {
-			do {
-				$this->consumeWhiteSpace();
-				$sName .= $this->parseIdentifier(false, true) . ' ';
-			} while (!$this->comes(']'));
-		} catch (UnexpectedTokenException $e) {
-			if(!$this->oParserSettings->bLenientParsing && (!$sName || !$this->comes(']'))) {// This handles constructs like this [ linename ] in non lenient mode
-				throw $e;
+		$this->consumeWhiteSpace();
+		$aNames = array();
+		do {
+			if($this->oParserSettings->bLenientParsing) {
+				try {
+					$aNames[] = $this->parseIdentifier(false, true);
+				} catch(UnexpectedTokenException $e) {}
+			} else {
+				$aNames[] = $this->parseIdentifier(false, true);
 			}
-		}
+			$this->consumeWhiteSpace();
+		} while (!$this->comes(']'));
 		$this->consume(']');
-		return new LineName(rtrim($sName), $this->iLineNo);
+		return new LineName($aNames, $this->iLineNo);
 	}
 
 	private function parseColorValue() {

lib/Sabberworm/CSS/Value/LineName.php

@@ -2,28 +2,17 @@
 
 namespace Sabberworm\CSS\Value;
 
-class LineName extends PrimitiveValue {
-	private $sName;
-
-	public function __construct($sName, $iLineNo = 0) {
-		parent::__construct($iLineNo);
-		$this->sName = $sName;
-	}
-
-	public function setName($sName) {
-		$this->sName = $sName;
-	}
-
-	public function getName() {
-		return $this->sName;
+class LineName extends ValueList {
+	public function __construct($aComponents = array(), $iLineNo = 0) {
+		parent::__construct($aComponents, ' ', $iLineNo);
 	}
 
 	public function __toString() {
 		return $this->render(new \Sabberworm\CSS\OutputFormat());
 	}
 
 	public function render(\Sabberworm\CSS\OutputFormat $oOutputFormat) {
-		return "[{$this->sName}]";
+		return '[' . parent::render(\Sabberworm\CSS\OutputFormat::createCompact()) . ']';
 	}
 
 }