Make changes for render method in Carbon_Fields\Container\Block_Container.php

[htmldiz]

Version

• Carbon Fields: ^3.6
• WordPress:6.6.1
• PHP:7.4

Expected Behavior

Expected $block variable in the function render_block

Actual Behavior

I tried to get post id in Gutenberg block Query loop. And found that usesContext are available only inside block.
Than I font that Carbon fields don't have $block variable inside render_callback method.

Before Changes

	public function render_block( $attributes, $content ) {
		$fields = $attributes['data'];
		$post_id = $this->get_post_id();
		// Unset the "data" property because we
		// pass it as separate argument to the callback.
		unset($attributes['data']);
		ob_start();
		call_user_func( $this->render_callback , $fields, $attributes, $content, $post_id );
		return ob_get_clean();
	}

After cahnges:

	public function render_block( $attributes, $content, \WP_Block $block = null ) { // added variable at the end 
		$fields = $attributes['data'];
		$post_id = $this->get_post_id();
		// Unset the "data" property because we
		// pass it as separate argument to the callback.
		unset($attributes['data']);
		ob_start();
		call_user_func( $this->render_callback , $fields, $attributes, $content, $post_id, $block ); // added variable at the end 
		$toReturn = ob_get_contents();
		ob_clean(); // changed way to get content from object otherwise $block is null
		return $toReturn;
	}
// are easy to reproduce

Steps to Reproduce the Problem

I tried to get post id in Gutenberg block Query loop. And found that usesContext are available only inside block.
Than I font that Carbon fields don't have $block variable inside render_callback method.

Comments

Please add any other comments here

[htmldiz]

You can reed more about it here https://developer.wordpress.org/news/2023/03/03/creating-a-custom-block-that-stores-post-meta/
Start in section: Adding usesContext to block.json