> Drupal\search_api\Plugin\search_api\processor\AggregatedField::alterPropertyDefinitions

There's that docs comment, and also test methods called testAlterPropertyDefinitions() that possibly need to be renamed.

CommentFileSizeAuthor
#3 search_api-test_name_renamed-2748323-3-D8.patch933 bytessinn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim created an issue. See original summary.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey as a volunteer commented
Issue tags: +release target
sinn’s picture

sinn CreditAttribution: sinn at Adyax commented
Status: Active » Needs review
FileSize
search_api-test_name_renamed-2748323-3-D8.patch933 bytes

Looks like this test covers \Drupal\search_api\Plugin\search_api\processor\AggregatedField::getPropertyDefinitions().

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented
Status: Needs review » Reviewed & tested by the community

  • drunken monkey committed f1dc1b7 on 8.x-1.x authored by sinn 
    Issue #2748323 by sinn: Fixed comment reference to removed method...
drunken monkey’s picture

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

Yes, looks good. Thanks!
Committed.

Status: Fixed » Closed (fixed)

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