Using include in product description

[Ajeh]

$products_options->fields['products_options_values_name'] != ' ' is not the same as:
$products_options->fields['products_options_values_name'] != ''

You want to test when that field is blank and when blank show nothing ...

[jeking]

I'm understanding the logic but not the coding. This is what I have understood, but it's not working right.

// Read Only - just for display purposes

if (($products_options_names->fields['products_options_type'] == PRODUCTS_OPTIONS_TYPE_READONLY) and ($products_options->fields['products_options_values_name'] != '')) {

$tmp_html .= $products_options_details . '<br />';

} else {

$zv_display_select_option ++;

}

What did I misunderstand?

[Ajeh]

If you look for the constant, I believe there are 2 places to reference in that file ...

[jeking]

Linda,

What is the constant you're referring to? Is it PRODUCTS_OPTIONS_TYPE_READONLY?

If so, I did find two other references but I don't know what to do with them.

[Ajeh]

The constant is the:
PRODUCTS_OPTIONS_TYPE_READONLY

I forget off hand the code in 1.2.7 but it is similar ...

In the first location I change the code to:

                  // Read Only - just for display purposes
                  if ($products_options_names->fields['products_options_type'] == PRODUCTS_OPTIONS_TYPE_READONLY) {
                    //            $tmp_html .= '<input type="hidden" name ="id[' . $products_options_names->fields['products_options_id'] . ']"' . '" value="' . stripslashes($products_options->fields['products_options_values_name']) . ' SELECTED' . '" />  ' . $products_options->fields['products_options_values_name'];
                    if (empty($products_options_details)) {
                      $tmp_html = '';
                    } else {
                      $tmp_html .= $products_options_details . '<br />';
                    }
                  } else {
                    $zv_display_select_option ++;
                  } 

In the next location I change the code:

                  // READONLY
                  case ($products_options_names->fields['products_options_type'] == PRODUCTS_OPTIONS_TYPE_READONLY):
                  if (empty($tmp_html)) {
                  } else {
                  $options_name[] = $products_options_names->fields['products_options_name'];
                  $options_menu[] = $tmp_html . "\n";
                  $options_comment[] = $products_options_names->fields['products_options_comment'];
                  $options_comment_position[] = ($products_options_names->fields['products_options_comment_position'] == '1' ? '1' : '0');
                  }
                  break; 

[jeking]

Linda,

You are a dream! Thank you, thank you for not giving up on me. Now it works perfectly!

[Ajeh]

You are most welcome ...

Keep in mind when you add vs edit ...

It will probably wipe out the Option Value on the edit if you add HTML code in there so keep a copy handy ...

This is a way to "cheat" on maintaining and updating common data on products ... it is not by any means perfected ...

Plain text is fine, but get into the fancy stuff and it makes messies on the edit ...

[jeking]

Linda,

Is the below code right for 1.3.7? I think I figured out why my readonly attributes where displaying when empty. I didn't carry over the mod when I upgraded. When I install it new, I'm getting a parse error.

The constant is the:
PRODUCTS_OPTIONS_TYPE_READONLY

I forget off hand the code in 1.2.7 but it is similar ...

In the first location I change the code to:

                  // Read Only - just for display purposes
                  if ($products_options_names->fields['products_options_type'] == PRODUCTS_OPTIONS_TYPE_READONLY) {
                    //            $tmp_html .= '<input type="hidden" name ="id[' . $products_options_names->fields['products_options_id'] . ']"' . '" value="' . stripslashes($products_options->fields['products_options_values_name']) . ' SELECTED' . '" />  ' . $products_options->fields['products_options_values_name'];
                    if (empty($products_options_details)) {
                      $tmp_html = '';
                    } else {
                      $tmp_html .= $products_options_details . '<br />';
                    }
                  } else {
                    $zv_display_select_option ++;
                  } 

In the next location I change the code:

                  // READONLY
                  case ($products_options_names->fields['products_options_type'] == PRODUCTS_OPTIONS_TYPE_READONLY):
                  if (empty($tmp_html)) {
                  } else {
                  $options_name[] = $products_options_names->fields['products_options_name'];
                  $options_menu[] = $tmp_html . "\n";
                  $options_comment[] = $products_options_names->fields['products_options_comment'];
                  $options_comment_position[] = ($products_options_names->fields['products_options_comment_position'] == '1' ? '1' : '0');
                  }
                  break;