Home » Forums » Ubercart » Bug Reports
8 replies [Last post]
Sat, 11/10/2007 - 19:29
dkashen
dkashen's picture
Offline
Internationalizationizer
Joined: 08/08/2007
Juice: 180
Was this information Helpful?

Hi,
after completing a credit card checkout on an estore running
drupal 5.1
and
ubercart version 7c with a
I am now getting this error

warning: Invalid argument supplied for foreach() in
/includes/common.inc(1347) : eval()'d code on line 147.

which I didn't get "before".
if I changed/added/updated another module that created this problem
it got past me at the time.

so, we've got a test site that is running
drupal 5.3
and
ubercart version 8 ( via ubercart-5.x-1.0-alpha8.tar.gz )
and
it coughs up the __same error.

that is very surprising.
Any thoughts ?

Thanks,
DK

Top
Sat, 11/10/2007 - 22:59
#1 View page
mcneelycorp
mcneelycorp's picture
Offline
Joined: 10/26/2007
Juice: 72
Re: php error generated on checkout (at least by credit card)

are you using the default ubercart credit card form or an actual gateway credit card module? IE authorize.net, linkpoint etc.

===
Elvis McNeely
Blogging about Drupal: http://www.elvisblogs.org/drupal

Top
Sat, 11/10/2007 - 23:10
#2 View page
dkashen
dkashen's picture
Offline
Internationalizationizer
Joined: 08/08/2007
Juice: 180
I'm using authorize.net this

I'm using authorize.net
this ___was working without this error message on the original install .
"something" broke it .
but the reinstall which uses a backed up version of the original DB,
with new drupal and ubercart code has the same error.

details:

from
/admin/store/settings/payment

Payment settings:
Payment tracking is enabled.
Payments may be deleted by approved users.
Payments are tracked in the order logs.

Payment methods:
PayPal Express Checkout is disabled for checkout.
PayPal Website Payments Standard is enabled for checkout.
COD is disabled for checkout.
Check is enabled for checkout.
Credit Card is enabled for checkout.
Other is disabled for checkout.

Payment gateways:
Authorize.net is enabled.
PayPal Website Payments Pro is disabled.

Top
Sat, 11/10/2007 - 23:15
#3 View page
Ryan
Ryan's picture
Offline
Joined: 08/07/2007
Juice: 15438
Re: I'm using authorize.net this

I'm not sure what in the world would be using eval() in the credit card code... I'd check common.inc to see what function that line is a part of to help trace down the bug.

Top
Sat, 11/10/2007 - 23:46
#4 View page
dkashen
dkashen's picture
Offline
Internationalizationizer
Joined: 08/08/2007
Juice: 180
common.inc

here's the excerpt from common.inc
if you believe the error message , then I think it's complaining about an commented out line. .
doesn't make much sense yet.


  129 /**
  130 * Set an HTTP response header for the current page.
  131 *
  132 * Note: when sending a Content-Type header, always include a 'charset' type
  133 * too. This is necessary to avoid security bugs (e.g. UTF-7 XSS).
  134 */
  135 function drupal_set_header($header = NULL) {
  136   // We use an array to guarantee there are no leading or trailing delimiters.
  137   // Otherwise, header('') could get called when serving the page later, which
  138   // ends HTTP headers prematurely on some PHP versions.
  139   static $stored_headers = array();
  140
  141   if (strlen($header)) {
  142     header($header);
  143     $stored_headers[] = $header;
  144   }
  145   return implode("\n", $stored_headers);
  146 }
  147
  148 /**
Top
Sun, 11/11/2007 - 00:51
#5 View page
Ryan
Ryan's picture
Offline
Joined: 08/07/2007
Juice: 15438
Re: common.inc

I should've been more specific. You need line 1347 from common.inc. Line 147 refers to some other included code... it makes me think it may be a template issue or something. Have you created a custom invoice or notification template?

Top
Sun, 11/11/2007 - 13:38
#6 View page
dkashen
dkashen's picture
Offline
Internationalizationizer
Joined: 08/08/2007
Juice: 180
yes, I had a custom template

yes,
I had a custom template and it was broken.

Thanks!

Top
Sun, 11/11/2007 - 13:41
#7 View page
dkashen
dkashen's picture
Offline
Internationalizationizer
Joined: 08/08/2007
Juice: 180
this was also affecting the weblogs

in this thread
http://www.ubercart.org/forum/support/1721/lines_cartcheckout_arent_show...
I was noting that my weblogs weren't reflecting checkout lines.

fixing the notification template also fixed this problem.

Thanks,
DK

Top
Mon, 11/12/2007 - 10:07
#8 View page
Ryan
Ryan's picture
Offline
Joined: 08/07/2007
Juice: 15438
Re: this was also affecting the weblogs

Rock on.

Top