shineblue

Yes, of course. My mistake…
It is returning false, which it should.

After some further testing I found out there appears to be no issue with using ‘/amp/’ instead of ‘/?amp’. All posts/pages work with both endpoints perfectly.

The main issue is that CM Tooltip Glossary is not accepting ‘/amp/’ as an endpoint for terms.

As a temp solution I’ve changed a small bit of code in amp-helper-functions.php
if ( $use_query_var ) { (rule 119)
changed into
if ( $use_query_var || 'glossary' === get_post_type( $post_id )) {

So now I have ‘/amp/’ for all posts/pages, and ‘/?amp’ for the CM Tooltip Glossary terms. @creativemindssolutions, could you look into this? I have no idea where to start.

• This reply was modified 5 years, 8 months ago by shineblue.

is_post_type_hierarchical( get_post_type( $post_id ) ) returns false for both ‘post’ and ‘glossary’.

What I did find out is that empty ( $structure ) is returning 1. Basically this means the option ‘permalink_structure’ is somewhere set to true.

Hi Weston

Already done that (a couple of times).
Alas

Hi James

I’m the client of @creativemindssolutions with the 404 issue.

Since I’ve installed AMP a couple of days ago, all pages/posts work fine with AMP. Except for the terms pages of CM Tooltip Glossary.

As stated above, url + “amp/” gives a 404 error, url + “?amp” works fine.

Deactivating WP Rocket is not making any difference in behaviour.

Regards
Winnie