I have an entity reference field which is required. There is no way to select NONE for the second or greater value. I have a patch.

CommentFileSizeAuthor
#10 2866543-10.patch2.93 KBmrinalini9
#2 2866543.patch2.9 KBdarrick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

darrick created an issue. See original summary.

darrick’s picture

darrick CreditAttribution: darrick commented
FileSize
2866543.patch2.9 KB

The patch. Not sure if the best way but seems to work okay. I tested with both required TRUE and FALSE.

colepacak’s picture

colepacak CreditAttribution: colepacak commented

Thanks for the patch, @darrick.

LNakamura’s picture

LNakamura CreditAttribution: LNakamura at Mediacurrent commented

This patch works great for me!

Tichris59’s picture

Tichris59 CreditAttribution: Tichris59 commented

Patch is OK , thanks !

jeffam’s picture

jeffam
Primary language English
CreditAttribution: jeffam commented
Status: Active » Reviewed & tested by the community

Works in my testing, too.

khaled.zaidan’s picture

khaled.zaidan CreditAttribution: khaled.zaidan at Reading Room commented

Tested here, too... looking good.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Patch no longer applies.

mrinalini9’s picture

mrinalini9 CreditAttribution: mrinalini9 at Srijan | A Material+ Company for Drupal India Association commented
Assigned: Unassigned » mrinalini9
mrinalini9’s picture

mrinalini9 CreditAttribution: mrinalini9 at Srijan | A Material+ Company for Drupal India Association commented
Assigned: mrinalini9 » Unassigned
Status: Needs work » Needs review
FileSize
2866543-10.patch2.93 KB

Rerolled patch #2, please review.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Issue tags: -Needs reroll

  • JeroenT committed 5584a9e on 8.x-1.x authored by darrick 
    Issue #2866543 by darrick, mrinalini9, JeroenT: Support required fields
  • JeroenT committed df7b70b on 8.x-1.x
    Issue #2866543 by darrick, mrinalini9, JeroenT: Support required fields...
JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Status: Needs review » Fixed

Committed and pushed to 8.x-1.x. Thanks!

Status: Fixed » Closed (fixed)

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