Page MenuHomePhabricator

Clean up Diffusion branch query a bit
ClosedPublic

Authored by epriestley on Oct 30 2013, 3:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 12, 12:41 AM
Unknown Object (File)
Mon, Mar 11, 8:02 PM
Unknown Object (File)
Mon, Mar 11, 8:02 PM
Unknown Object (File)
Mon, Mar 11, 8:02 PM
Unknown Object (File)
Feb 9 2024, 6:45 PM
Unknown Object (File)
Feb 7 2024, 9:04 PM
Unknown Object (File)
Feb 6 2024, 7:20 PM
Unknown Object (File)
Feb 6 2024, 6:07 PM
Subscribers

Details

Summary

Ref T2716.

  • Serve from DiffusionCommitQuery, not PhabricatorAuditCommitQuery (which should probably die).
  • Fix logic for limit, which incorrectly failed to display the "Showing %d branches." text.
  • Clean up things a touch.
  • I didn't end up actually needing needCommitData(), but left it in there since I think it will be needed soon.
  • Removed a "TODO" because I don't remember what "etc etc" means.
Test Plan

Looked at branches in several repositories.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan added inline comments.
src/applications/diffusion/view/DiffusionBranchTableView.php
64

Added here - https://secure.phabricator.com/rP485b5e5ded5e545f09549f234053e46a5943fb9a - doesn't really shed more light on the mystery though.

haha

iiam

I legitimately have no clue what it might have meant