Restrict Digital Downloads [Support Thread]

[RixStix]

You're welcome, I think.

But I'm also thinking that in order to have an alternative choice for the digital download for the EU customers, at least a second attribute should be required for the choice of "Email the file to me" instead of the automated download.

That would be a "my problem" not a lat9 problem to resolve? OR attribute to add?

OK, I will hold off wiping the sandbox.

[lat9]

You're welcome, I think.

But I'm also thinking that in order to have an alternative choice for the digital download for the EU customers, at least a second attribute should be required for the choice of "Email the file to me" instead of the automated download.

That would be a "my problem" not a lat9 problem to resolve? OR attribute to add?

OK, I will hold off wiping the sandbox.

I'll go for a "combined" problem. I believe that you should offer your customers an alternative-delivery choice, but the plugin should be bullet-proof.

[lat9]

My internet access has been funky; I thought that I'd posted the corrections that are included in the (just uploaded) v1.0.1.

Anyway, here are the two files that are changed in that version. First /includes/extra_cart_actions/restrict_digital_downloads.php:

// -----
// Part of the "Restrict Digital Downloads" plugin
// 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) {
      $the_options = array ();
      if (isset($_POST['id']) && is_array($_POST['id'])) {
        $the_options = $_POST['id'];
        
      }
      if (isset ($_POST['download_options']) && is_array ($_POST['download_options'])) {
        foreach ($_POST['download_options'] as $option => $value) {
          if (!isset ($the_options[$option])) {
            $the_options[$option] = $value;
            
          }
        }
      }
      foreach ($the_options 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;
  }
}

Secondly, /includes/templates/YOUR_TEMPLATE/{product_type}/extra_main_template_vars/restrict_digital_downloads.php:

