mirror of
https://github.com/kamilwylegala/cakephp2-php8.git
synced 2024-11-15 03:18:26 +00:00
Fixing issues with double base dir in FormHelper::create(). Fixes #1568
This commit is contained in:
parent
2145fd8078
commit
af16b13e5f
2 changed files with 19 additions and 8 deletions
|
@ -239,7 +239,7 @@ class FormHelper extends AppHelper {
|
|||
$this->_inputDefaults = $options['inputDefaults'];
|
||||
unset($options['inputDefaults']);
|
||||
if ($options['action'] === null && $options['url'] === null) {
|
||||
$options['action'] = $this->request->here;
|
||||
$options['action'] = $this->request->here(false);
|
||||
if (!isset($options['id'])) {
|
||||
$options['id'] = $this->domId($this->request['action'] . 'Form');
|
||||
}
|
||||
|
|
|
@ -662,12 +662,13 @@ class FormHelperTest extends CakeTestCase {
|
|||
function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
Configure::write('App.base', '');
|
||||
$this->Controller = new ContactTestController();
|
||||
$this->View = new View($this->Controller);
|
||||
|
||||
$this->Form = new FormHelper($this->View);
|
||||
$this->Form->Html = new HtmlHelper($this->View);
|
||||
$this->Form->request = new CakeRequest(null, false);
|
||||
$this->Form->request = new CakeRequest('contacts/add', false);
|
||||
$this->Form->request->here = '/contacts/add';
|
||||
$this->Form->request['action'] = 'add';
|
||||
$this->Form->request->webroot = '';
|
||||
|
@ -5664,12 +5665,21 @@ class FormHelperTest extends CakeTestCase {
|
|||
'/div'
|
||||
);
|
||||
$this->assertTags($result, $expected);
|
||||
}
|
||||
|
||||
$this->Form->request->here = '/contacts/add/Contact:1';
|
||||
$result = $this->Form->create();
|
||||
/**
|
||||
* test create() with automatic url generation
|
||||
*
|
||||
* @return void
|
||||
*/
|
||||
function testCreateAutoUrl() {
|
||||
Router::setRequestInfo(array(array(), array('base' => '/base_url')));
|
||||
$this->Form->request->here = '/base_url/contacts/add/Contact:1';
|
||||
$this->Form->request->base = '/base_url';
|
||||
$result = $this->Form->create('Contact');
|
||||
$expected = array(
|
||||
'form' => array(
|
||||
'id' => 'ContactAddForm', 'method' => 'post', 'action' => '/contacts/add/Contact:1',
|
||||
'id' => 'ContactAddForm', 'method' => 'post', 'action' => '/base_url/contacts/add/Contact:1',
|
||||
'accept-charset' => 'utf-8'
|
||||
),
|
||||
'div' => array('style' => 'display:none;'),
|
||||
|
@ -5679,11 +5689,12 @@ class FormHelperTest extends CakeTestCase {
|
|||
$this->assertTags($result, $expected);
|
||||
|
||||
$this->Form->request['action'] = 'delete';
|
||||
$this->Form->request->here = '/contacts/delete/10/User:42';
|
||||
$result = $this->Form->create();
|
||||
$this->Form->request->here = '/base_url/contacts/delete/10/User:42';
|
||||
$this->Form->request->base = '/base_url';
|
||||
$result = $this->Form->create('Contact');
|
||||
$expected = array(
|
||||
'form' => array(
|
||||
'id' => 'ContactDeleteForm', 'method' => 'post', 'action' => '/contacts/delete/10/User:42',
|
||||
'id' => 'ContactDeleteForm', 'method' => 'post', 'action' => '/base_url/contacts/delete/10/User:42',
|
||||
'accept-charset' => 'utf-8'
|
||||
),
|
||||
'div' => array('style' => 'display:none;'),
|
||||
|
|
Loading…
Reference in a new issue