Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

Hoster upgrade to php 5.6


Guest

Recommended Posts

yes they did.... unoeuro...

 

oh well not so much time bitching about that... i had to change the 2 app top files to get rid of the warnings and i can make an order just fine again...

 

 

thou there are some warnings during checkout i cant figure...  at checkout_payment.php.... where i select payment method is see this above the continue button:

Warning: Illegal string offset 'type' in /var/www/pizzacapri.dk/public_html/includes/functions/html_output.php on line 346

Warning: Illegal string offset 'type' in /var/www/pizzacapri.dk/public_html/includes/functions/html_output.php on line 349

Warning: Illegal string offset 'type' in /var/www/pizzacapri.dk/public_html/includes/functions/html_output.php on line 350

Warning: Illegal string offset 'type' in /var/www/pizzacapri.dk/public_html/includes/functions/html_output.php on line 353

Warning: Illegal string offset 'type' in /var/www/pizzacapri.dk/public_html/includes/functions/html_output.php on line 363

Warning: Illegal string offset 'type' in /var/www/pizzacapri.dk/public_html/includes/functions/html_output.php on line 370

Warning: Illegal string offset 'type' in /var/www/pizzacapri.dk/public_html/includes/functions/html_output.php on line 379

Warning: Illegal string offset 'iconpos' in /var/www/pizzacapri.dk/public_html/includes/functions/html_output.php on line 391

Warning: Illegal string offset 'iconpos' in /var/www/pizzacapri.dk/public_html/includes/functions/html_output.php on line 394

any idea what can be the issue here?

Link to comment
Share on other sites

I'm still looking for firm information, but my suspicion is that PHP 5.6 has tightened up some rules. In html_output.php's tep_draw_button(), the default for $parms is null. That may no longer be valid as an empty array base, where you then define elements such as $params['type'] = ... You might try changing the argument list entry from $params = null to $params = array(). Please report back whether or not the errors go away (I don't have PHP 5.6 to play with).

Link to comment
Share on other sites

will do so during weekend... i have to play with it when store is closed... (store open 15-21 CET)

 

i also wonder if its worth the effort to upgrade to latest version while at it or if the problem would still be there?

 

anyway off for work... back in 5 hours or so... might even setup a test server during weekend on the side... 

Link to comment
Share on other sites

ok... will do... store opened 1 hour ago so will have to wait...

 

thou on the side i'm building up the shop from scratch on latest ver... just gathering all the plugins... thought it could not hurt as oscommerce dev has moved on

Link to comment
Share on other sites

Yes, that line. It didn't help? Well, that's all I could think of at the moment, assuming that you properly uploaded the changed file, etc.

 

You might look at who's calling the function, to see if an explicit argument is being given that is invalid (e.g., '' or null in the call, rather than array() or just leaving it out (default) since it's the last parameter). I just took a look through the catalog side, and didn't see any calls to tep_draw_button() which specified a fifth argument ($params). Maybe an add-on does?

 

While version 2.3.3 is not terribly out of date, it is 5 updates behind 2.3.4. You should try not to fall too far behind, as it gets harder and harder to update/migrate when you let things slip for a long time. You also become more vulnerable to hacks, as well as getting blindsided by your host's PHP upgrades.

Link to comment
Share on other sites

but no the errors is still there.... illegal offset?

That error usually occurs when the array element hasn't been declared or it is the wrong type. If you going to use an array named my_array and that has an element named type, then before using the array, declare it as

$my_array = array('type');

The error shouldn't occur then. Or, you can just check it first as in

id (isset($my_array['type']))
 ...do array stuff here

The latter can cause the code to not work as expected if the array isn't setup properly in the first place.

Support Links:

For Hire: Contact me for anything you need help with for your shop: upgrading, hosting, repairs, code written, etc.

All of My Addons

Get the latest versions of my addons

Recommended SEO Addons

Link to comment
Share on other sites

in the end i started building a shop from scratch with latest version...

 

thou in admin i get some warnings and errors and in the shop i cant login...

 

should i create a new thread for that?

Link to comment
Share on other sites

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...