Hide block/view if No results behavior

@coaston how do you come to this conclusion?

Comparing the commit from the linked issue
https://git.drupalcode.org/project/entity_extra_field/-/commit/94a2ab3dc...
to the latest branch:
https://git.drupalcode.org/project/entity_extra_field/-/blob/2.1.x/src/P...

it's present!

it's present!

Coaston means to say that the original commit is incomplete, not that the commit is missing.

The MR created by jnettik implemented the fix in a different way from the patch file submitted by artyom. Namely:

• The MR implementation simply calls Element::isEmpty()
• The patch file implementation additionally executes the view to check if the rendered output is empty

This is significant in the special case of views because as explained 🐛 Empty content view Fixed :

ViewExecutable will build a render array regardless of whether it has anything to display, because the view is not executed until render time, and hence just checking the render array it not reliable, which is all Element::isEmpty() does.

The Drupal Core views block display plugin (\Drupal\views\Plugin\views\display\Block::execute()) suggests that the correct way to check is to actually execute the view with arguments, then check $view->getDisplay()->outputIsEmpty(), which is how the plugin implements its "Hide block if the view output is empty" option.