Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

Recommended Posts

  • 1 month later...
  • Replies 186
  • Created
  • Last Reply

Has anyone done anything about patches needed to make Frozen fully PHP 7.2-compatible? It seems that my host abruptly dropped PHP 7.1 on grounds that it has "severe security issues", so right now I have "deprecated" warnings suppressed (private test install at this point). Is there a laundry list anywhere of things to look for and fix, or has anyone already started on 7.2 patches? Is there an existing thread on that subject? I know @burt is working away on Edge's 7.2 compatibility, and doesn't seem interested in back-porting anything to Frozen.

Besides clean PHP 7.2-compatibility (so "deprecated" warnings can be re-enabled), I also need to find out what patches are needed for having PayPal 5.018 out-of-the-box, and there are one or two other PayPal issues floating around. When all this stuff gets fixed (or first, the non-7.2 fixes to PayPal et al.) I'll update the patch files (probably incremental updates). If there are any other patches in this thread, or elsewhere, that are missing from my patch set, please bring them to my attention.

Link to comment
Share on other sites

On 9/13/2018 at 4:43 AM, JcMagpie said:

Came across a small issue with tpl_cm_header_breadcrumb.php when doing some editing. Looks like the name has been duplicated,

<div class="col-sm-<?php echo $content_width; ?> cm-header-breadcrumb breadcrumbs">

 

If it's allready been spoted the sorry for duplication. 

So just confirming the breadcrumbs needs to be removed from /includes/modules/content/header/templates/tpl_cm_header_breadcrumb.php

Link to comment
Share on other sites

  • 3 weeks later...

back button of account_history_info.php - lines 196 - 198:

  <div class="buttonSet">
    <?php echo tep_draw_button(IMAGE_BUTTON_BACK, 'fa fa-angle-left', tep_href_link('account_history_info.php', tep_get_all_get_params(array('order_id')), 'SSL')); ?>
  </div>

Should be:

  <div class="buttonSet">
    <?php echo tep_draw_button(IMAGE_BUTTON_BACK, 'fa fa-angle-left', tep_href_link('account_history.php', tep_get_all_get_params(array('order_id')), 'SSL')); ?>
  </div>

 

Link to comment
Share on other sites

11 hours ago, Hotclutch said:

Should be:


  <div class="buttonSet">
    <?php echo tep_draw_button(IMAGE_BUTTON_BACK, 'fa fa-angle-left', tep_href_link('account_history.php', tep_get_all_get_params(array('order_id')), 'SSL')); ?>
  </div>

you don't actually need the tep_get_all_get_params(array('order_id')) that can be set to NULL

No longer giving free advice. Please place deposit in meter slot provided.  Individual: [=] SME: [==] Corporation: [===]
If deposit does not fit one of the slots provided then you are asking too much! :P

Is your Osc dated try Phoenix  raising oscommerce from the ashes.

Link to comment
Share on other sites

includes/functions/html_output.php

    if ( !isset($params['type']) ) {
      $params['type'] = 'submit';
    }

change to

    if ( !isset($params['type']) ) {
		$params = array('type' => '');
		$params['type'] = 'submit';
    }

the addition is

	$params = array('type' => '');

 

No longer giving free advice. Please place deposit in meter slot provided.  Individual: [=] SME: [==] Corporation: [===]
If deposit does not fit one of the slots provided then you are asking too much! :P

Is your Osc dated try Phoenix  raising oscommerce from the ashes.

Link to comment
Share on other sites

18 minutes ago, 241 said:

includes/functions/html_output.php


    if ( !isset($params['type']) ) {
      $params['type'] = 'submit';
    }

change to


    if ( !isset($params['type']) ) {
		$params = array('type' => '');
		$params['type'] = 'submit';
    }

the addition is


	$params = array('type' => '');

 

The above fix by 231 was in answer to my error log in includes/functions/html_output.php lines 303, 306, 307, 310, 320, 329, 348-  which begins as:

__________________________________

////
// Output a jQuery UI Button
  function tep_draw_button($title = null, $icon = null, $link = null, $priority = null, $params = null, $style = null) {
    static $button_counter = 1;

    $types = array('submit', 'button', 'reset');

    if ( !isset($params['type']) ) {
      $params['type'] = 'submit';
    }

___________________________________

So making the change just in the beginning it becomes

___________________________________

////
// Output a jQuery UI Button
  function tep_draw_button($title = null, $icon = null, $link = null, $priority = null, $params = null, $style = null) {
    static $button_counter = 1;

    $types = array('submit', 'button', 'reset');

    //if ( !isset($params['type']) ) {
    //  $params['type'] = 'submit';
    //}
    
    if ( !isset($params['type']) ) {
        $params = array('type' => '');
        $params['type'] = 'submit';
    }

____________________________________

and the errors go away. Hope this helps.

-Barbie
 

 

 

 

Link to comment
Share on other sites

Forgot to mention above that this is in FROZEN on php 7.2 which threw so many errors I have to go back and try to piecethem together for you.

For example in the modules there will be errors for "undefined constant" when certain constants were left out of the language files such as

  define('MODULE_CONTENT_BOOTSTRAP_ROW_DESCRIPTION', '<p>Content Width can be 12 or less per column per row.</p><p>12/12 = 100% width, 6/12 = 50% width, 4/12 = 33% width.</p><p>Total of all columns in any one row must equal 12 (eg:  3 boxes of 4 columns each, 1 box of 12 columns and so on).</p>');


in includes/languages/English/modules/content/navigation/cm_navbar.php

Everything defined in the module file must also be included in the corresponding language file. Drove me nuts in the beginning!

 

Link to comment
Share on other sites

1 hour ago, 241 said:

includes/functions/html_output.php


    if ( !isset($params['type']) ) {
      $params['type'] = 'submit';
    }

change to


    if ( !isset($params['type']) ) {
		$params = array('type' => '');
		$params['type'] = 'submit';
    }

the addition is


	$params = array('type' => '');

 

I suggest

    if ( ! is_array($params) ){
        $params = array();
    } 
    if ( ! isset($params['type']) ) {
        $params['type'] = 'submit';
    }

instead. That prevents you losing other parameters if type is not set.

Contact me for work on updating existing stores - whether to Phoenix or the new osC when it's released.

Looking for a payment or shipping module? Maybe I've already done it.

Working on generalising bespoke solutions for Quickbooks integration, Easify integration and pay4later (DEKO) integration at 2.3.x

Link to comment
Share on other sites

The back buttons on checkout_payment_address.php and checkout_shipping_address.php

It's like this in official osC also.

Should be:

  <div class="buttonSet">
    <?php echo tep_draw_button(IMAGE_BUTTON_BACK, 'fa fa-angle-left', tep_href_link('checkout_payment.php', '', 'SSL')); ?>
  </div>  

  <div class="buttonSet">
    <?php echo tep_draw_button(IMAGE_BUTTON_BACK, 'fa fa-angle-left', tep_href_link('checkout_shipping.php', '', 'SSL')); ?>
  </div>

 

Link to comment
Share on other sites

Archived

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

×
×
  • Create New...