Okay
  Public Ticket #206661
recommendations for next update
Closed

Comments

  • mino started the conversation
    hi, if possible, in the next update, I'd ask you to consider this couple of things: - I had problems with microdata from schema.org. Google said that the date wasn't in correct ISO format. I fix it by changing the function get_the_date () in get_the_date('c'), but I recommend to make this change in the next update. This is important to get the snippet stars in Google's serps. - Another thing I suggest is to change all the include(); with the function get_template_part(), otherwise it will not work well child themes thanks
  •  93
    djwd replied

    Hi there,

    thanks for your feedback.

    May I know what the 'c' in get_the_date('c') stands for? Couldn't find this in WP codex.

    Include() has different purpose then get_template_part(), although I could edit hard coded paths with variables if this is causing problems (didn't have complains so far). 

    Regards
    djwd  

    Regards
    djwd

  •   mino replied privately
  •  93
    djwd replied

    Mmm It looks like it didn't as I can't see no replies after mine

    Regards
    djwd

  • mino replied

    the get_the_date('c') function returns date in ISO 8061 format. If i change the date format on my admin panel, for example in [DD-MM-YYYY], Google don't accept the snippet and I can't have star on result serp. I use this tool for check the schema.org  snippet

    http://www.google.com/webmasters/tools/richsnippet...

    I tried to change the date function (above) with my child-theme. I copy this file \partials\part-review-stars.php on my child directory powermag-child\partials\part-review-stars.php and modify the file. But this solution don't work. I think the problem is include() function, because i read this " PHP has a built in include() statement for this purpose, but these WordPress template tags make including certain specific files much easier." in this codex http://codex.wordpress.org/Include_Tags

    and "Using with theme subfolder" section in http://codex.wordpress.org/Function_Reference/get_... explain that the get_template_part is used for include template file. But I'm not very sure that this solution work well.

  •  93
    djwd replied

    I also used that tool and reviews as this one shows up fine:

    http://pmag.wpengine.com/blog/lara-croft-is-back-g...

    I'll dig into this a bit more though.

    I'm a wordpress theme developer I know WP include tags ;) , get_template_part() is used in 90% of cases throughout the theme (all files in "partials" folder are included as such), although for a number of reason is necessary to use the php include() tag in some cases. I can probably fix any chil theme compatibility issue by setting a variable path though.

    Thanks for your feedback.

    Regards
    djwd

    Regards
    djwd