Page MenuHomePhorge
Differential D3038

Code de-duplication, API unification
ClosedPublic
Actions

Authored by machniak on Nov 18 2021, 1:10 PM.
Tags
None
Referenced Files
F10604263: D3038.id.diff
Wed, Dec 6, 11:57 AM
Unknown Object (File)
Tue, Nov 14, 6:35 AM
Unknown Object (File)
Nov 4 2023, 12:08 PM
Unknown Object (File)
Nov 4 2023, 4:24 AM
Unknown Object (File)
Oct 25 2023, 10:47 PM
Unknown Object (File)
Oct 13 2023, 11:15 PM
Unknown Object (File)
Sep 30 2023, 1:26 AM
Unknown Object (File)
Sep 29 2023, 6:41 AM
View All 12 Files
Subscribers
mollekopf
Restricted Project

Details

Reviewers
mollekopf
Group Reviewers
Restricted Project
Commits
rK4211ad138d60: Code de-duplication, API unification
Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

machniak requested review of this revision.Nov 18 2021, 1:10 PM
machniak created this revision.
Harbormaster completed remote builds in B37112: Diff 8651.Nov 18 2021, 1:10 PM
machniak updated this revision to Diff 8669.Nov 18 2021, 4:16 PM
  • De-duplicate code for handling status update process
Harbormaster completed remote builds in B37130: Diff 8669.Nov 18 2021, 4:16 PM
machniak updated this revision to Diff 8675.Nov 19 2021, 8:41 AM
  • Even more deduplication
Harbormaster completed remote builds in B37136: Diff 8675.Nov 19 2021, 8:41 AM
mollekopf accepted this revision.Nov 19 2021, 2:39 PM
mollekopf subscribed.
mollekopf added inline comments.
src/app/Http/Controllers/Controller.php
210

Personally I would prefer passing $objectProps as argument instead of $full, unless $objectProps becomes useful for other functions as well.

It would make it a bit clearer IMO what's going on: Either you serialize everything, or just the requested $objectProps.

That's not a blocker though.

This revision is now accepted and ready to land.Nov 19 2021, 2:39 PM
machniak added inline comments.Nov 19 2021, 2:53 PM
src/app/Http/Controllers/Controller.php
210

Yes, but this way we define the common props once (per object type), so we don't have to pass them all the time as an argument. This makes also the API responses unified "automatically", i.e. reseller, admin, user end-points, all of them return the same set of object properties.

Closed by commit rK4211ad138d60: Code de-duplication, API unification (authored by machniak). · Explain WhyNov 19 2021, 2:55 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
app/
Http/
Controllers/
API/
V4/
Admin/
10 lines
15 lines
4 lines
117 lines
116 lines
Reseller/
10 lines
15 lines
10 lines
113 lines
142 lines
tests/
Unit/
Controller/

Diff 8681

View Options

src/app/Http/Controllers/API/V4/Admin/DomainsController.php

Loading...
View Options

src/app/Http/Controllers/API/V4/Admin/GroupsController.php

Loading...
View Options

src/app/Http/Controllers/API/V4/Admin/UsersController.php

Loading...
View Options

src/app/Http/Controllers/API/V4/DomainsController.php

Loading...
View Options

src/app/Http/Controllers/API/V4/GroupsController.php

Loading...
View Options

src/app/Http/Controllers/API/V4/Reseller/DomainsController.php

Loading...
View Options

src/app/Http/Controllers/API/V4/Reseller/GroupsController.php

Loading...
View Options

src/app/Http/Controllers/API/V4/Reseller/UsersController.php

Loading...
View Options

src/app/Http/Controllers/API/V4/UsersController.php

Loading...
View Options

src/app/Http/Controllers/Controller.php

Loading...
View Options

src/tests/Unit/Controller/DomainsTest.php

Loading...
View Options

src/tests/Unit/Controller/UsersTest.php

Loading...