You are here:  » Hyphenated URL's for custom 'breadcrumb' in Product page

Active Forum Topics


Hyphenated URL's for custom 'breadcrumb' in Product page

Submitted by nanaz on Wed, 2017-10-11 14:46 in

Hi David,

Hope you are well! In PTO 3.0 I would like to add a custom 'breadcrumb' in the Product page with links to category, brand and custom field(s).

I'm using the existing %DB_myfield% and added the code to be able to use %DBH_myfield% as a proper hyphenated url as described in https://www.pricetapestry.org/node/420.

WP code would be something like:

<p class="pto_pmr"><a href="/gender/%DBH_gender%/">%DB_gender%</a>
&gt;
<a href="/brand/%DBH_brand%/">%DB_brand%</a></p>

Both %DB_myfield% are okay, but the DBH field is not recognized/declared. What am I doing wrong?

Hope you can help and thank you as always,

Marlies

Submitted by support on Thu, 2017-10-12 09:08

Hello Marlies,

The modification from node 420 only applies to the Prices / Each template. To do the same for Product / Main edit pto_product.php and look for the following code at line 528:

    $html .= $html_product;

...and REPLACE with:

    if (strpos($html_product,"%DBH_"))
    {
      preg_match_all('/%DBH_(.*)%/U',$html_product,$matches);
      foreach($matches[1] as $field)
      {
        $html_product = str_replace("%DBH_".$field."%",pto_common_hyphenate($product->$field),$html_product);
      }
    }
    $html .= $html_product;

Your template code above should then work fine!

Cheers,
David.
--
PriceTapestry.com

Submitted by nanaz on Thu, 2017-10-12 09:40

Thank you David!

Submitted by nanaz on Tue, 2017-11-07 16:39

Hi David,
I'm having a problem with the custom field in the breadcrumb on the productpage. So far I've got:

<p style="text-align:right;" class="pto_pmr">
%IF_DB_category%<a href='/c/%DBH_category%/'>%DB_category%</a> ▹ %ENDIF_DB_category%
%IF_DB_gender%<a href='/gender/%DBH_gender%/'>%DB_gender%</a> ▹ %ENDIF_DB_gender%
%IF_DB_brand%<a href='/merk/%DBH_brand%/'>%DB_brand%</a>%ENDIF_DB_brand%
</p>

Category & brand work fine, but the created url (Base href) for the custom field 'gender' isn't right and results in a 404.

{link saved}

When I change its settings to <a href='/shop?pto_q=gender:%DBH_gender%'>%DB_gender%</a> the url is right, but then the IF-ENDIF statement is not recognized anymore (and I need it, because sometimes there is no gender available).

Hope you can help, thank you!

cheers, Marlies

Submitted by support on Wed, 2017-11-08 10:23

Hello Marlies,

%IF_DB_field% and %ENDIF_DB_field% should be %ENDIFDB_field% and %ENDIFDB_field% - that should be all it is, have a go with:

<p style="text-align:right;" class="pto_pmr">
%IFDB_category%<a href='/c/%DBH_category%/'>%DB_category%</a> ▹ %ENDIFDB_category%
%IFDB_gender%<a href='/shop?pto_q=gender:%DBH_gender%'>%DB_gender%</a> ▹ %ENDIFDB_gender%
%IFDB_brand%<a href='/merk/%DBH_brand%/'>%DB_brand%</a>%ENDIFDB_brand%
</p>

Cheers,
David.
--
PriceTapestry.com

Submitted by nanaz on Wed, 2017-11-08 11:55

Hi David,

The %IFDB_field% and %ENDIFDB_field% are not recognized on the productpage and are printed as text in the breadcrumb.

I made the changes described on node 420 and also used the code submitted by george-p on Tue, 2014-10-14 17:21 on the end of the page and changed pto_product.php:

    if (strpos($html_product,"%DB_"))
    {
      preg_match_all('/%DB_(.*)%/U',$html_product,$matches);
      foreach($matches[1] as $field)
      {
        $html_product = str_replace("%DB_".$field."%",$product->$field,$html_product);
      }
    }

into

if (strpos($html_product,"%DB_"))
    {
      preg_match_all('/%DB_(.*)%/U',$html_product,$matches);
      foreach($matches[1] as $field)
      {
        if ($product->$field)
        {
          $html_product = str_replace("%IF_DB_".$field."%","",$html_product);
          $html_product = str_replace("%ENDIF_DB_".$field."%","",$html_product);
          $html_product = str_replace("%DB_".$field."%",$product->$field,$html_product);
        }
        else
        {
          $html_product = str_replace("%DB_".$field."%","",$html_product);
          $html_product = preg_replace('/%IF_DB_'.$field.'%(.*)%ENDIF_DB_'.$field.'%/U','',$html_product);
        }
      }
    }

That's why I used the breadcrumb code with the _ which works with category, gender & brand, but stops being recognized when the link for gender is setup as '/shop?pto_q=gender:%DBH_gender%' instead of '/gender/%DBH_gender%/' that produces a non-existing url.

I made no further changes to pto_product.php except the ones on node 420 & this page.

Thanks,

Marlies

Submitted by support on Wed, 2017-11-08 13:14

Hello Marlies,

Ah my apologies, when I implemented the IF/ENDIF support in version 3 I didn't incorporate the underscore so I just re-created my test using the code from above and it worked OK however I did notice that the modification to swap out %DBH_fieldname% with the hyphenated version was not present so I included this resulting in the following;

  if (strpos($html_product,"%DB_"))
  {
    preg_match_all('/%DB_(.*)%/U',$html_product,$matches);
    foreach($matches[1] as $field)
    {
      if ($product->$field)
      {
        $html_product = str_replace("%IF_DB_".$field."%","",$html_product);
        $html_product = str_replace("%ENDIF_DB_".$field."%","",$html_product);
        $html_product = str_replace("%DB_".$field."%",$product->$field,$html_product);
        $html_product = str_replace("%DBH_".$field."%",str_replace(" ","-",$product->$field),$html_product);
      }
      else
      {
        $html_product = str_replace("%DB_".$field."%","",$html_product);
        $html_product = preg_replace('/%IF_DB_'.$field.'%(.*)%ENDIF_DB_'.$field.'%/U','',$html_product);
      }
    }
  }

...and this seems to be working OK with or without gender using:

<p style="text-align:right;" class="pto_pmr">
%IF_DB_category%<a href='/c/%DBH_category%/'>%DB_category%</a> ▹ %ENDIF_DB_category%
%IF_DB_gender%<a href='/shop?pto_q=gender:%DBH_gender%'>%DB_gender%</a> ▹ %ENDIF_DB_gender%
%IF_DB_brand%<a href='/merk/%DBH_brand%/'>%DB_brand%</a>%ENDIF_DB_brand%
</p>

If you could perhaps check that combination (if not what's already in place) and if still no joy if you could email me your modified pto_product.php I'll check it out further with you...

Cheers,
David.
--
PriceTapestry.com

Submitted by nanaz on Wed, 2017-11-08 14:01

Great, it's working now!
Thank you so much for your support!

Marlies