Reduce the number of calls to Refinery::createInstance()

Open on Drupal.org →

Created on 7 November 2023, about 1 year ago

Updated 11 November 2023, about 1 year ago

Problem/Motivation

In Refinery::getRefiners(), it seems an "if" statement could be added to be able to reuse Refinery instances. This keeps the number of refinery instances under control.

Steps to reproduce

-

Proposed resolution

Store the Refinery instances in memory and reuse them when applicable.

Remaining tasks

-

User interface changes

-

API changes

-

Data model changes

-

🐛 Bug report

Status

Fixed

Version

4.0

Component

Code

Created by

🇨🇦Canada jigarius Montréal

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Comments & Activities

Issue created by @jigarius
@jigarius opened merge request.
Issue was unassigned.
Status changed to Needs review about 1 year ago6:47pm 7 November 2023
Comment about 1 year ago →
🇨🇦Canada jigarius Montréal
I've created an MR for it. Kindly let me know if it works.

Comment about 1 year ago →

jigarius → authored 65b6a1ee on 4.x

Issue #3399934: Reduce the number of calls to Refinery::createInstance()

Status changed to Fixed about 1 year ago7:44am 11 November 2023
Comment about 1 year ago →
🇨🇦Canada ksere Montreal
Comment about 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024