Check of $value not covering LinkItem cases in link contraint validators

Here a basic patch to illustrate the idea of how to change this. The added code should probably be moved into a base class if this approach makes sense at all.

Thank you for reporting.

Wonder if we could get a test-only patch showing the issue.

Akhil Babu → made their first commit to this issue’s fork.

Akhil Babu → changed the visibility of the branch 3387013-check-of-value-11 to hidden.

Akhil Babu → changed the visibility of the branch 3387013-check-of-value to hidden.

I was able to recreate this issue, but only with the help of custom code.

• Go to admin/config/development/logging, select 'All' and save.
• Add a link field with Unlimited cardinality to article content type.
• Create a new article, add one link and save
• Add a form after to article edit form
• Load the node and call validate() method.[$entity = $form_state->getFormObject()->getEntity(); $entity->validate();]
• Now try to edit the previously created node.

The errors appear only when validate() is called on node object when we are in the node edit form. For example

/**
 * Implements hook_preprocess_HOOK() for page.html.twig.
 */
function MY_MODULE_preprocess_page($variables) {
  \Drupal::entityTypeManager()->getStorage('node')->load(1)->validate();
}

will not trigger any errors when the node is viewd. But will trigger error when edit form is loaded.

When the edit form is loaded after adding a link item to an unlimited cardinality link field, Drupal automatically appends a new link widget below the existing one to add a second link item. Somehow, this second item is also validated when the node is validated from the edit form, and this triggers the NULL warning. This won't happen when the node is loaded from any place other than the node edit form. So, maybe the logic in the validate() method needs to be fixed.

I have created a test to show the bug.

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

dcam → changed the visibility of the branch 3387013- to hidden.

dcam → changed the visibility of the branch 11.x to hidden.

I converted the patch in #2 to an MR and added Unit tests.