How Does the Meta Block in Base Template Work? #613

wesleyboar · 2020-06-25T17:19:58Z

Description

I am confused by purpose of the meta block in base.html template code.

Steps to reproduce

Confirm working djangocms-meta setup:
1. Have working djangocms-blog install.
2. Have custom djangocms-blog app templates, that still has this code in base.html:
```
{% block meta %}
    {%  if meta %}
        {% include "meta/meta.html" %}
    {% endif %}
{% endblock meta %}
```
3. Have working configuration for djangocms-meta app. This includes this code in base.html CMS template:
```
<head …>
    …
   {% include "meta/meta.html" %}
    …
</head>
```
Try to break working djangocms-meta setup:
- Remove the {% block meta %} block of code from base.html app template (not the CMS template).
Try, with more force, to break working djangocms-meta setup:
- Remove the {% include "meta…" %} line of code from base.html CMS template (not the app template).

Expected result

💡 The code renders something somewhere.
💡 Removal of the code does not render something somewhere.
💡 Removal of the code does not render something somewhere.

Actual result

✓ The code renders <meta> tags in the <head>.
🤷‍♂️ Removal of the code seems to render everything the same (as in step 1) everywhere.
✓ Removal of the code does not render <meta> tags in the <head>.

The text was updated successfully, but these errors were encountered:

yakky · 2020-07-11T18:21:43Z

@tacc-wbomar could you check if the documentation and comment added in #628 helps clarifying the social meta tags behavior and the role of the {% block meta %}?

wesleyboar · 2020-08-11T14:53:07Z

Yes. The {% block meta %} is clear*, now. Thanks!

(I will try out the "complete social meta tag rendering", like BLOG_FB next.)

* Update: Or not. See #613 (comment).

wesleyboar · 2020-08-11T15:22:34Z

Regarding BLOG_FB, BLOG_TWITTER, BLOG_GPLUS/BLOG_SCHEMAORG in settings:

~~I cannot find any results in djangocms-blog repo (searching via Github), nor (desperation) any results on the Internet (Google search). Are these future (1.2) configuration options?~~

Update: I stumbled upon the docs for these settings. I could not find them in the djangocms-blog repo (I think) because that file is not in the develop branch.

wesleyboar · 2020-08-11T18:41:55Z

After experimenting with {% if meta %}, I have found that it is not as reliable as expected. This may be because of my ignorance. Below is what I learned and my workarounds. Please, tell me whether any of these problems should be a new issue in this repo or another repo.

Setup

I have functional CMS.
I have functional page meta.
I have functional blog (article) meta.

Problems

1. Page Meta: Making it Work (with Blog Meta?)

✓ Resolved by option "c" in new documentation.

Problem

Using {% include "meta/meta.html" %} to render page metadata (source djangocms-page-meta → source djangocms-meta) did not work for me.

Workaround

…/templates/base.html

{% load … meta page_meta_tags … %}
{% page_meta request.current_page as page_meta %}
<!doctype html>
<html>

<head {% meta_namespaces %}>
  …
  {% block meta %}
    {% include 'djangocms_page_meta/meta.html' with meta=page_meta %}
  {% endblock meta %}

I do not remember from where I copied/learned the pieces of the workaround code.

2. Logic: Overwrite Block Even If There Is No `meta`

✓ There has since been uncertainty about original solution and a new working solution.

Problem

Using {% block meta %} {% if meta %} … {% endif meta %} {% endblock %} (as documentation offers) will overwrite the block, even if I have no meta with which to overwrite the block.

Workaround

Using {% if meta %} {% block meta %} … {% endblock meta %} {% endif %}, so that I overwrite the block only if I have meta with which to overwrite the block.

3. Data: At Least One `meta` is Available to Template

✓ The here since been a re-explanation and a new working solution.

Problem

Using the documentation, and having both page meta and blog meta:

These values are available on a blog article page (in this order):
1. 'meta': <meta.views.Meta object at 0x7fef6214aeb8>,*
2. 'meta': <module 'meta' from '/usr/local/lib/python3.6/site-packages/meta/__init__.py'>,*
This values is available on any page:
- 'meta': <module 'meta' from '/usr/local/lib/python3.6/site-packages/meta/__init__.py'>,*

