tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

BUG Fix buttonClicked() error

Browse Source 
Fixes #3208
This commit is contained in:
Damian Mooyman 2016-06-08 12:10:01 +12:00
parent 39d1ef7a4b
commit bf00810e1f
No known key found for this signature in database
GPG Key ID: 78B823A10DE27D1A
2 changed files with 18 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
forms/Form.php
View File

@ -873,7 +873,7 @@ class Form extends RequestHandler {
/**
* Set the target of this form to any value - useful for opening the form contents in a new window or refreshing
* another frame
*
*
* @param string|FormTemplateHelper
*/
public function setTemplateHelper($helper) {
@ -1635,11 +1635,17 @@ class Form extends RequestHandler {
* @return FormAction
*/
public function buttonClicked() {
foreach($this->actions->dataFields() as $action) {
if($action->hasMethod('actionname') && $this->buttonClickedFunc == $action->actionName()) {
$actions = $this->actions->dataFields();
if(!$actions) {
return null;
}
foreach($actions as $action) {
if($action instanceof FormAction && $this->buttonClickedFunc == $action->actionName()) {
return $action;
}
}
return null;
}
/**

12
tests/forms/FormTest.php
View File

@ -19,7 +19,7 @@ class FormTest extends FunctionalTest {
Config::inst()->update('Director', 'rules', array(
'FormTest_Controller' => 'FormTest_Controller'
));
// Suppress themes
Config::inst()->remove('SSViewer', 'theme');
}
@ -324,7 +324,7 @@ class FormTest extends FunctionalTest {
public function testDisableSecurityTokenAcceptsSubmissionWithoutToken() {
SecurityToken::enable();
$expectedToken = SecurityToken::inst()->getValue();
$response = $this->get('FormTest_ControllerWithSecurityToken');
// can't use submitForm() as it'll automatically insert SecurityID into the POST data
$response = $this->post(
@ -537,6 +537,12 @@ class FormTest extends FunctionalTest {
$this->assertEquals('bar', $attrs['foo']);
}
public function testButtonClicked() {
$form = $this->getStubForm();
$action = $form->buttonClicked();
$this->assertNull($action);
}
public function testAttributesHTML() {
$form = $this->getStubForm();
@ -622,7 +628,7 @@ class FormTest extends FunctionalTest {
$formData = $form->getData();
$this->assertEmpty($formData['ExtraFieldCheckbox']);
}
protected function getStubForm() {
return new Form(
new FormTest_Controller(),