Patch updates print_query_string_encode() to match the latest drupal_query_string_encode().

CommentFileSizeAuthor
#1 print.module-301192-1_DRUPAL-6--1.patch1.06 KBdarren oh
print.module_DRUPAL-6--1.patch1.21 KBdarren oh

Comments

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Active » Needs review
StatusFileSize
new print.module-301192-1_DRUPAL-6--1.patch1.06 KB

Wrong patch. Here is the one I was describing.

jcnventura’s picture

jcnventura
Primary language English
commented

Hi,

There was a reason why the print version is not recursive (and does not need the parent parameter): it's supposed that the query in the original URL doesn't have arrays anymore!

How did you come by this problem?

João

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented

I created a view with exposed filters. One of the filters was numeric, which returns an array of values for comparison (min, max, value). Having a printer-friendly link on this page caused the following error when filters were applied:

warning: rawurlencode() expects parameter 1 to be string, array given in /var/www/sites/all/modules/contrib on line 490.

fonant’s picture

fonant commented

I think this can be fixed more cleanly, also solving other double-encoded problems, by fixing drupal_query_string_encode() itself: http://drupal.org/node/285496

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs review » Fixed

Hi,

Just committed the patch you provided. Thanks!

João

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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