Page MenuHomeDevCentral
Differential D333

APIClient and APIFactory contracts
ClosedPublic
Actions

Authored by dereckson on Mar 9 2016, 05:56.
Tags
None
Referenced Files
F27194852: D333.id1477.diff
Thu, Apr 30, 17:09
F27191007: D333.diff
Thu, Apr 30, 12:57
Unknown Object (File)
Wed, Apr 29, 17:15
Unknown Object (File)
Wed, Apr 29, 16:02
Unknown Object (File)
Wed, Apr 29, 15:57
Unknown Object (File)
Wed, Apr 29, 14:52
Unknown Object (File)
Sat, Apr 25, 02:47
Unknown Object (File)
Thu, Apr 23, 09:09
View All Files
Subscribers
alken-orin

Details

Reviewers
dereckson
Commits
rNOTIFab12e8289918: APIClient and APIFactory contracts
Summary

To improve dependency injection in ProjectsMap, we create two
interface contracts, APIClient and APIFactory, which abstract
an API client and a class to instantiate them.

That allow to easily inject an API client and increase components
independance.

Test Plan

A variation of fea5b9c has successfully been tested.

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 793.Mar 9 2016, 05:56
dereckson retitled this revision from to APIClient and APIFactory contracts.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
alken-orin added a subscriber: alken-orin.Mar 9 2016, 05:57

Build started: https://ci.nasqueron.org/job/test-notifications-phab/118/ (console: https://ci.nasqueron.org/job/test-notifications-phab/118/console)

alken-orin added a comment.Mar 9 2016, 06:09

Build is green https://ci.nasqueron.org/job/test-notifications-phab/118/ for more details.

dereckson updated this revision to Diff 1475.Aug 10 2016, 19:17
dereckson edited edge metadata.

Rebased against master.

dereckson mentioned this in D334: API dependency injection for ProjectsMap.Aug 10 2016, 19:24
dereckson accepted this revision.Aug 10 2016, 19:47
dereckson edited edge metadata.
This revision is now accepted and ready to land.Aug 10 2016, 19:47
Closed by commit rNOTIFab12e8289918: APIClient and APIFactory contracts (authored by dereckson, committed by dereckson). · Explain WhyAug 10 2016, 19:49
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
app/
Contracts/
24 lines
15 lines
Phabricator/
34 lines
4 lines
29 lines

Diff 1477

View Options

app/Contracts/APIClient.php

Loading...
View Options

app/Contracts/APIFactory.php

Loading...
View Options

app/Phabricator/PhabricatorAPI.php

Loading...
View Options

app/Phabricator/PhabricatorAPIFactory.php

Loading...
View Options

app/Phabricator/ProjectsMap.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator