[SOLVED] Tags link not working properly?

[Ray UK]

Hi,

Im running PS 8.1.7.

So Ive decided to use the tags feature as found in Product/Options/Tags.

Ive managed to get the tags to show on the product page as follows

But when I click, say the "3 for £30" tag, it will go to /search?tag=3+for+£30 but shows no results.

Also, how do I go about saying "If tag exists {code to display tag}" so that an empty tag isnt showing on every product.

for example, if !exists="tag"

Any Ideas on how to get these working. 

Thanks

Edited September 30, 2024 by Ray UK (see edit history)

[Ray UK]

Does this tag function, not function anymore ?

There doesn't seem to be a "Tag block" module any more, and the tag search doesn't work either.

[Ray UK]

This is what I currently have.

{if $product.tag|@count > 0}
             <ul class="tags">
               {foreach from=Tag::getProductTags(Tools::getValue('id_product')) key=k item=v}
               {foreach from=$v key=lang_id item=value}
               <li><a href="{$link->getPageLink('search', true, NULL, "tag={$value|urlencode}")}">
                 {$value|escape:html:'UTF-8'}</a></li>
               {/foreach}
              {/foreach}
            </ul>
            {/if}

Its the first line that must be wrong.

Maybe I need to count the tags and put them in a variable so I can just use it like this.

{if $producttagcount > 0}

 

[Andrei H]

Hello,

The 'Tag' functionality should still work as expected as stated in the documentation, but the official 'Tag block' module seems to be achieved for quite some time now as you can see in the official public repository

Did you display these tags through a custom module/custom code?

From what I can see, the structure for the search tag should be the normal one (eg /search?controller=search&s=Zero+Nicotine), not /search?tag=Zero+Nicotine

The smarty template code does look good. You would want to debug that $product.tag to see what it contains, maybe it has an empty element, thus count being 1, instead of 0

[Ray UK]

Hi Andrew,

thanks for your reply.

I have only added that above code to my product.tpl.  If I remove the “if statement” then the rest of the code does show the tag correctly and makes those links, ie “/search?tag=3+for+£30”

But it showss an empty tag, when the product has not tags.

[Andrei H]

Hello,

Sorry for the delay.

Does the following code work for you?

{if Tag::getProductTags(Tools::getValue('id_product'))}
  <ul class="tags" style="text-align: right">
    {foreach from=Tag::getProductTags(Tools::getValue('id_product')) key=k item=v}
    {foreach from=$v key=lang_id item=value}
    <li><a href="{$link->getPageLink('search', true, NULL, "controller=search&s={$value|urlencode}")}">
      {$value|escape:html:'UTF-8'}</a></li>
    {/foreach}
    {/foreach}
  </ul>
{/if}

I think it should work fine for 'Zero Nicotine'. For '3 for £30' it does not seem to work. I think it is because it contains too few letters.

[Ray UK]

15 hours ago, Andrei H said:

Hello,

Sorry for the delay.

Does the following code work for you?

{if Tag::getProductTags(Tools::getValue('id_product'))}
  <ul class="tags" style="text-align: right">
    {foreach from=Tag::getProductTags(Tools::getValue('id_product')) key=k item=v}
    {foreach from=$v key=lang_id item=value}
    <li><a href="{$link->getPageLink('search', true, NULL, "controller=search&s={$value|urlencode}")}">
      {$value|escape:html:'UTF-8'}</a></li>
    {/foreach}
    {/foreach}
  </ul>
{/if}

I think it should work fine for 'Zero Nicotine'. For '3 for £30' it does not seem to work. I think it is because it contains too few letters.

Thank you Andrei,

the "if" statement is working fine now, and the search is also working fine.

The word "for" was in the fuzzy search word list to ignore so I removed that and it worked but showed lost of results so I changed that tag to "3-for-£30" and its working fine. (https://www.northwestecigs.uk/disposable-vapes/558-1786-lost-mary-bm6000-disposable-vape-2-nic-salt.html#/432-flavour-banana_volcano)

Many thanks

[Divine]

Hello,

Don't forget to edit your first post and add [ Solved] in the title

[Reeder]

On 9/27/2024 at 6:44 PM, Andrei H said:

Hello,

Sorry for the delay.

Does the following code work for you?

{if Tag::getProductTags(Tools::getValue('id_product'))}
  <ul class="tags" style="text-align: right">
    {foreach from=Tag::getProductTags(Tools::getValue('id_product')) key=k item=v}
    {foreach from=$v key=lang_id item=value}
    <li><a href="{$link->getPageLink('search', true, NULL, "controller=search&s={$value|urlencode}")}">
      {$value|escape:html:'UTF-8'}</a></li>
    {/foreach}
    {/foreach}
  </ul>
{/if}

I think it should work fine for 'Zero Nicotine'. For '3 for £30' it does not seem to work. I think it is because it contains too few letters.

 

 

Hello,
how can I improve this code to take languages into account?

After using it, I am getting blocked tags in two languages in my product.

Thank you for your help

 

 

[Andrei H]

Hello,

For displaying only the tags for the current language you can use the following code:

{assign var="all_tags" value=Tag::getProductTags(Tools::getValue('id_product'))}
{if !empty($all_tags)}
  <ul class="tags" style="text-align: right">
    {foreach from=$all_tags key=k item=v}
      {if $k === $language.id}
        {foreach from=$v key=lang_id item=value}
          <li>
            <a href="{$link->getPageLink('search', true, NULL, "controller=search&s={$value|urlencode}")}">
              {$value|escape:html:'UTF-8'}
            </a>
          </li>
        {/foreach}
      {/if}
    {/foreach}
  </ul>
{/if}

I haven't noticed it in the initial posts, but that code was running the same database query twice. The above code will run it only once.

Edited May 8 by Andrei H (see edit history)

[Reeder]

 

4 hours ago, Andrei H said:

Hello,

For displaying only the tags for the current language you can use the following code:

{assign var="all_tags" value=Tag::getProductTags(Tools::getValue('id_product'))}
{if !empty($all_tags)}
  <ul class="tags" style="text-align: right">
    {foreach from=$all_tags key=k item=v}
      {if $k === $language.id}
        {foreach from=$v key=lang_id item=value}
          <li>
            <a href="{$link->getPageLink('search', true, NULL, "controller=search&s={$value|urlencode}")}">
              {$value|escape:html:'UTF-8'}
            </a>
          </li>
        {/foreach}
      {/if}
    {/foreach}
  </ul>
{/if}

I haven't noticed it in the initial posts, but that code was running the same database query twice. The above code will run it only once.

It works perfectly, thank you very much!