Homec4science

Allow CelerityResourceResponse to hold resources from multiple maps

Authored by epriestley <git@epriestley.com> on Jan 2 2014, 20:59.

Description

Allow CelerityResourceResponse to hold resources from multiple maps

Summary:
Ref T4222. Currently, CelerityResourceResponse holds response resources in flat maps. Instead, specify which map resources appear in.

Also, provide requireResource() and initBehavior() APIs on the Controller and View base classes. These provide a cleaner abstraction over require_celerity_resource() and Javelin::initBehavior(), but are otherwise the same. Move a few callsites over.

Test Plan:

  • Reloaded pages.
  • Browsed around Differential.

Reviewers: btrahan, hach-que

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T4222

Differential Revision: https://secure.phabricator.com/D7876

Details

Committed
epriestley <git@epriestley.com>Jan 2 2014, 20:59
Pushed
aubortJan 31 2017, 17:16
Parents
rPH09341be10fc4: Remove repository shortcuts
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH31b6f69ff7c2: Allow CelerityResourceResponse to hold resources from multiple maps (authored by epriestley <git@epriestley.com>).Jan 2 2014, 20:59