process_summary() fails to set new summary properly

[jwodder]

• Python version: 3.8.3
• Pelican version: commit 6289452e9580a95fa2a4b87eec28ca8d7142e4d8
• pelican-render-math version: 1.0.0
• BeautifulSoup4 version: 4.9.1

After process_summary() is called on an article, article.summary remains the same, even if it contains mathematics. I believe the problem is that article._summary isn't used by any code paths after you set it; perhaps you should set article.metadata["summary"] instead.

This gist serves as a demonstration of the bug. Use it as follows:

• Create a new Pelican project with the given pelicanconf.py and with article.rst placed in content/.
  • Note that pelicanconf.py sets THEME = 'simple'. This is needed to get a theme that shows the summary for the most recent article.
• Create a fresh virtual environment and install the packages listed in requirements.txt into it.
• Apply math.diff to {virtualenv}/lib/python3.8/site-packages/pelican/plugins/render_math/math.py. This adds some debugging output to pelican-render-math so we can see the value of article.summary before and after processing.
• Run pelican in the virtualenv. You will see this output:

* Summary before processing: '<p class="first last">Here we prove that <span class="math">\\(\\sum_{i=1}^n i = \\frac{n^2+n}{2}\\)</span></p>\n'
* Summary after processing: '<p class="first last">Here we prove that <span class="math">\\(\\sum_{i=1}^n i = \\frac{n^2+n}{2}\\)</span></p>\n'
Done: Processed 1 article, 0 drafts, 0 pages, 0 hidden pages and 0 draft pages in 0.13 seconds.

Note that the summary remains the same. Also note that MathJax is not present in output/index.html.

[justinmayer]

Hi @jwodder. Is this issue something you would be willing to help out with? We could really use the assistance! 😅

[jwodder]

@justinmayer My suspicion about how to fix the problem turned out to be correct; I have submitted a patch as PR #9.

[justinmayer]

@jwodder: Many thanks for fixing this! Addressed via https://github.com/pelican-plugins/render-math/commit/93f53605d47163bcb4d73631f1719ffb39f1fb92 and released as Render Math 1.0.2.