From 72b6a8da12693bc99e163002c56fcae4da8ec1d1 Mon Sep 17 00:00:00 2001 From: Ryan Potter Date: Mon, 1 Jul 2019 09:37:12 +1200 Subject: [PATCH 1/3] Added a method to set the button name in a GridFieldAddByDBField component --- .idea/misc.xml | 6 + .idea/modules.xml | 8 + .idea/silverstripe-blog.iml | 11 ++ .idea/vcs.xml | 6 + .idea/workspace.xml | 157 ++++++++++++++++++ src/Forms/GridField/GridFieldAddByDBField.php | 30 +++- 6 files changed, 216 insertions(+), 2 deletions(-) create mode 100644 .idea/misc.xml create mode 100644 .idea/modules.xml create mode 100644 .idea/silverstripe-blog.iml create mode 100644 .idea/vcs.xml create mode 100644 .idea/workspace.xml diff --git a/.idea/misc.xml b/.idea/misc.xml new file mode 100644 index 0000000..28a804d --- /dev/null +++ b/.idea/misc.xml @@ -0,0 +1,6 @@ + + + + + \ No newline at end of file diff --git a/.idea/modules.xml b/.idea/modules.xml new file mode 100644 index 0000000..91ce0e0 --- /dev/null +++ b/.idea/modules.xml @@ -0,0 +1,8 @@ + + + + + + + + \ No newline at end of file diff --git a/.idea/silverstripe-blog.iml b/.idea/silverstripe-blog.iml new file mode 100644 index 0000000..c179ff9 --- /dev/null +++ b/.idea/silverstripe-blog.iml @@ -0,0 +1,11 @@ + + + + + + + + + + + \ No newline at end of file diff --git a/.idea/vcs.xml b/.idea/vcs.xml new file mode 100644 index 0000000..94a25f7 --- /dev/null +++ b/.idea/vcs.xml @@ -0,0 +1,6 @@ + + + + + + \ No newline at end of file diff --git a/.idea/workspace.xml b/.idea/workspace.xml new file mode 100644 index 0000000..ad793fd --- /dev/null +++ b/.idea/workspace.xml @@ -0,0 +1,157 @@ + + + + + + + + + $PROJECT_DIR$/composer.json + + + + + + + + + + + + + + + + + + + + _t( + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - 1561930166221 - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - \ No newline at end of file From d531356863a48ab27d0516874d96781aa189f4d3 Mon Sep 17 00:00:00 2001 From: Ryan Potter Date: Mon, 1 Jul 2019 10:32:19 +1200 Subject: [PATCH 3/3] Set docblock for the setButtonName method, and don't invalidate localisation text. --- .gitignore | 1 - src/Forms/GridField/GridFieldAddByDBField.php | 6 +++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/.gitignore b/.gitignore index bbe398c..80e016d 100755 --- a/.gitignore +++ b/.gitignore @@ -1,5 +1,4 @@ /vendor/* -/.idea .sass-cache/* /node_modules/ **/*.js.map diff --git a/src/Forms/GridField/GridFieldAddByDBField.php b/src/Forms/GridField/GridFieldAddByDBField.php index 7a4b06f..43a0c43 100644 --- a/src/Forms/GridField/GridFieldAddByDBField.php +++ b/src/Forms/GridField/GridFieldAddByDBField.php @@ -152,7 +152,7 @@ class GridFieldAddByDBField implements GridField_ActionProvider, GridField_HTMLP /** * Set the button name * - * @param $name + * @param $name string * @return $this */ public function setButtonName($name) @@ -211,7 +211,7 @@ class GridFieldAddByDBField implements GridField_ActionProvider, GridField_HTMLP if (!$this->buttonName) { // provide a default button name, can be changed by calling {@link setButtonName()} on this component $objectName = $obj->i18n_singular_name(); - $this->buttonName = _t(__CLASS__ . '.ButtonName', 'Add {name}', array('name' => $objectName)); + $this->buttonName = _t(__CLASS__ . '.ButtonName', '{name}', ['name' => $objectName]); } $addAction = GridField_FormAction::create( @@ -219,7 +219,7 @@ class GridFieldAddByDBField implements GridField_ActionProvider, GridField_HTMLP 'add', _t( __CLASS__ . '.Add', - '{name}', + 'Add {name}', 'Add button text', ['name' => $this->buttonName] ),