"Add to Cart" problem

Your missing the closing tag on the link (in red below)

<% If pcf_AddToCart(pIdProduct)=True Then %><a href="instPrd.asp?idproduct=<%=pIdProduct%>"><img src="<%=rslayout("add2")%>" alt="<%=dictLanguage.Item(Session("language")&"_altTag_2")& pDescription %>"></a></p>
<% Else %>
            <p class="pcShowProductLink"><a href="viewPrd.asp?idproduct=<%=pIdProduct%>&idcategory=<%=pIdCategory%>"> <img src="<%=rslayout("add2")%>" alt="<%=dictLanguage.Item(Session("language")&"_altTag_2")& pDescription %>"></a></p>    
<% End If %> 

Matt, I leave it up to you guys to post it into the WIKI.  I think a lot of PC users will like this simple adjustment.  Besides, you are the one that posted the original fix.  I just tweaked it a little.  Thanks for pointing me into the right direction if the first place.  I'm just so happy right now, and no more dump calls or emails asking when the items are back in stock.  Crazy that a little visual missing image can cause so much confusion among shoppers.  

Well, that is way beyond my ASP knowledge.  Newer learned ASP, just C over 15 years ago.  I just looked at the code in the pcShowProductH.asp file you mentioned and kind of copied it.   Simply tried to make sense out of what I saw.  It's version 3.51.  Perhaps someone else can give a better code segment.  For now I'm happy, because there is no fake button which returned be back to the category.  Which makes the customer wonder what is going on.   Now it goes to the product details page, and gives the apparel choices.  It looks better that way on the site, and I don't get confused customers.

Matt, thanks for the code.  It was almost right.  I did change it a little bit, and it works.  That's from someone who does not know ASP.  LOL  So now when you have apparel add one products you will see a link from the "Add to cart" image to the product details page.  That will fix the issue of customers thinking the items are out of stock, since it shows an add to cart button.  Can someone please confirm that the code is correct from it's syntax.  It does work so I think it is ok.

However, the general idea is to add an "Else" statement.  Here is an example using the horizontal product view "pcShowProductH.asp".

Find:

        <% If pcf_AddToCart(pIdProduct)=True Then %><p class="pcShowProductLink"><a href="instPrd.asp?idproduct=<%=pIdProduct%>"><img src="<%=rslayout("add2")%>" alt="<%=dictLanguage.Item(Session("language")&"_altTag_2")& pDescription %>"></a></p><% End If %>

Replace with:
<% If pcf_AddToCart(pIdProduct)=True Then %><a href="instPrd.asp?idproduct=<%=pIdProduct%>"><img src="<%=rslayout("add2")%>" alt="<%=dictLanguage.Item(Session("language")&"_altTag_2")& pDescription %>"></a></p>
<% Else %>
            <p class="pcShowProductLink"><a href="viewPrd.asp?idproduct=<%=pIdProduct%>&idcategory=<%=pIdCategory%>" <img src="<%=rslayout("add2")%>" alt="<%=dictLanguage.Item(Session("language")&"_altTag_2")& pDescription %>"></a></p>    
<% End If %> 

I guess one that links you to the product details page.  PC needs it.