AgoraCart Free User Forums

I've been adding a small amount of custom code to my productPage-search.inc page to allow for the inclusion of two
different 'Display' links based on the %%CategoryID%% token.

The code is shown below:

<TD COLSPAN=2 align=right>

<!--agorascript-pre
&capture_STDOUT;

# Conditional Link Printing - we need to display two different page layouts, one for BAC libraries
# and the other for Nylon Filters/Pools and Superpools

my $product_name = "%%CategoryID%%";

if($product_name eq 'Nylon_Filters' || $product_name eq 'Pools_and_Superpools')
{

print"<A HREF=\"%%scripturl%%?cart_id=&p_id=%%ProductID%%&xm=on&ppinc=1a1\">";

}
elsif($product_name eq 'BAC_Library')
{

print"<A HREF=\"%%scripturl%%?cart_id=&p_id=%%ProductID%%&xm=on&ppinc=1a\">";

}
# End conditional html printing

&uncapture_STDOUT;

-->

<FONT FACE=ARIAL>Display</FONT></A>
</TD>

The trouble is that no links are printed, obviously because the 'if' statements are not returning true. However, they should be. Printing out
the variable $product_name when debugging correctly returns one of the three product categories we have in our store: Nylon_Filters, Pools_and_Superpools, and BAC_Library. Since these are the only three possibilities, and they are covered by my if statement, I don't understand why the links are not being conditionally displayed in my browser.

Is it the way agora.cgi is handling the code, or have I just been staring at the page for too long and can't see the glaring error that is looming there?

Thanks once again for your help


_________________
I could set the building on fire..
http://www.punchcard-pcs.com/gaming_graveyard

you need to encapsulate your tests:

if ( (test1) || (Test2) ) {

}


_________________
a common man built the Arc,
Modern Engineers built the Titanic.

The order of precedence is 'eq' before 'or', and in any case trying that change did not resolve the issue. Thank you though for the suggestion.

Can anyone else offer a solution?

Thanks again,


_________________
I could set the building on fire..
http://www.punchcard-pcs.com/gaming_graveyard

hell-o
instead of %%CategoryID%% and %%ProductID%% just try %%product%%. this will return the category name.
regards,
dan

I tried %%product%%, but that value doesn't return anything to the variable when I try to assign it to my $product_name variable.

I've tried pattern matching expressions in this code as well to see if maybe there was some sort of hidden formatting or something else I was missing, and it just is not working. I'm scratching my head over this one.

If you want to check it out, here is the store URL:

http://www.genomex.com/cgi-bin/library_store/agora.cgi

Run a search on something like 'deer', and you'll see it doesn't come
up with any 'Display' links, but I am having it print the value of $product_name in its place.

Thanks for the ideas thus far.


_________________
I could set the building on fire..
http://www.punchcard-pcs.com/gaming_graveyard

hell-o

looking at your code again you have only 1/3 of the href link. you need to have the link name printed and the </a>:


<!--agorascript-pre
&capture_STDOUT;

# Conditional Link Printing - we need to display two different page layouts, one for BAC libraries
# and the other for Nylon Filters/Pools and Superpools

my $product_name = "%%CategoryID%%";

if($product_name eq 'Nylon_Filters' || $product_name eq 'Pools_and_Superpools')
{

print"<A HREF=\"%%scripturl%%?cart_id=&p_id=%%ProductID%%&xm=on&ppinc=1a1\"> %%ProductID%% </a>";

}
elsif($product_name eq 'BAC_Library')
{

print"<A HREF=\"%%scripturl%%?cart_id=&p_id=%%ProductID%%&xm=on&ppinc=1a\"> %%ProductID%% </a>";

}
# End conditional html printing

&uncapture_STDOUT;

-->

without the link name and the </a> you wont get anything printed on the screen though it maybe there in the source as a malformed link.

regards,
dan