Merge pull request #111 from yajra/buttons-option

yajra · web-flow · commit 1ca76c164a65 · 2019-10-02T10:52:25.000+08:00
[4.0] Allow array parameter on buttons option for api consistency.
diff --git a/src/Html/Options/Plugins/Buttons.php b/src/Html/Options/Plugins/Buttons.php
@@ -15,12 +15,16 @@ trait Buttons
     /**
      * Attach multiple buttons to builder.
      *
-     * @param mixed ...$buttons
+     * @param array|mixed ...$buttons
      * @return $this
      * @see https://www.datatables.net/extensions/buttons/
      */
     public function buttons(...$buttons)
     {
+        if (is_array($buttons[0])) {
+            $buttons = $buttons[0];
+        }
+
         foreach ($buttons as $button) {
             $this->attributes['buttons'][] = $button instanceof Arrayable ? $button->toArray() : $button;
         }

Original file line number	Diff line number	Diff line change
`@@ -15,12 +15,16 @@ trait Buttons`
`15`	`15`	`/**`
`16`	`16`	`* Attach multiple buttons to builder.`
`17`	`17`	`*`
`18`		`- * @param mixed ...$buttons`
	`18`	`+ * @param array\|mixed ...$buttons`
`19`	`19`	`* @return $this`
`20`	`20`	`* @see https://www.datatables.net/extensions/buttons/`
`21`	`21`	`*/`
`22`	`22`	`public function buttons(...$buttons)`
`23`	`23`	`{`
	`24`	`+ if (is_array($buttons[0])) {`
	`25`	`+ $buttons = $buttons[0];`
	`26`	`+ }`
	`27`	`+`
`24`	`28`	`foreach ($buttons as $button) {`
`25`	`29`	`$this->attributes['buttons'][] = $button instanceof Arrayable ? $button->toArray() : $button;`
`26`	`30`	`}`