Support Thread: Checkbox Textbox Icon Addon for Listings

[linuxguy2]

Excellent! Changing true to 'true' in the modified template files fixed the add to cart issue.

However the code you listed for the "shopping_cart classes doesn't exist in any of the code I have ever downloaded for ZC or the CBTB plugin. Yes, I always use "developers toolkit " to search.
i.e.
version "CheckBoxTextBoxIconV1.0.3" nor version "CheckBoxTextBoxIcon06V1.0.0"
Am I missing something?

        if ($multiAddCheck == true && (PRODUCT_ALL_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_FEATURED_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_NEW_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_LISTING_CHECKBOX_TEXTBOX_ACTIVE == '1')){
          //reset($_POST['products_id2']);
          
          foreach ($_POST['products_id2'] as $key2 => $val2) {

[mc12345678]

Excellent! Changing true to 'true' in the modified template files fixed the add to cart issue.

However the code you listed for the "shopping_cart classes doesn't exist in any of the code I have ever downloaded for ZC or the CBTB plugin. Yes, I always use "developers toolkit " to search.
i.e.
version "CheckBoxTextBoxIconV1.0.3" nor version "CheckBoxTextBoxIcon06V1.0.0"
Am I missing something?

        if ($multiAddCheck == true && (PRODUCT_ALL_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_FEATURED_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_NEW_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_LISTING_CHECKBOX_TEXTBOX_ACTIVE == '1')){
          //reset($_POST['products_id2']);
          
          foreach ($_POST['products_id2'] as $key2 => $val2) {

That code segment is/was there, but the if statement was a little different having $multiAddCheck at the end of the logic instead of at the beginning. After some four years of additional ZC php coding experience it seemed that the logic should be modified a little to possibly bypass that section of code "faster". A similar developers tool kit search for $multiAddCheck should identify the section referenced above.

[linuxguy2]

NOTE:
A substantial amount of code has been added to ZC V15.5f shopping_cart classes.

Yes, that logic is there but coded differently.
It appears the "foreach" was intended to be implemented but never was.
See comment: * @todo change while loop to a foreach

/**
   * Method to handle cart Action - multiple add products
   *
   * @param string forward destination
   * @param url parameters
   * @todo change while loop to a foreach
   */
  function actionMultipleAddProduct($goto, $parameters) {
    global $messageStack;
    $addCount = 0;

    if (is_array($_POST['products_id']) && sizeof($_POST['products_id']) > 0) {
		// Begin mc12345678 Checkbox/TextBox
	    if ($_POST['check_text_active'] == true) {
		    $multiAddCheck = true;
		} else {
		    $multiAddCheck = false;
		}
		// End mc12345678 Checkbox/TextBox
	    while ( list( $key, $val ) = each($_POST['products_id']) ) {

		// Begin mc12345678 Checkbox/TextBox
		    if ((PRODUCT_ALL_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_FEATURED_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_NEW_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_LISTING_CHECKBOX_TEXTBOX_ACTIVE == '1') && $multiAddCheck == true){
			    //reset($_POST['products_id2']);
			    
			    while ( list( $key2, $val2 ) = each($_POST['products_id2']) ) {
				 if ($key == $key2) {
					 if ($val > 0) 
				{

[mc12345678]

NOTE:
A substantial amount of code has been added to ZC V15.5f shopping_cart classes.

Yes, that logic is there but coded differently.
It appears the "foreach" was intended to be implemented but never was.
See comment: * @todo change while loop to a foreach

/**
   * Method to handle cart Action - multiple add products
   *
   * @param string forward destination
   * @param url parameters
   * @todo change while loop to a foreach
   */
  function actionMultipleAddProduct($goto, $parameters) {
    global $messageStack;
    $addCount = 0;

    if (is_array($_POST['products_id']) && sizeof($_POST['products_id']) > 0) {
        // Begin mc12345678 Checkbox/TextBox
        if ($_POST['check_text_active'] == true) {
            $multiAddCheck = true;
        } else {
            $multiAddCheck = false;
        }
        // End mc12345678 Checkbox/TextBox
        while ( list( $key, $val ) = each($_POST['products_id']) ) {

        // Begin mc12345678 Checkbox/TextBox
            if ((PRODUCT_ALL_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_FEATURED_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_NEW_CHECKBOX_TEXTBOX_ACTIVE == '1' || PRODUCT_LISTING_CHECKBOX_TEXTBOX_ACTIVE == '1') && $multiAddCheck == true){
                //reset($_POST['products_id2']);
                
                while ( list( $key2, $val2 ) = each($_POST['products_id2']) ) {
                 if ($key == $key2) {
                     if ($val > 0) 
                {

Yes, I forgot that I made my portion of the code more 7.x compatible by removing the list() = each() style from the code added by this plugin. I believe 1.5.6 already has modified that area for its own loop.

I may also further modify the logic of:

(PRODUCT_ALL_CHECKBOX_TEXTBOX_ACTIVE == '1' ||  PRODUCT_FEATURED_CHECKBOX_TEXTBOX_ACTIVE == '1' ||  PRODUCT_NEW_CHECKBOX_TEXTBOX_ACTIVE == '1' ||  PRODUCT_LISTING_CHECKBOX_TEXTBOX_ACTIVE == '1')

to:

!(PRODUCT_ALL_CHECKBOX_TEXTBOX_ACTIVE != '1' &&  PRODUCT_FEATURED_CHECKBOX_TEXTBOX_ACTIVE != '1' &&  PRODUCT_NEW_CHECKBOX_TEXTBOX_ACTIVE != '1' &&  PRODUCT_LISTING_CHECKBOX_TEXTBOX_ACTIVE != '1')