fix: a block post keeps its excerpt when the content filters are off

[dereuromark]

Found on a live site: seven of ten posts on the homepage had an empty excerpt. The three that did not were authored in another block format whose plugin handles this case.

Cause

registerFilters() gates the excerpt filter behind the content options:

if ($this->options['enable_posts'] || $this->options['enable_pages']) {
    add_filter('the_content', [$this, 'filterContent'], 5);
    add_filter('get_the_excerpt', [$this, 'filterExcerpt'], 5);
}

The block does not share that gate. It renders through its own render_callback whatever those options say. So a site with both off - one that authors in blocks rather than through the content filter - renders its posts correctly and shows nothing on every archive. I confirmed on the live site that filterExcerpt was simply absent from the get_the_excerpt callback list.

Empty rather than raw, because of core: wp_trim_excerpt() builds a missing excerpt through excerpt_remove_blocks(), which keeps an allowlist of core blocks and drops every dynamic third-party block. A post whose whole body is one wpdjot/djot block has nothing left to trim.

Fix

The filter is registered unconditionally. Doing only that would hand this plugin every excerpt on the site, including posts it has nothing to do with, so filterExcerpt() returns early when the content filters are off and the post carries no djot block.

Verified

Applied to the live site and re-fetched the homepage after a cache flush: all ten posts now carry an excerpt, where seven were empty before. composer test (107 tests), stan and cs-check all clean.

Not covered by a test

filterExcerpt() needs add_filter, has_block and a global post, and tests/bootstrap.php stubs only a few WordPress functions - there is no Plugin-level test to extend. Worth adding, but it is a bigger change than this fix and I did not want to bundle them.

Does not overlap #58, which touches filterContent/shouldFilterContent rather than registration or the excerpt.

[Copilot]

Pull request overview

This PR fixes empty excerpts for posts authored entirely as wpdjot/djot (or legacy wp-djot/djot) blocks when the plugin’s content filters are disabled, by ensuring excerpt filtering is still available for block-based content.

Changes:

• Register the get_the_excerpt filter unconditionally so block-only posts can still produce excerpts even when content filters are off.
• Add a conditional early-return in filterExcerpt() intended to avoid touching excerpts for posts that don’t contain Djot blocks when content filters are disabled.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Suppressed comments (1)

src/Plugin.php:256

• The new bailout that avoids rewriting unrelated excerpts runs after the manual-excerpt branch (if ($excerpt) { ... }). With content filters disabled, the excerpt filter is now always registered, so manual excerpts for posts that do not contain a Djot block will still be run through convertExcerpt() and wp_strip_all_tags(), which is a behavior change for content the plugin doesn’t own.

        // With the content filters off this runs only to rescue a block
        // excerpt; anything else is not this plugin's to rewrite.
        if (
            !$this->options['enable_posts'] && !$this->options['enable_pages']
            && !has_block('wpdjot/djot', $post) && !has_block('wp-djot/djot', $post)

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Suppressed comments (1)

src/Plugin.php:256

• The new early-return guard only runs when the excerpt is empty. When content filters are disabled and a non-Djot post has a manual excerpt, this filter will still run the excerpt through Converter::convertExcerpt(), which contradicts the intent of “anything else is not this plugin's to rewrite” and can unexpectedly rewrite excerpts site‑wide even with both content options off. Consider moving the ownership guard (based on options + has_block) before the manual-excerpt conversion so non-Djot posts are left untouched when filters are off.

        // With the content filters off this runs only to rescue a block
        // excerpt; anything else is not this plugin's to rewrite.
        if (
            !$this->options['enable_posts'] && !$this->options['enable_pages']
            && !has_block('wpdjot/djot', $post) && !has_block('wp-djot/djot', $post)