Project:Prepopulate
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:patch (to be ported)

Issue Summary

Thanks for this module.

The only issue I face so far is that with a long list of fields and/or field data, I get a watchdog warning about overlong referer when I submit the new node form. This is because of use of URL parameters passed via the GET method.

Could we add a POST method to give users a choice of submitting the prepopulated fields without encountering the overlong fields/data URL warning ?

I did a quick hack to convert from $_GET to $_POST. Additional, I added a urldecode on data that contains % (which arises when a space was encountered). So far this works with my data which are either alphanumeric+space or decimal numeric.

If there's demand, perhaps, we could incorporate a dual handling for GET and POST methods, whereby if no URL parameters were found, it looks for POST variables. Or alternatively, just a simple GET vs POST priority (whichever has at least one variable); OR even a combination of both method's variables whereupon duplicate variables encountered are resolved via priority of one over the other.

Any interest in this ?

Mine has mainly been to avoid the watchdog warnings clog up my logs.

Comments

#1

Version:5.x-1.3» 6.x-1.x-dev

If you can supply a patch that moves in this direction, I'd definitely look it over. A patch for new features should go against HEAD so I'm changing the Version on the issue.

#2

Priority:minor» normal

Prepopulate rocks my socks off, and so do POST requests. Subscribe.

#3

Status:active» needs review

Moved over from $_GET to $_REQUEST.

AttachmentSize
242749.patch 2.96 KB

#4

Priority:normal» critical

Ping! Is this module maintained? Last commit was in 2008. Does it need help? If so, I'm willing to help out....

#5

Here it is for both HEAD (Drupal 6) and DRUPAL-5 branches with documentation.

AttachmentSize
242749-drupal5.patch 1.8 KB
242749-drupal6.patch 1.81 KB

#6

Status:needs review» reviewed & tested by the community

Sticking it in the RTBC queue.

#7

Priority:critical» normal
Status:reviewed & tested by the community» patch (to be ported)

Committed to 6.x-1.x branch. Marking as to be ported for 5.x and 6.x-2.x branches which I need to get the branches coordinated.

#8

The DRUPAL-5 patch is right here, from #5:
http://drupal.org/files/issues/242749-drupal5.patch

#9

Thanks Rob... Sorry I wasn't more clear. I'm not setup with Drupal 5 at the moment but will be soon and will get your Drupal 5 patch when I do.

#10

Ah, haha :-) .

nobody click here