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
F15389298: D12122.id.diff
Sat, Mar 15, 5:05 AM
F15381389: D12122.diff
Fri, Mar 14, 7:23 AM
F15380225: D12122.id29148.diff
Fri, Mar 14, 2:07 AM
Unknown Object (File)
Tue, Feb 25, 7:58 AM
Unknown Object (File)
Feb 8 2025, 3:08 AM
Unknown Object (File)
Feb 8 2025, 3:08 AM
Unknown Object (File)
Feb 8 2025, 3:08 AM
Unknown Object (File)
Feb 8 2025, 3:08 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