Page MenuHomePhabricator

FlowHooks::getOccupationController() should not be called on every page view
Open, LowPublic

Description

It should be possible to lazy load this?

Event Timeline

Legoktm created this task.Jun 11 2015, 7:17 PM
Legoktm updated the task description. (Show Details)
Legoktm raised the priority of this task from to Needs Triage.
Legoktm added a project: StructuredDiscussions.
Legoktm added a subscriber: Legoktm.
Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptJun 11 2015, 7:17 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

It's called in both the extension function and:

// Initialized in FlowHooks to facilitate only loading the flow container
// when flow is specifically requested to run. Extension initialization
// must always happen before calling flow code.
$c['occupation_controller'] = FlowHooks::getOccupationController();
Catrope triaged this task as Normal priority.
Catrope set Security to None.
Catrope lowered the priority of this task from Normal to Low.