I have Printer, email & PDF installed.
When I attempt to login to home page I get the following error:
Recoverable fatal error: Argument 1 passed to drupal_http_build_query() must be an array, string given, called in /var/www/includes/common.inc on line 2157 and defined in drupal_http_build_query() (line 473 of /var/www/includes/common.inc).
If I attempt to login via a page that does not exist I am able to login OK.
I have been logging this via http://drupal.org/node/970786 and this leads me think it is a problem with this module. I f I disable the module it works OK. I have tried just enabling Print and not the other options and the error still occurs.
I attach the file upload from the above thread.
I am running Drupal 7.0 and print 7.x-1.x-dev (2010-Oct-12).
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | print_query_string_encode.patch | 245 bytes | Fenwick |
| loginerror.txt | 91.24 KB | Anonymous (not verified) |
Comments
Comment #1
Anonymous (not verified) commentedBump! Any update on a fix? Thanks.
Comment #2
R.Muilwijk commentedStarting from drupal 7 the l() and url() functions expect 'query' to be an array of key value. Within the print module this is still a string and therefor not working properly.
The patch should proberly check all instances of:
$query = print_query_string_encode(...);
and just return the unencoded array instead
Comment #3
mgiffordI disabled & uninstalled this module and the error went away.
I also found this error listed here, and put a point back here:
http://drupal.org/node/1028518
Recoverable fatal error: Argument 1 passed to drupal_http_build_query() must be an array, string given, called in /DRUPAL/includes/common.inc on line 2157 and defined in drupal_http_build_query() (line 473 of /DRUPAL/includes/common.inc).
It showed up on an alias where "The page you requested does not exist."
Comment #4
Fenwick commentedIt does seem to be as simple as returning the query as an array rather than a string. I haven't tested it rigorously, but the attached patch appears to have fixed my problem without breaking anything else.
Comment #5
vitok-dupe commentedyep, your patch make it works. thx Fenwick
Comment #6
elachlan commentedMarking as needs review for the test bot.
Comment #7
mgifford@Fenwick- this worked fine for me. Pretty simple patch. How do we get this into a stable release soon for this project?
Comment #8
elachlan commentedI refer you to http://drupal.org/node/1057502
We just wait until the maintainer returns.
Comment #9
mgiffordWell there are a few folks here http://drupal.org/node/12219/committers who should be able to add this to the cvs at least.
I tweeted something about it to @jcnventura
Comment #10
jcnventuraThanks for the patch! I've committed it to git.