display field form errors in ObjectBrowserWidget.

[alexandreIFB]

Describe the bug
When using the ObjectBrowserWidget, external errors (e.g. required field) do not display visual alerts in the form field. This prevents the manager from clearly identifying the issues.

Example in Teaser:
Current behavior:

Expected behavior:

Simulate

To simulate the issue and verify the fix:

Save Teaser without providing Target ( href).

Additional context
The problem is related to the validateManualLink and onSubmitManualLink methods, where internal errors are not being merged with external errors.

volto/packages/volto/src/components/manage/Widgets/ObjectBrowserWidget.jsx

Lines 238 to 242 in 91b2258

	if (error && url !== '') {
	this.setState({ errors: [error] });
	} else {
	this.setState({ errors: [] });
	}

volto/packages/volto/src/components/manage/Widgets/ObjectBrowserWidget.jsx

Line 358 in 91b2258

error={this.state.errors}

A possible solution is to combine this.state.errors with this.props.error in the render method before passing them to the FormFieldWrapper.

[sahith-ch]

hi @alexandreIFB can i work on this issue

the solution is i have changed line 358
error={[...(this.state.errors || []), ...(this.props.error || [])]}

[wesleybl]

@alexandreIFB does any core block have this problem? If not, could you suggest a change in the code to simulate the problem?

[alexandreIFB]

@alexandreIFB does any core block have this problem? If not, could you suggest a change in the code to simulate the problem?

To simulate the issue and verify the fix:

Edit the schema of the Teaser block (e.g., line 109) and mark the title field as required.

Attempt to save the block without providing a title.

[wesleybl]

To simulate the issue and verify the fix:

Edit the schema of the Teaser block (e.g., line 109) and mark the title field as required.

Attempt to save the block without providing a title.

@alexandreIFB could you please put this in the issue description?

[wesleybl]

Edit the schema of the Teaser block (e.g., line 109) and mark the title field as required.

@alexandreIFB the title is a regular text field:

volto/packages/volto/src/components/manage/Blocks/Teaser/schema.js

Lines 87 to 89 in 3c93951

	title: {
	title: intl.formatMessage(messages.title),
	},

I think the href field, which is already required, serves as an example, correct?

[alexandreIFB]

Edit the schema of the Teaser block (e.g., line 109) and mark the title field as required.

@alexandreIFB the title is a regular text field:

volto/packages/volto/src/components/manage/Blocks/Teaser/schema.js

Lines 87 to 89 in 3c93951

title: {
title: intl.formatMessage(messages.title),
},
I think the href field, which is already required, serves as an example, correct?

You are right, I ended up confusing it with another problem, updating the problem description with more information