Caching the result of the SQL queries performed will increase the performance of the code, which would not need anymore to execute a query that is likely to return the same result, until the user doesn't modify the filters / rules list.

Comments

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 6.x-2.x-dev » 6.x-1.x-dev

The result of the SQL queries has been cached in a PHP static variable.
The code must be ported to the 6.x-1 branch.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Active » Fixed
Issue tags: +beta3

The code has been changed.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Issue tags: -beta3
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Issue tags: +6.x-1.0-beta3
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Fixed » Active
Issue tags: -6.x-1.0-beta3

The code I implemented must be changed; I will use a real cache, and not a static PHP variable (the database table is already created by the installation code; it just needs to be used).

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 6.x-2.x-dev » 6.x-1.x-dev
Status: Active » Patch (to be ported)
Issue tags: +6.x-2.0-beta1

The code has been changed.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Patch (to be ported) » Fixed
Issue tags: +6.x-1.0-beta3

Status: Fixed » Closed (fixed)
Issue tags: -6.x-2.0-beta1, -6.x-1.0-beta3

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

Issue tags: +6.x-2.0-beta1, +6.x-1.0-beta3

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.