Page MenuHomePhorge

Tenant scope for CLI commands
ClosedPublic

Authored by machniak on Apr 12 2021, 3:06 PM.
Tags
None
Referenced Files
F11588172: D2452.id7222.diff
Thu, Mar 28, 7:26 PM
Unknown Object (File)
Sat, Mar 23, 3:21 PM
Unknown Object (File)
Mon, Mar 11, 5:05 PM
Unknown Object (File)
Tue, Mar 5, 6:49 PM
Unknown Object (File)
Tue, Mar 5, 6:49 PM
Unknown Object (File)
Tue, Mar 5, 6:49 PM
Unknown Object (File)
Tue, Mar 5, 6:49 PM
Unknown Object (File)
Tue, Mar 5, 6:47 PM
Subscribers
Restricted Project

Details

Reviewers
mollekopf
Group Reviewers
Restricted Project
Commits
rK6aa312a4a5db: Tenant scope for CLI commands
Summary

T421559

Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Branch
dev/cli-tenant
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 33868
Build 13426: arc lint + arc unit

Event Timeline

machniak created this revision.
  • Couple of fixes, tenant-condition for wallets
machniak retitled this revision from [WIP] Tenant scope for CLI commands to Tenant scope for CLI commands.Jun 1 2021, 3:26 PM
mollekopf subscribed.

I'm wondering whether there isn't a more elegant solution to replacing all queries with a custom function that everything has to go through, so that e.g. a \App\Wallet::find or \App\Wallet::all would automatically take the tentant into account.

That said, this seems otherwise reasonable, so go ahead with it.
Please not I'm not fully on top of the dev/reseller branch, so I just looked it over for obvious mistakes.

This revision is now accepted and ready to land.Jun 4 2021, 12:19 PM

Merged into dev/reseller.