Restrict Digital Downloads [Support Thread]

[lat9]

If you move that error_log statement, do you still get the log?

// -----
// Part of the "Restrict Digital Downloads" plugin by Cindy Merkin ([email protected])
// Copyright (c) 2014 Vinos de Frutas Tropicales
//
switch ($_GET['action']) {
  /*----
  ** If a product is being added to the cart and that product includes attributes (the $_POST['id'] array
  ** is set), then check each of the attributes being added to see if there is a download/virtual product amongst them.  If so,
  ** don't allow the duplicate download/virtual product to be added to the cart ... or just add 1 if this is the original add.
  */
  case 'add_product': {
    if ($_SESSION['is_restricted_ip'] && isset($_POST['products_id']) && $_POST['cart_quantity'] > 0 && isset($_POST['id']) && is_array($_POST['id'])) {
      error_log (__FILE__ . ':' . print_r ($_POST, true));
      foreach($_POST['id'] as $option => $value) {
        if (is_digital_download ($_POST['products_id'], $option, $value)) {
          $messageStack->add ('header', sprintf (CART_MESSAGE_DOWNLOAD_PRODUCT_RESTRICTED, zen_get_products_name ($_POST['products_id'])), 'caution');
          unset ($_GET['action']);
          break;

        }
      }
    }
    break;
  }
}

[RixStix]

If you move that error_log statement, do you still get the log?

yes, the log is generated with similar content.

[lat9]

This is (obviously) kicking my bu++. How about this edit:

// -----
// Part of the "Restrict Digital Downloads" plugin by Cindy Merkin ([email protected])
// Copyright (c) 2014 Vinos de Frutas Tropicales
//
switch ($_GET['action']) {
  /*----
  ** If a product is being added to the cart and that product includes attributes (the $_POST['id'] array
  ** is set), then check each of the attributes being added to see if there is a download/virtual product amongst them.  If so,
  ** don't allow the duplicate download/virtual product to be added to the cart ... or just add 1 if this is the original add.
  */
  case 'add_product': {
    if ($_SESSION['is_restricted_ip'] && isset($_POST['products_id']) && $_POST['cart_quantity'] > 0 && isset($_POST['id']) && is_array($_POST['id'])) {
       foreach($_POST['id'] as $option => $value) {
        error_log ('rdd add_product:  products_id = ' . $_POST['products_id'] . ", options_id = $option, value_id = $value, is_digital_download = " . ((is_digital_download ($_POST['products_id'], $option, $value)) ? 'Yes' : 'No'));
        if (is_digital_download ($_POST['products_id'], $option, $value)) {
          $messageStack->add ('header', sprintf (CART_MESSAGE_DOWNLOAD_PRODUCT_RESTRICTED, zen_get_products_name ($_POST['products_id'])), 'caution');
          unset ($_GET['action']);
          break;

        }
      }
    }
    break;
  }
}

[RixStix]

Different logfile

[16-Dec-2014 07:52:38 America/Los_Angeles] rdd add_product:  products_id = 776, options_id = 6, value_id = Array, is_digital_download = No

[lat9]

Arggh, I should have seen that before. Do you have a customized /includes/modules/YOUR_TEMPLATE/attributes.php? For a 'out-of-the-box' Zen Cart attributes.php, the 'id' value will come back as an array in options_id => options_values_id format, like the following:

[14-Dec-2014 14:56:50 America/Los_Angeles] /home/XXXXXXXX/public_html/ZEN153/includes/extra_cart_actions/restrict_digital_downloads.php:Array
(
    [securityToken] => Not sure if it is OK to post that
    [cart_quantity] => 1
    [products_id] => 776
    [id] => Array
        (
            [6] => 61
 
        )

)

[RixStix]

Arggh, I should have seen that before. Do you have a customized /includes/modules/YOUR_TEMPLATE/attributes.php? For a 'out-of-the-box' Zen Cart attributes.php, the 'id' value will come back as an array in options_id => options_values_id format, like the following:

No, I do not. Searched using Developers Took Kit and didn't find anything by that name.

I did 'drop' the fileset into template_default instead of installing a vanilla template, did nothing on the admin side other than change template to classic_green.

Same result.

Work is calling my name so I'll be in and out for the next few hours.

[lat9]

There's got to be something different; I've tested this on a Zen Cart v1.5.1 and a Zen Cart v1.5.3 store and both stores log the 'id' variable as 'id'[options_id][values_id].

[RixStix]

Sorry I couldn't reply sooner. This forum does not play nice with Surface tablets which is my only access in the manufacturing room.

I was going to suggest that I wipe the sandbox and start from scratch instead of you tracking down ghosts or other unknowns.

[RixStix]

That is the format of the logfile prior to the last suggested edit.

includes/modules/attributes.php default file is being used.

It looks like your download product examples have more than a single attribute choice. Ours currently only have a single attribute. Any chance that is related?

Arggh, I should have seen that before. Do you have a customized /includes/modules/YOUR_TEMPLATE/attributes.php? For a 'out-of-the-box' Zen Cart attributes.php, the 'id' value will come back as an array in options_id => options_values_id format, like the following:

[14-Dec-2014 14:56:50 America/Los_Angeles] /home/XXXXXXXX/public_html/ZEN153/includes/extra_cart_actions/restrict_digital_downloads.php:Array
(
    [securityToken] => Not sure if it is OK to post that
    [cart_quantity] => 1
    [products_id] => 776
    [id] => Array
        (
            [6] => 61
 
        )

)

[lat9]

That is the format of the logfile prior to the last suggested edit.

includes/modules/attributes.php default file is being used.

It looks like your download product examples have more than a single attribute choice. Ours currently only have a single attribute. Any chance that is related?

You nailed it! I've got this replicated now on my local test site and should have a correction by the morning. Thanks a bunch!