Page MenuHomePhabricator
Differential D12122

Allow repositories to be ordered by commit count
ClosedPublic
Actions

Authored by epriestley on Mar 21 2015, 11:28 AM.
Tags
None
Referenced Files
F18870117: D12122.id.diff
Nov 4 2025, 3:30 PM
F18866611: D12122.diff
Nov 3 2025, 6:13 PM
F18854529: D12122.id.diff
Nov 1 2025, 12:21 AM
F18821051: D12122.id29148.diff
Oct 22 2025, 6:44 PM
F18783609: D12122.id29172.diff
Oct 13 2025, 8:25 AM
F18783335: D12122.id29172.diff
Oct 13 2025, 6:01 AM
F18743428: D12122.diff
Oct 2 2025, 11:37 PM
F18712495: D12122.diff
Sep 29 2025, 5:25 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T7640: Add "number of commits" as order by option in search
Commits
Restricted Diffusion Commit
rP6eadfe6a6fce: Allow repositories to be ordered by commit count
Summary

Fixes T7640.

Test Plan

Screen_Shot_2015-03-21_at_3.59.31_AM.png (756×339 px, 65 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 29148.Mar 21 2015, 11:28 AM
epriestley retitled this revision from to Allow repositories to be ordered by commit count.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T7640: Add "number of commits" as order by option in search.
Herald added a subscriber: epriestley. · View Herald TranscriptMar 21 2015, 11:28 AM
epriestley added inline comments.Mar 21 2015, 11:28 AM
src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
287–289

Per recent fixes to Maniphest, this lays some groundwork for doing ORDER BY clauses in a more consistent/general way.

btrahan accepted this revision.Mar 23 2015, 3:57 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 23 2015, 3:57 PM
Closed by commit rP6eadfe6a6fce: Allow repositories to be ordered by commit count (authored by epriestley, committed by epriestley). · Explain WhyMar 23 2015, 4:10 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
repository/
query/
63 lines
2 lines
infrastructure/
query/
policy/
38 lines

Diff 29172

View Options

src/applications/repository/query/PhabricatorRepositoryQuery.php

Loading...
View Options

src/applications/repository/query/PhabricatorRepositorySearchEngine.php

Loading...
View Options

src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php

Loading...
Phabricator · Built by Phacility