Remove completely pointless load of every repository when viewing a repository URI
ClosedPublic
Actions

Authored by epriestley on Feb 8 2018, 8:43 PM.

Details

Reviewers: None
Commits: rPd1e273daf62f: Remove completely pointless load of every repository when viewing a repository…

Summary

See D18176. This query has no effect (other than wasting resources) and the result is unused.

$repository already has the URI loaded because we load them unconditionally during request initialization.

Test Plan

Viewed repository URIs.

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

epriestley requested review of this revision.Feb 8 2018, 8:44 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 8 2018, 8:47 PM

This revision was automatically updated to reflect the committed changes.

(T6928 could have caught this statically.)

Path

Size

src/

applications/

diffusion/

controller/

6 lines