Check if PriceGroup is Assigned to Product

[lesliekirk]

I'm wanting to use the code snippet (a modified version of this) on the CTGY page

<mvt:do file="g.Module_Feature_PGR_DB" name="l.ok" value="PriceGroup_Load_Name( 'Volume Pricing', l.price_group )" />
<mvt:if expr="l.price_group:id GT 0">
<mvt:do file="g.Module_Feature_PGR_DB" name="l.b" value="PriceGroupXProduct_Load( l.price_group:id , l.settings:product:id, l.settings:pricegroupvar )" />
<mvt:if expr="NOT ISNULL l.settings:pricegroupvar">
Assigned to this product<br />
</mvt:if>
</mvt:if>
</mvt:if>

The snippet works as expected on the PROD page, but on the CTGY it's displaying for every product. What have I goofed up this time?

[Scot - ScotsScripts.com]

It's possible that the pricegroupsvar array is already populated. When you add to an array it doesn't get rid of previous values, so you could put this right above the PriceGroupXProduct_Load line to clear entries that are already in it.

<mvt:assign name="l.settings:pricegroupvar" value="''" />

OR, do you have it loading while it's in the mvt:while or mvt:foreach loop? That could load more values into it as well.

OR, it's in a loop and you still need to reset the variable with the code above so it doesn't keep loading.

I'm not 100% sure what you're doing there so basically these are the things I'd personally first look at.

[ids]

My typical blurb, not sure what the goal is.

If you need to identify if each product in that category is assigned a pgrp, you need a loop, like Foreach to grab the status of each product ID in the category product listing. I don't think speed will be an issue, but the loop already exists for rendering each product.

Scott

[lesliekirk]

My typical blurb, not sure what the goal is.


Scott

The goal is to display an icon on the category page that indicates there is Volume Pricing for the product if the customer is assigned to that price group.

[Bruce - PhosphorMedia]

Try this:

<mvt:do file="g.Module_Feature_PGR_DB" name="l.ok" value="PriceGroup_Load_Name( 'Volume Pricing', l.price_group )" />
<mvt:if expr="l.price_group:id GT 0">
     <mvt:do file="g.Module_Feature_PGR_DB" name="l.b" value="PriceGroupXProduct_Load( l.price_group:id , l.settings:product:id, l.settings:pricegroupvar )" />
     <mvt:if expr="NOT ISNULL l.settings:pricegroupvar">
          Assigned to this product<br />
     </mvt:if>

</mvt:if>
<mvt:assign name="l.settings:pricegroupvar" value="''"/>

Also, I'd consider putting the

<mvt:do file="g.Module_Feature_PGR_DB" name="l.ok" value="PriceGroup_Load_Name( 'Volume Pricing', l.price_group )" />

function globally and change l.price_group to a global (i.e., g.price_group_name) so you don't need to keep calling it inside loops. Probably not the end of the world, but certainly not good coding practice.

[lesliekirk]

Thanks, Bruce - PhosphorMedia - that got me in the direction I needed to go!