Fatal TypeError in class-dynamic-tags.php when processing arrays in render_tag()

[kbrookes]

Description:

Bug Description

Fatal PHP error occurs when render_tag() receives an array instead of a string, specifically when processing featured images in Bricks query loops.

Error Message

Fatal error: Uncaught TypeError: strpos(): Argument #1 ($haystack) must be of type string, array given in
/wp-content/plugins/bricks-api-bridge/includes/class-dynamic-tags.php:79

Steps to Reproduce

1. Create a Bricks query loop that displays posts
2. Add an image element using {featured_image} dynamic data
3. View the page on the frontend

Root Cause

In class-dynamic-tags.php line 79, the render_tag() method calls strpos($tag, '{bab_var:') without checking if $tag is a string. When Bricks passes image
metadata (an array) through the bricks/dynamic_data/render_tag filter, this causes a type error.

Proposed Fix

Add a type check at the beginning of the render_tag() method:

public function render_tag( $tag, $post, $context ) {
    // If $tag is not a string (e.g., array from image metadata), return it unchanged
    if ( ! is_string( $tag ) ) {
        return $tag;
    }

    if ( strpos( $tag, '{bab_var:' ) !== 0 ) {
        return $tag;
    }

    // ... rest of method
}

Environment

- Plugin Version: 1.0.0
- PHP Version: 7.4+
- Bricks Builder: Latest
- WordPress: 5.6+

Impact

This is a fatal error that crashes pages using query loops with featured images.

[developer2013]

Thanks for the detailed report and the proposed fix — your analysis is spot on. 🙏

Good news: this is already fixed in the latest release. The exact guard you proposed (plus the same guard on the sibling render_content() method, which would have crashed the same way on the next array pass) shipped in the plugin v1.0.2 zip:

public function render_tag( $tag, $post, $context ) {
    if ( ! is_string( $tag ) || strpos( $tag, '{bab_var:' ) !== 0 ) {
        return $tag;
    }
    // ...
}

Your environment shows Plugin Version: 1.0.0, which predates the fix — that's why you're still hitting it.

Fix: grab the current plugin zip from the latest release and re-install (it carries plugin v1.0.2):
https://github.com/developer2013/bricks-mcp-open/releases/latest/download/bricks-api-bridge.zip

This is effectively a duplicate of #3 (same strpos type error, just triggered via the {featured_image} query-loop path), so I'll close it as resolved. If you update and still see the crash, please reopen with the new plugin version number and we'll dig in. Thanks again for taking the time to file a clean, reproducible report! 🚀