Page MenuHomePhabricator
Differential D8560

Modernize OAuthServer PHIDs and Queries
ClosedPublic
Actions

Authored by epriestley on Mar 18 2014, 1:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Feb 15, 1:33 AM
Unknown Object (File)
Sat, Feb 15, 1:33 AM
Unknown Object (File)
Fri, Feb 14, 4:45 PM
Unknown Object (File)
Sun, Feb 2, 3:12 PM
Unknown Object (File)
Sun, Feb 2, 3:06 PM
Unknown Object (File)
Sun, Feb 2, 1:27 PM
Unknown Object (File)
Thu, Jan 30, 5:19 PM
Unknown Object (File)
Tue, Jan 28, 8:51 PM
View All Files
Subscribers
chad
epriestley

Details

Reviewers
btrahan
Commits
Restricted Diffusion Commit
rP8909f8ec599b: Modernize OAuthServer PHIDs and Queries
Summary
  • Modernize PHID types.
  • Implement PhabricatorPolicyInterface.
  • Make queries policy aware.
Test Plan

Browsed client and authorization lists.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20313.Mar 18 2014, 1:32 AM
epriestley retitled this revision from to Modernize OAuthServer PHIDs and Queries.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
Herald added a subscriber: epriestley.Mar 18 2014, 1:32 AM
btrahan accepted this revision.Mar 18 2014, 1:48 AM
btrahan edited edge metadata.

sweet. existing policy stuff worked really well here IMO.

This revision is now accepted and ready to land.Mar 18 2014, 1:48 AM
epriestley added a subscriber: chad.Mar 18 2014, 2:43 AM

Yeah -- the OAuthServer application is in pretty good shape in general, it's just an iteration behind on UI kit and infrastructure since we wrote it before policies, applications, appsearch, @chad made everything pretty, etc.

epriestley closed this revision.Mar 18 2014, 8:28 PM
epriestley updated this revision to Diff 20325.

Closed by commit rP8909f8ec599b (authored by @epriestley).

Revision Contents
Changeset List

Diff 20325

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/oauthserver/controller/client/PhabricatorOAuthClientListController.php

Loading...
View Options

src/applications/oauthserver/controller/clientauthorization/PhabricatorOAuthClientAuthorizationListController.php

Loading...
View Options

src/applications/oauthserver/phid/PhabricatorOAuthServerPHIDTypeClient.php

Loading...
View Options

src/applications/oauthserver/phid/PhabricatorOAuthServerPHIDTypeClientAuthorization.php

Loading...
View Options

src/applications/oauthserver/query/PhabricatorOAuthClientAuthorizationQuery.php

Loading...
View Options

src/applications/oauthserver/query/PhabricatorOAuthServerClientQuery.php

Loading...
View Options

src/applications/oauthserver/storage/PhabricatorOAuthClientAuthorization.php

Loading...
View Options

src/applications/oauthserver/storage/PhabricatorOAuthServerClient.php

Loading...
View Options

src/applications/phid/PhabricatorPHIDConstants.php

Loading...
Phabricator · Built by Phacility