From 239b4a01f73dd64b7ff85655fd440f557f4476b6 Mon Sep 17 00:00:00 2001 From: Sean Harvey Date: Fri, 23 Mar 2012 10:51:23 +1300 Subject: [PATCH] MINOR Rename argument to GridFieldDeleteAction to match the property name $removeRelation MINOR Tidy up coding standards in GridFieldDeleteAction --- forms/gridfield/GridFieldDeleteAction.php | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/forms/gridfield/GridFieldDeleteAction.php b/forms/gridfield/GridFieldDeleteAction.php index 87f0d8c82..1325df92f 100644 --- a/forms/gridfield/GridFieldDeleteAction.php +++ b/forms/gridfield/GridFieldDeleteAction.php @@ -26,10 +26,10 @@ class GridFieldDeleteAction implements GridField_ColumnProvider, GridField_Actio /** * - * @param boolean $unlinkRelation - true if removing the item from the list, but not deleting it + * @param boolean $removeRelation - true if removing the item from the list, but not deleting it */ - public function __construct($unlinkRelation = false) { - $this->removeRelation = $unlinkRelation; + public function __construct($removeRelation = false) { + $this->removeRelation = $removeRelation; } /** @@ -39,8 +39,9 @@ class GridFieldDeleteAction implements GridField_ColumnProvider, GridField_Actio * @param array $columns */ public function augmentColumns($gridField, &$columns) { - if(!in_array('Actions', $columns)) + if(!in_array('Actions', $columns)) { $columns[] = 'Actions'; + } } /**