Thus:

✓ On a blog article page, I get blog meta.
X On a blog article page, I do not get page meta.
- Because, the 'meta': <module 'meta' …> makes {% if meta %} evaluate to true.

* Data retrieved via {% debug %}.

Workaround

…/templates/djangocms_blog/base.html

…
{# A certain code block should be here, but it would cause a bug. #}
{# SEE: `./post_detail.html` #}
{# SEE: https://github.com/nephila/djangocms-blog/issues/613 #}
…

…/templates/djangocms_blog/post_detail.html

…
{# This conditional is irrelevant, because we will always have
   `meta` in this template. This code block belongs in `./base.html`,
   which would benefit from a conditional, but also cause a bug. #}
{# SEE: https://github.com/nephila/djangocms-blog/issues/613 #}
{% if meta %}
  {% block meta %}
    {% include "meta/meta.html" with meta=meta %}
  {% endblock meta %}
{% endif %}
…

yakky · 2020-08-19T22:41:51Z

@tacc-wbomar this further documentation update should help clarify the proper template layout #630

djangocms-page-meta documentation is actually outdated and I will amend that as well

I also included your suggestion at point 2

Regarding point 3, blog overrides page meta on purpose as you can't have both

yakky · 2020-08-19T22:46:52Z

check updated documentation https://djangocms-blog--630.org.readthedocs.build/en/630/features/meta.html

and updated settings documentation https://djangocms-blog--630.org.readthedocs.build/en/630/autodoc/settings.html (which is not updated on RTD due to a build failure)

wesleyboar · 2020-08-26T16:34:21Z

Thank you.

Status

Problem 1 is solved via option "c".
Problem 2 workaround was me inexpertly trying to solve Problem 3. I trust what you decide.
Problem 3 I explained poorly. I gave data without stating the problem.

Expected Result

Metadata is rendered for Home page, Blog page, and any Blog post.

Actual Result

Metadata is rendered for Home page, and any Blog post.
Metadata is not rendered for Blog page.

Workaround

Move Meta Include Logic From `djangocms_blog/base.html` to `djangocms_blog/post_detail.html`

This does does change Actual Result to match Expected Result.
I am not suggesting this, because I don't know. I am merely reporting what seemed to work after trial and error.

…
    …
        {% include "meta/meta.html" %}
    …
…

Failed Solutions

Remove Surrounding Meta Include Logic from `djangocms_blog/base.html`

This does does change Actual Result, but the Blog post has Blog page metadata.

{% include "meta/meta.html" %}

Change Meta Include Logic In `djangocms_blog/base.html`

This does not change Actual Result.

{% block meta %}
    {% if meta %}
        {% include "meta/meta.html" %}
    {% endif %}
{% endblock meta %}

This does not change Actual Result.

{% if meta %}
    {% block meta %}
        {% include "meta/meta.html" %}
    {% endblock meta %}
{% endif %}

wesleyboar · 2020-08-26T18:47:51Z

Is my Expected Result strange?

Perhaps, Blog page should not have page metadata. I really don't understand how users use social media, I'm just trying to get the metadata present on all the pages so I can complete implementation of our blog/news/social-media-metadata feature.

yakky · 2020-08-26T20:43:39Z

@tacc-wbomar I don't understand if you can see any metadata in the post detail view, or you can see the blog post ones and not the CMS page

If you can't see any metadata, that's of course an issue because the it's the whole point of the django-meta integration :)

I will try to replicate the issue in the https://github.com/yakky/djangocms-blog-poc repo

wesleyboar · 2020-08-26T21:34:04Z

With the documented setup, I do not see any meta tags on the CMS's Blog page (I see meta tags on Home page, and on a Blog post page).

I think this is all the relevant info to show my setup. If helpful, I'll include bits of settings.py.

Variations

`/base.html` w/ "If & Block" Meta Inclusion

templates/djangocms_blog/base.html

{% if meta %}
    {% block meta %}
        {% include "meta/meta.html" with meta=meta %}
    {% endblock meta %}
{% endif %}

The templates/djangocms_blog/post_detail.html has no meta inclusion.

✅ Home page (has meta)
❌ Blog page (has no meta)
✅ Blog post (has meta)

`/base.html` w/ Direct Meta Inclusion

templates/djangocms_blog/base.html

{% include "meta/meta.html" with meta=meta %}

The templates/djangocms_blog/post_detail.html has no meta inclusion.

✅ Home page (has meta)
✅ Blog page (has meta)
❌ Blog post (has Blog page meta)

`/post_detail.html` w/ "If & Block" Meta Inclusion

templates/djangocms_blog/post_detail.html

{% if meta %}
    {% block meta %}
        {% include "meta/meta.html" with meta=meta %}
    {% endblock meta %}
{% endif %}

The templates/djangocms_blog/base.html has no meta inclusion.

✅ Home page (has meta)
✅ Blog page (has meta)
✅ Blog post (has meta)

Unchanged Files

templates/base.html

{# FAQ: Conditional includes are intentionally verbose #}
{# SEE: https://confluence.tacc.utexas.edu/x/QwI9Cw #}

{% load cms_tags staticfiles sekizai_tags cache i18n %}
<!doctype html>
<html>

{% if settings.FEATURES.blog %}
  {% with path="djangocms_blog_head.html" %}{% include path %}{% endwith %}
{% else %}
<head>
{% endif %}
  …

  <!-- Metadata. -->
  <title>{% block title %} This is my new project home page {% endblock title %}</title>
  <meta charset="utf-8">
  <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
  {% if settings.FEATURES.blog %}
    {% block meta %}
      {% with path="djangocms_page_meta.html" %}{% include path %}{% endwith %}
    {% endblock meta %}
  {% endif %}

templates/djangocms_blog_head.html

{% load meta %}

<head {% meta_namespaces %}>

templates/djangocms_page_meta.html

{% load page_meta_tags %}
{% page_meta request.current_page as meta %}

{% include 'djangocms_page_meta/meta.html' %}

yakky · 2020-08-26T21:38:02Z

@tacc-wbomar the settings and a pip freeze output would be of great help, thanks!

wesleyboar · 2020-08-26T21:49:05Z

Okay.

settings.py
pip freeze
Note: The content of this file was updated to be accurate 3 minutes after sending this comment.

yakky · 2020-08-26T22:04:04Z

thanks for the details
I confirm I have been able to reproduce the issue

Can you try replacing in /templates/djangocms_blog/base.html

{% block meta %}
    {% comment %}
    This is needed if you can't add ``{% include "meta/meta.html" %}`` in the django CMS template attached to the blog
    page.
    If you have something like the snippet below in the main template used by the django CMS page, you don't need
    this block and you can safely remove it when customizing the blog templates

    ...
    <html>
        <head>
        <title>{% block title %}{% page_attribute 'title' %}{% endblock title %}</title>
        {% include "meta/meta.html" %}
        ...

    {% endcomment %}
    {% if meta %}
        {% include "meta/meta.html" %}
    {% endif %}
{% endblock meta %}

with

{% block meta %}
    {% if meta %}
        {% include "meta/meta.html" %}
    {% else %}
        {{ block.super }}
    {% endif %}
{% endblock meta %}

wesleyboar · 2020-08-26T22:06:46Z

Yes! That works.

yakky · 2020-08-26T22:09:18Z

awesome!

i updated #630 to fix the base template

Thanks a lot for your patience and your super detailed reports!

yakky · 2020-08-26T22:16:15Z

@tacc-wbomar thanks again. Closing this via merge of #630

wesleyboar mentioned this issue Jun 25, 2020

New 1.0.0 Install, No Meta Tags Rendered (meta is empty) #550

Closed

yakky added type: documentation status: approved labels Jun 27, 2020

yakky mentioned this issue Jul 11, 2020

Improve documentation to enable social meta tags rendering #628

Merged

5 tasks

yakky closed this as completed in #628 Jul 15, 2020

yakky mentioned this issue Aug 19, 2020

Doc improvements for usage with djangocms-page-meta #630

Merged

5 tasks

yakky reopened this Aug 19, 2020

yakky added the type: bug label Aug 26, 2020

yakky closed this as completed in #630 Aug 26, 2020

wesleyboar mentioned this issue Jan 21, 2021

Social Media Metadata & Blog/News Support TACC/Core-CMS#30

Closed

7 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

How Does the Meta Block in Base Template Work? #613

How Does the Meta Block in Base Template Work? #613

wesleyboar commented Jun 25, 2020

yakky commented Jul 11, 2020

wesleyboar commented Aug 11, 2020 •

edited

Loading

wesleyboar commented Aug 11, 2020 •

edited

Loading

wesleyboar commented Aug 11, 2020 •

edited

Loading

yakky commented Aug 19, 2020

yakky commented Aug 19, 2020

wesleyboar commented Aug 26, 2020 •

edited

Loading

wesleyboar commented Aug 26, 2020

yakky commented Aug 26, 2020

wesleyboar commented Aug 26, 2020 •

edited

Loading

yakky commented Aug 26, 2020

wesleyboar commented Aug 26, 2020 •

edited

Loading

yakky commented Aug 26, 2020

wesleyboar commented Aug 26, 2020

yakky commented Aug 26, 2020

yakky commented Aug 26, 2020

How Does the Meta Block in Base Template Work? #613

How Does the Meta Block in Base Template Work? #613

Comments

wesleyboar commented Jun 25, 2020

Description

Steps to reproduce

Expected result

Actual result

yakky commented Jul 11, 2020

wesleyboar commented Aug 11, 2020 • edited Loading

wesleyboar commented Aug 11, 2020 • edited Loading

wesleyboar commented Aug 11, 2020 • edited Loading

Setup

Problems

1. Page Meta: Making it Work (with Blog Meta?)

Problem

Workaround

2. Logic: Overwrite Block Even If There Is No meta

Problem

Workaround

3. Data: At Least One meta is Available to Template

Problem

Workaround

yakky commented Aug 19, 2020

yakky commented Aug 19, 2020

wesleyboar commented Aug 26, 2020 • edited Loading

Status

Expected Result

Actual Result

Workaround

Move Meta Include Logic From djangocms_blog/base.html to djangocms_blog/post_detail.html

Failed Solutions

Remove Surrounding Meta Include Logic from djangocms_blog/base.html

Change Meta Include Logic In djangocms_blog/base.html

wesleyboar commented Aug 26, 2020

yakky commented Aug 26, 2020

wesleyboar commented Aug 26, 2020 • edited Loading

Variations

/base.html w/ "If & Block" Meta Inclusion

/base.html w/ Direct Meta Inclusion

/post_detail.html w/ "If & Block" Meta Inclusion

Unchanged Files

yakky commented Aug 26, 2020

wesleyboar commented Aug 26, 2020 • edited Loading

yakky commented Aug 26, 2020

wesleyboar commented Aug 26, 2020

yakky commented Aug 26, 2020

yakky commented Aug 26, 2020

wesleyboar commented Aug 11, 2020 •

edited

Loading

wesleyboar commented Aug 11, 2020 •

edited

Loading

wesleyboar commented Aug 11, 2020 •

edited

Loading

2. Logic: Overwrite Block Even If There Is No `meta`

3. Data: At Least One `meta` is Available to Template

wesleyboar commented Aug 26, 2020 •

edited

Loading

Move Meta Include Logic From `djangocms_blog/base.html` to `djangocms_blog/post_detail.html`

Remove Surrounding Meta Include Logic from `djangocms_blog/base.html`

Change Meta Include Logic In `djangocms_blog/base.html`

wesleyboar commented Aug 26, 2020 •

edited

Loading

`/base.html` w/ "If & Block" Meta Inclusion

`/base.html` w/ Direct Meta Inclusion

`/post_detail.html` w/ "If & Block" Meta Inclusion

wesleyboar commented Aug 26, 2020 •

edited

Loading