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).

CommentFileSizeAuthor
#4 print_query_string_encode.patch245 bytesFenwick
loginerror.txt91.24 KBAnonymous (not verified)

Comments

Anonymous’s picture

Bump! Any update on a fix? Thanks.

R.Muilwijk’s picture

Starting 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

mgifford’s picture

I 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."

Fenwick’s picture

StatusFileSize
new245 bytes

It 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.

vitok-dupe’s picture

yep, your patch make it works. thx Fenwick

elachlan’s picture

Status: Active » Needs review

Marking as needs review for the test bot.

mgifford’s picture

Status: Needs review » Reviewed & tested by the community

@Fenwick- this worked fine for me. Pretty simple patch. How do we get this into a stable release soon for this project?

elachlan’s picture

I refer you to http://drupal.org/node/1057502

We just wait until the maintainer returns.

mgifford’s picture

Well 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

jcnventura’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for the patch! I've committed it to git.

Status: Fixed » Closed (fixed)
Issue tags: -drupal_http_build_query

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