What it says in the title: even though we set 'is_list' => TRUE for reverse entity-reference properties (ReverseEntityReferences processor), isList() will be FALSE since the setting is ignored.

CommentFileSizeAuthor
#2 3053603-2--entity_processor_property_lists--tests_only.patch645 bytesdrunken monkey
#2 3053603-2--entity_processor_property_lists.patch1.09 KBdrunken monkey

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Active » Needs review
StatusFileSize
new 3053603-2--entity_processor_property_lists.patch1.09 KB
new 3053603-2--entity_processor_property_lists--tests_only.patch645 bytes

Trivial fix.

Status: Needs review » Needs work

The last submitted patch, 2: 3053603-2--entity_processor_property_lists--tests_only.patch, failed testing. View results

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
Status: Needs work » Reviewed & tested by the community

Trivial indeed.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Reviewed & tested by the community » Fixed

Good to hear, thanks for reviewing!
Committed.

  • drunken monkey committed 6cc35ed on 8.x-1.x 
    Issue #3053603 by drunken monkey, borisson_: Fixed reported cardinality...

Status: Fixed » Closed (fixed)

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

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
commented

I am concerned this bugfix might cause problems when updating existing sites. Please see my comment #2906905-15: Custom fields automatically treated as multi-value.