Improve performance when all sequencial terms selected and all items text provided

Created on 2 June 2023, over 1 year ago

Postpone until tests in place.

Basically we don't pass the all items text into getSequencedElements, so we cannot bail on doing the sequencing if that text is present.

    $all_items_text = $this->getSetting('all_items_text');
    $sequences = $this->getSequencedElements($elements, $items, $treat_as_one, $treat_as_one_label, $output_as_link, $langcode);

But we also use the all_items info as a check to see if we *do* the sequencing, so short-circuiting would probably mess up logic in the view method.

πŸ“Œ Task
Status

Active

Version

3.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States mlncn Minneapolis, MN, USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024