<?php
// -----
// Part of the "Restrict Digital Downloads" plugin
// Copyright (c) 2014 Vinos de Frutas Tropicales
//
// NOTE:  $products_options_names, $options_menu and $order_by are previously set by /includes/modules/attributes.php's processing.
//
if (isset ($_SESSION['is_restricted_ip']) && isset ($products_options_names)) {
  $products_options_names->Move (0);
  $products_options_names->MoveNext ();  //-Rewind the object to the beginning
  
  $download_selectors = '';
  $downloads_present = false;
  $option_count = 0;
  while (!$products_options_names->EOF) {
    $options_id = (int)$products_options_names->fields['products_options_id'];
    $products_options = $db->Execute ("SELECT pov.products_options_values_id, pov.products_options_values_name, pa.*
                                         from " . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_OPTIONS_VALUES . " pov
                                        where pa.products_id = '" . (int)$_GET['products_id'] . "'
                                          and pa.options_id = '$options_id'
                                          and pa.options_values_id = pov.products_options_values_id
                                          and pov.language_id = '" . (int)$_SESSION['languages_id'] . "' " . $order_by);
    $option_has_download = false;
    while (!$products_options->EOF) {
      $options_values_id = $products_options->fields['options_values_id'];
      $download_check = $db->Execute ("SELECT products_attributes_id FROM " . TABLE_PRODUCTS_ATTRIBUTES_DOWNLOAD . " WHERE products_attributes_id = " . $products_options->fields['products_attributes_id'] . " LIMIT 1");
      if (!$download_check->EOF) {
        $option_has_download = true;
        $downloads_present = true;
        $products_options_type = zen_get_attributes_type ($products_options->fields['products_attributes_id']);
        switch ($products_options_type) {
          case PRODUCTS_OPTIONS_TYPE_SELECT: {
            $download_selectors .= (($products_options->RecordCount ()) == 1) ? ('#attrib-' . $options_id . '-' . $options_values_id . ', ') : ('#attrib-' . $options_id . ' option[value="' . $options_values_id . '"], ');
            break;
          }
          case PRODUCTS_OPTIONS_TYPE_RADIO: {
            $download_selectors .= '#attrib-' . $options_id . '-' . $options_values_id . ', ';
            break;
          }
          default: {
            break;
          }
        }
      }
      if ($option_has_download && $products_options->RecordCount () == 1 && $products_options_type != PRODUCTS_OPTIONS_TYPE_CHECKBOX) {
        $options_menu[$option_count] .= zen_draw_hidden_field ("download_options[$options_id]", $options_values_id);
        
      }
      $products_options->MoveNext ();
      
    }
    $option_count++;
    $products_options_names->MoveNext ();
    
  }
  if ($downloads_present != '') {
    $messageStack->add ('product_info', PRODUCT_MESSAGE_DOWNLOAD_PRODUCT_RESTRICTED, 'caution');
    
  }
  if ($download_selectors != '') {
    $download_selectors = substr ($download_selectors, 0, -2);  //-Strip trailing ', '
?>
<script type="text/javascript">
$(document).on ('ready', function(){
  $('<?php echo $download_selectors; ?>').each(function(){
    $(this).removeAttr( 'selected checked' );
    $(this).attr( 'disabled', 'disabled' );
  });
});
</script>
<?php
  }
}

[RixStix]

TNX lat9

I know funky internet . I'm lucky we have not had snow yet. Snow on the mountaintops usually plays havoc with our 1MB connection.

I don't see any difference in functionality other than no MyDebug logs are generated.


I get the yellow warning banner but am still able to add the product to the cart and complete the checkout.

I have not changed any product details, nor have I added an attribute for a non-pdf choice.

Before I wipe the sandbox and test with my database/product setup without any plugins or responsive template, do you have a specific downloadable product w/attributes setup example that I could use to generate a new product within the catalog for testing.

[lat9]

I've got an option (radio-button) named "Delivery Method" that has two option values: I'll download it and Email it to me. I apply those two option/value pairs via the Attributes Controller to my downloads (the I'll download it option has the download file associated).

If the responsive template you're using is one of Picaflor-Azul's, she's got a tendency to mis-code calls to get_template_dir like this:

<script src="<?php echo $template->get_template_dir('',DIR_WS_TEMPLATE, $current_page_base,'jscript') . '/back_to_top.min.js'; ?>" type="text/javascript"></script>

where the proper coding is

<script src="<?php echo $template->get_template_dir('back_to_top.min.js',DIR_WS_TEMPLATE, $current_page_base,'jscript') . '/back_to_top.min.js';  ?>" type="text/javascript"></script>

I ran into that gotcha in another customization that I created that uses the template-override method used by RDD.

[RixStix]

I'm gonna have to wipe everything and start from scratch.

Yes I am using one of Anne's templates. Made the edit for get_template.

Create new temp digital download product category
Create new product within that category
Add attribute for download pdf file and associate a pdf to that attribute
Add attribute for "Email the file to me"
attribute option type is "Checkbox" (a couple products contain multiple download files)
Is the checkbox the source of my headache?

I get the yellow alert message but am still able to add the product to the cart regardless of chosen attribute.

Checkout and receive the download link immediately.

DarN. It may be a day or two before I have the time to start from scratch. Physical Christmas orders are a bit more pressing.

[RixStix]

How dumb can a person feel?
Can I claim a "DUH" moment?

Repeat after me a kazillion times....

Radio, Radio, Radio. NOT checkbox, NOT checkbox, NOT checkbox.

Our initial reason for using checkbox is that we have a couple products which have 2 or more pdf associated with the product.
More reason for me to redo things.

Things appear to work when I use the proper setup. Says he with egg completely covering face.

[lat9]

When the product was using a checkbox attribute, was it that the product (without the checkbox-option) was added to the cart? I tested using a checkbox-type attribute, thinking that there might be a product that could be either physical (unchecked) or virtual (checked).

[RixStix]

When the product was using a checkbox attribute, was it that the product (without the checkbox-option) was added to the cart? I tested using a checkbox-type attribute, thinking that there might be a product that could be either physical (unchecked) or virtual (checked).

download file attribute can be added to cart if checked, regardless of warning.

product with no checkbox selected can be added to cart
product with download attribute selected can be added to cart
product with "email it to me" attribute selected can be added to cart

[lat9]

Can you post or PM a link?