MSRP contrib

[CharInLasVegas]

I just installed this mod using the original .zip and all the fixes mentioned throughout the thread but it has wiped out my footer & right column and in my right column I have this error:

"1054 Unknown column 'products_msrp' in 'field list'
in:
[select products_msrp, products_priced_by_attribute from zen_products where products_id = '1463']"

I also have the 1054 error in admin when trying to navigate to edit a product. I did install the sql patch through phpMyAdmin and didn't get any errors.

I have reuploaded my unchanged files but still have the error, so I guess I need to remove the sql OR figure out why I'm getting the 1054 error. Can someone please help me? I'm lost

[schoolboy]

I installed this on a client's site recently and had no problems.

I decided to do MANUAL EDITS to my files, rather than FTP the module, because my files (that the module otherwise overwrites) are already customized.

When you applied the sql patch, did you do this via Admin >>> Tools, ... or via phpMyAdmin ?

If you do it via phpMyAdmin, and your dbase has PREFIXES, then you will have problems.

Small SQL patches are always best loaded via the admin panel, as it will automatically append a prefix if you use one.

[CharInLasVegas]

I tried to do it via admin, but I couldn't get it to work. I did it then through phpMyadmin.

I thought I figured out what I did wrong - I didn't put on the prefix which when I look in my config file it says my db prefix is "zen_". So I tried reinstalling the patch with the prefix, but no luck.

I also tried to uninstall, including restoring my db but the error is still there

So, I'm down for maintenance until I get this straightened out...

[CharInLasVegas]

This is what happens when I try to install through zen admin:

1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'products ADD products_msrp DECIMAL( 15, 4 ) NOT NULL AFTER products_price' at line 1
in:
[ALTER TABLE zen_ products ADD products_msrp DECIMAL( 15, 4 ) NOT NULL AFTER products_price ;]
If you were entering information, press the BACK button in your browser and re-check the information you had entered to be sure you left no blank fields.

[CharInLasVegas]

Nevermind, stupid mistake on my part. It's now working perfectly.

Mentioned quite a few pages back -- has anyone figured out how to put the order as follows: MSRP, Our Price, You Save? I think that's a more sensible order.

This is a great mod, I'm so happy it's working

[countrycharm]

Hi I have a question. When I do a W3C validation I'm getting a lot of errors because of this add on. Love the add on but I need to some how correct this errors. You can see them here
What file do I need to look at. Thank you in advance.

Still looking a answer to this.

[CharInLasVegas]

Mentioned quite a few pages back -- has anyone figured out how to put the order as follows: MSRP, Our Price, You Save? I think that's a more sensible order.

This is a great mod, I'm so happy it's working

Does anyone have a clue as to where to look to change this?

[barco57]

@countrycharm:
You need to go into the template for that display and change all of the break tags to

<br /> from <br>

[barco57]

@countrycharm and CharInLasVegas
You should be able to rearrange the following section in includes/functions/functions_prices.php (about line 295 to 311):

 // don't show the $0.00
	  if(($display_msrp_price!='')&&($display_msrp_price!='0')){
	  $save=$display_msrp_price-$display_normal_price;
      	$final_display_price = '<span class="smalltext1"><s>'.TEXT_PRODUCT_MSRP.$currencies->display_price($display_msrp_price, zen_get_tax_rate($product_check->fields['products_tax_class_id'])).'</s></span><br><span class="save">'.TEXT_PRODUCT_SAVE .$currencies->display_price($save, '').'&nbsp;&nbsp;('. number_format((($display_msrp_price -  $display_normal_price) / $display_msrp_price), 2, '.', '') * 100 . '%)'.'</span><br><span class="norm">'.TEXT_PRODUCT_OUR.$show_special_price . $show_sale_price . $show_sale_discount.'</span>' ;
	  }else{
	  	$final_display_price = $show_special_price . $show_sale_price . $show_sale_discount;
	  }
    } else {
	 if(($display_msrp_price!='')&&($display_msrp_price!='0')){
	 $save=$display_msrp_price-$display_normal_price;
      	$final_display_price = '<span class="smalltext1"><s>'.TEXT_PRODUCT_MSRP.$currencies->display_price($display_msrp_price, zen_get_tax_rate($product_check->fields['products_tax_class_id'])).'</s></span><br><span class="save">'.TEXT_PRODUCT_SAVE .$currencies->display_price($save, '').'&nbsp;&nbsp;('. number_format((($display_msrp_price -  $display_normal_price) / $display_msrp_price), 2, '.', '') * 100 . '%)'.'</span><br><span class="norm">'.TEXT_PRODUCT_OUR.$show_normal_price . $show_special_price . $show_sale_price . $show_sale_discount.'</span>' ;
	  }else{
	  	$final_display_price = $show_normal_price . $show_special_price . $show_sale_price . $show_sale_discount;
	  }
    }
	    
// If Free, Show it

This would also be the section to fix the break tags

[countrycharm]

@countrycharm:
You need to go into the template for that display and change all of the break tags to

<br /> from <br>

Thank You that worked out great. I'm still struggling to do what CharInLasVegas suggested. I just can't seem to get it to rearrange in the right order. Hmmm Thanks again