Reset cached sorting on sorting configuration change

Components

Labels

Description

Plan:

  1. create getSortingPrefix($system_setting) method in ConfigurationEventHandler

  2. new method would inspect ConfigMapping setting of each unit configs, which prefix is mentioned in Modules database table (replace "adm" with "c" and skip "In-Portal" module)

  3. if system setting name, that is given to the method is matching DefaultSorting1Field or DefaultSorting2Field key values of ConfigMapping array, then return unit config prefix

  4. if no match found then return false

  5. call new method from ConfigurationEventHandler::OnAfterItemUpdate only in case if VariableValue field value was changed

  6. if new method returns unit config prefix, then remove any stored sorting, that might be associated with it from PersistentSessionData table

Context Information

None

Additional information (do not use)

None

Activity

Alex March 21, 2015 at 6:25 PM

alex closed D10: https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change by committing rINP16155: Fixes https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change.
alex updated the diff for D10: https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change.

http://qa.in-portal.org/D10

[API] Administrator March 21, 2015 at 6:25 PM

User committed a fix to 5.3.x. Commit Message:

Fixes https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change

Differential Revision: http://qa.in-portal.org/D10

Alex March 19, 2015 at 11:28 AM

alex accepted D10: https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change.
D10: https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change is now accepted and ready to land.

http://qa.in-portal.org/D10

Erik S. March 19, 2015 at 10:55 AM

erik updated the diff for D10: https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change.
erik updated the test plan for D10: https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change.
erik added a comment to D10: https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change.

CS fixes

http://qa.in-portal.org/D10

Erik S. March 19, 2015 at 10:38 AM

erik updated the test plan for D10: https://in-portal.atlassian.net/browse/INP-1315#icft=INP-1315 - Reset cached sorting on sorting configuration change.

http://qa.in-portal.org/D10

Fixed

Details

Priority

Assignee

Reporter

Developer

Reviewer

Change Log Group

Fixed

Change Log Message

Patch Instructions

Patches must be submitted through Phabricator.

Commit Revision

16,155

Story Points

Sprint

Fix versions

Created August 29, 2013 at 1:58 PM
Updated December 29, 2024 at 8:56 PM
Resolved March 21, 2015 at 6:25 PM

Flag notifications