E107: meta tags stopped work

[Jimmi08]

I noticed this with my plugin, but it is happened with news too. (on older version it works, on newest not)
I use e107::meta() function, but now it has be used before
require_once (HEADERF);
otherwise meta tags are not loaded.

It should work this way? I like it :)

[lonalore]

Yes, it should work this way. You need to set meta tags before printing header out.

By the way, I think we need to improve "meta-tag" section of e107. The best would be to provide support for commonly used meta tags, including the page title, canonical URL, description, etc, and for Facebook's Open Graph, Twitter Cards and Dublin Core.

The best would be to provide a way to set default values for each meta tag under the "Meta Tags" section:

Furthermore, to provide a way to override default values on each page/news/etc.. item. For example, on the "Create Page/Menu" form there would be a "Meta tags" tab to set unique values for the current page item. If not set, the default meta values would be used.

I collected a large number of commonly used meta tags here:
https://gist.github.com/lonalore/fdffcfdc4148bf8cb829

Goals:

• Global settings to control (default) meta tags on all pages.
• Ability to override meta tags on each content item (page, news item).

[Jimmi08]

So news and other pages need to rewrite because they use HEADERF before e107::meta(), this was changed.

By the way, awesome list. Thanks. I needed it 2 days ago :) From my experience (just used FB debugger and need only sharing plugin pages)

• FB tags needs right order of meta tags
• you can't use description with type article
• you should use type website just for homepage
• you can use your own types, so the list is neverending story.

I generate content of meta tags from other fields / not edited by user, there should still be this posibility. e107::meta() works fine, just to have possibility to add any of them in admin and set order and default value.

[Moc]

So news and others still require changing the order of e107::meta() and loading the header?

[Jimmi08]

No, I think that this was somehow solved, I can see meta description in news from latest github. In plugin you need at first use e107::meta() and then HEADERF

[Moc]

Alright