Problem/Motivation

We use still system_rebuild_module_data to get module names in some places. This can take 200ms+.

Proposed resolution

Use ModuleHandler::getName() instead of system_rebuild_module_data to get module names

Remaining tasks

Review patch

User interface changes

N/A

API changes

N/A

Beta evaluation

Marked bug because significant performance slowdown (system_rebuild_module_data can take 200ms+)

CommentFileSizeAuthor
#1 2480081-1.patch5.67 KBstefan.r

Comments

stefan.r’s picture

stefan.r commented
StatusFileSize
new 2480081-1.patch5.67 KB
stefan.r’s picture

stefan.r commented
Status: Active » Needs review

catch queued 1: 2480081-1.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1: 2480081-1.patch, failed testing.

stefan.r’s picture

stefan.r commented

@catch seems you had the same idea 3 weeks later: #2488610: Use ModuleHander::getName() instead of rebuilding module data on permissions page

:)

stefan.r’s picture

stefan.r commented
Status: Needs work » Closed (duplicate)