Merge pull request #7 from DataValues/options

JeroenDeDauw · JeroenDeDauw · commit c457713a3fa7 · 2014-11-25T13:57:52.000+01:00
Optional options constructor parameter
diff --git a/src/ValueFormatters/ValueFormatterBase.php b/src/ValueFormatters/ValueFormatterBase.php
@@ -22,10 +22,10 @@ abstract class ValueFormatterBase implements ValueFormatter {
 	/**
 	 * @since 0.1
 	 *
-	 * @param FormatterOptions $options
+	 * @param FormatterOptions|null $options
 	 */
-	public function __construct( FormatterOptions $options ) {
-		$this->options = $options;
+	public function __construct( FormatterOptions $options = null ) {
+		$this->options = $options ?: new FormatterOptions();
 
 		$this->options->defaultOption( ValueFormatter::OPT_LANG, 'en' );
 	}

Original file line number	Diff line number	Diff line change
`@@ -22,10 +22,10 @@ abstract class ValueFormatterBase implements ValueFormatter {`
`22`	`22`	`/**`
`23`	`23`	`* @since 0.1`
`24`	`24`	`*`
`25`		`- * @param FormatterOptions $options`
	`25`	`+ * @param FormatterOptions\|null $options`
`26`	`26`	`*/`
`27`		`- public function __construct( FormatterOptions $options ) {`
`28`		`- $this->options = $options;`
	`27`	`+ public function __construct( FormatterOptions $options = null ) {`
	`28`	`+ $this->options = $options ?: new FormatterOptions();`
`29`	`29`
`30`	`30`	`$this->options->defaultOption( ValueFormatter::OPT_LANG, 'en' );`
`31`	`31`	`}`