New param in drupalGet

Rok Žlender - January 12, 2008 - 20:35
Project:SimpleTest
Version:5.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

Our GHOP student boombatower made a great patch for drupalGet I'm putting it in its own issue so it doesn't get overlooked

AttachmentSize
drupal_test_case.php_.patch881 bytes

#1

boombatower - January 12, 2008 - 21:11

The same issue can occur when using drupalPostRequest() so I'm not sure if that function should be patched as well. In my case I used the drupalGet() method to add the necessary parameters and then submitted with the url being set to NULL.

$this->drupalGet('node/add/story', array('query' => array('translation' => $nid, 'language' => $language)));

$edit = array();
$edit['title'] = $title;
$edit['body'] = $body;

$this->drupalPostRequest(NULL, $edit, 'Save');

#2

Rok Žlender - January 13, 2008 - 22:57
Status:patch (code needs review)» fixed

Committed.

#3

webchick - January 20, 2008 - 19:01
Status:fixed» patch (to be ported)

#4

Rok Žlender - January 28, 2008 - 07:18
Status:patch (to be ported)» patch (code needs review)

Patch also adds url() call in drupalGet.

AttachmentSize
drupalGetd5.patch4.41 KB

#5

Rok Žlender - January 28, 2008 - 07:59

Rerolled.

AttachmentSize
drupalGetd5.patch6.23 KB

#6

Rok Žlender - January 28, 2008 - 08:14
Version:6.x-1.x-dev» 5.x-1.x-dev

And again

AttachmentSize
drupalGetd5.patch6.3 KB

#7

boombatower - March 17, 2008 - 22:53
Status:patch (code needs review)» fixed

I believe this has been committed and is working correctly.

#8

Anonymous (not verified) - March 31, 2008 - 23:02
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.