When the field cardinality is 1 and a user tries to add another field collection item then the message 'Too many items.' will be displayed and nothing else. A completely empty page otherwise, leaving the user confused.

Proposed solution: show the access denied page instead. Another field collection item cannot be added, so the appropriate response is to deny access. That is still not ideal for the end user, but at least the admin will see an access denied entry in the logs.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

klausi created an issue. See original summary.

klausi’s picture

Status: Active » Needs review
FileSize
766 bytes

Patch.

Status: Needs review » Needs work

The last submitted patch, 2: fc-too-many-items-2841004-2.patch, failed testing.

klausi’s picture

Status: Needs work » Needs review
FileSize
1.33 KB

Trying to fix the test.

janadam’s picture

Status: Needs review » Reviewed & tested by the community

Works fine on many our projects.

  • ram4nd committed 189329d on 7.x-1.x authored by klausi
    Issue #2841004 by klausi: Replace confusing empty 'Too many items.' page...
ram4nd’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.