Page MenuHomeDevCentral
Differential D333

APIClient and APIFactory contracts
ClosedPublic
Actions

Authored by dereckson on Mar 9 2016, 05:56.
Tags
None
Referenced Files
F3211373: D333.id1475.diff
Sat, Jun 29, 05:56
F3199691: D333.id.diff
Fri, Jun 28, 16:09
F3191046: D333.id1477.diff
Fri, Jun 28, 10:07
Unknown Object (File)
Fri, Jun 28, 01:25
Unknown Object (File)
Wed, Jun 26, 14:16
Unknown Object (File)
Tue, Jun 25, 21:22
Unknown Object (File)
Tue, Jun 25, 21:22
Unknown Object (File)
Tue, Jun 25, 21:21
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