Differential D1702

Better structure for console commands, and unification of code/behavior
Needs ReviewPublic
Actions

Authored by vanmeeuwen on Mon, Oct 5, 11:45 AM.

Details

Reviewers

None

Group Reviewers

Restricted Project

Summary

Merge branch 'master' into arcpatch-D1684

Standardize downloading a file

Demonstrate command restructuring with magic

Test Plan

./phpunit (tests not updated)

Diff Detail

Repository

rK kolab

Branch

arcpatch-D1684

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 31171
Build 11725: arc lint + arc unit

Event Timeline

vanmeeuwen requested review of this revision.Mon, Oct 5, 11:45 AM

vanmeeuwen created this revision.

Harbormaster completed remote builds in B31120: Diff 4522.Mon, Oct 5, 11:45 AM

Further reduce duplication

Harbormaster completed remote builds in B31123: Diff 4528.Mon, Oct 5, 11:53 AM

show-case implementation of 'relations'

Harbormaster completed remote builds in B31129: Diff 4534.Mon, Oct 5, 1:32 PM

Commentary and documentation
user:wallets translation to the new system
More magic

Harbormaster completed remote builds in B31135: Diff 4540.Mon, Oct 5, 6:23 PM

And... another iteration to experiment with.

Harbormaster completed remote builds in B31141: Diff 4546.Mon, Oct 5, 7:05 PM

Some create object logic

Harbormaster completed remote builds in B31171: Diff 4576.Tue, Oct 6, 12:48 PM

Large Diff

This large diff affects 158 files. Files without inline comments have been collapsed. Expand All Files

Revision Contents
Changeset List

		Path
A	M	src/app/Console/Command.php (104 lines)
V		src/app/Console/Commands/{DB/PingCommand.php ← DBPing.php} (16 lines)
V		src/app/Console/Commands/{DBPing.php → DB/PingCommand.php}
V		src/app/Console/Commands/{Data/Import/CountriesCommand.php ← DataCountries.php} (69 lines)
A	M	src/app/Console/Commands/Data/ImportCommand.php (40 lines)
V		src/app/Console/Commands/{DataCountries.php → Data/Import/CountriesCommand.php}
A	M	src/app/Console/Commands/Discount/CreateCommand.php (12 lines)
A	M	src/app/Console/Commands/Discount/ReadCommand.php (12 lines)
D	M	src/app/Console/Commands/DiscountList.php (61 lines)
A	M	src/app/Console/Commands/DiscountsCommand.php (12 lines)
V		src/app/Console/Commands/{Domain/Add.php ← DomainAdd.php} (6 lines)
A	M	src/app/Console/Commands/Domain/CreateCommand.php (12 lines)
V		src/app/Console/Commands/{Domain/Delete.php ← DomainDelete.php} (19 lines)
V		src/app/Console/Commands/{Domain/ListDomains.php ← DomainList.php} (16 lines)
A	M	src/app/Console/Commands/Domain/ReadCommand.php (12 lines)
V		src/app/Console/Commands/{Domain/SetStatus.php ← DomainSetStatus.php} (9 lines)
V		src/app/Console/Commands/{Domain/SetWallet.php ← DomainSetWallet.php} (8 lines)
V		src/app/Console/Commands/{Domain/Status.php ← DomainStatus.php} (25 lines)
V		src/app/Console/Commands/{Domain/Suspend.php ← DomainSuspend.php} (21 lines)
V		src/app/Console/Commands/{Domain/Unsuspend.php ← DomainUnsuspend.php} (21 lines)
A	M	src/app/Console/Commands/Domain/UsersCommand.php (13 lines)
V		src/app/Console/Commands/{DomainAdd.php → Domain/Add.php}
V		src/app/Console/Commands/{DomainDelete.php → Domain/Delete.php}
V		src/app/Console/Commands/{DomainList.php → Domain/ListDomains.php}
D	M	src/app/Console/Commands/DomainListUsers.php (83 lines)
V		src/app/Console/Commands/{DomainSetStatus.php → Domain/SetStatus.php}
V		src/app/Console/Commands/{DomainSetWallet.php → Domain/SetWallet.php}
V		src/app/Console/Commands/{DomainStatus.php → Domain/Status.php}
V		src/app/Console/Commands/{DomainSuspend.php → Domain/Suspend.php}
V		src/app/Console/Commands/{DomainUnsuspend.php → Domain/Unsuspend.php}
A	M	src/app/Console/Commands/DomainsCommand.php (12 lines)
A	M	src/app/Console/Commands/Entitlement/CreateCommand.php (55 lines)
A	M	src/app/Console/Commands/Entitlement/ReadCommand.php (12 lines)
A	M	src/app/Console/Commands/EntitlementsCommand.php (12 lines)
M		src/app/Console/Commands/Job/DomainCreate.php (6 lines)
M		src/app/Console/Commands/Job/DomainUpdate.php (6 lines)
M		src/app/Console/Commands/Job/UserCreate.php (6 lines)
M		src/app/Console/Commands/Job/UserUpdate.php (6 lines)
M		src/app/Console/Commands/Job/WalletCheck.php (3 lines)
M		src/app/Console/Commands/MollieInfo.php (8 lines)
A	M	src/app/Console/Commands/Package/ReadCommand.php (12 lines)
A	M	src/app/Console/Commands/Package/SkusCommand.php (45 lines)
D	M	src/app/Console/Commands/PackageSkus.php (51 lines)
A	M	src/app/Console/Commands/PackagesCommand.php (12 lines)
A	M	src/app/Console/Commands/Plan/ReadCommand.php (12 lines)
M		src/app/Console/Commands/PlanPackages.php (12 lines)
A	M	src/app/Console/Commands/PlansCommand.php (12 lines)
A	M	src/app/Console/Commands/Sku/ReadCommand.php (12 lines)
M		src/app/Console/Commands/StripeInfo.php (7 lines)
A	M	src/app/Console/Commands/Transaction/ReadCommand.php (12 lines)
A	M	src/app/Console/Commands/TransactionsCommand.php (12 lines)
V		src/app/Console/Commands/{User/AddAlias.php ← UserAddAlias.php} (19 lines)
A	M	src/app/Console/Commands/User/AliasesCommand.php (13 lines)
A	M	src/app/Console/Commands/User/CreateCommand.php (12 lines)
V		src/app/Console/Commands/{User/Delete.php ← UserDelete.php} (19 lines)
V		src/app/Console/Commands/{User/Discount.php ← UserDiscount.php} (22 lines)
A	M	src/app/Console/Commands/User/DomainsCommand.php (13 lines)
A	M	src/app/Console/Commands/User/EntitlementsCommand.php (13 lines)
V		src/app/Console/Commands/{User/ForceDelete.php ← UserForceDelete.php} (17 lines)
A	M	src/app/Console/Commands/User/ReadCommand.php (12 lines)
V		src/app/Console/Commands/{User/Status.php ← UserStatus.php} (21 lines)
V		src/app/Console/Commands/{User/Suspend.php ← UserSuspend.php} (21 lines)
V		src/app/Console/Commands/{User/Unsuspend.php ← UserUnsuspend.php} (21 lines)
V		src/app/Console/Commands/{User/Verify.php ← UserVerify.php} (21 lines)
A	M	src/app/Console/Commands/User/WalletCommand.php (13 lines)
A	M	src/app/Console/Commands/User/WalletsCommand.php (13 lines)
V		src/app/Console/Commands/{UserAddAlias.php → User/AddAlias.php}
V		src/app/Console/Commands/{UserDelete.php → User/Delete.php}
V		src/app/Console/Commands/{UserDiscount.php → User/Discount.php}
D	M	src/app/Console/Commands/UserDomains.php (53 lines)
D	M	src/app/Console/Commands/UserEntitlements.php (65 lines)
V		src/app/Console/Commands/{UserForceDelete.php → User/ForceDelete.php}
A	M	src/app/Console/Commands/UserSetting/ReadCommand.php (12 lines)
A	M	src/app/Console/Commands/UserSettingsCommand.php (12 lines)
V		src/app/Console/Commands/{UserStatus.php → User/Status.php}
V		src/app/Console/Commands/{UserSuspend.php → User/Suspend.php}
V		src/app/Console/Commands/{UserUnsuspend.php → User/Unsuspend.php}
V		src/app/Console/Commands/{UserVerify.php → User/Verify.php}
D	M	src/app/Console/Commands/UserWallets.php (50 lines)
A	M	src/app/Console/Commands/UsersCommand.php (12 lines)
V		src/app/Console/Commands/{Wallet/AddTransaction.php ← WalletAddTransaction.php} (17 lines)
V		src/app/Console/Commands/{Wallet/Balances.php ← WalletBalances.php} (16 lines)
V		src/app/Console/Commands/{Wallet/Charge.php ← WalletCharge.php} (17 lines)
A	M	src/app/Console/Commands/Wallet/ControllersCommand.php (13 lines)
A	M	src/app/Console/Commands/Wallet/EntitlementsCommand.php (13 lines)
V		src/app/Console/Commands/{Wallet/Expected.php ← WalletExpected.php} (20 lines)
V		src/app/Console/Commands/{Wallet/GetBalance.php ← WalletGetBalance.php} (17 lines)
V		src/app/Console/Commands/{Wallet/GetDiscount.php ← WalletGetDiscount.php} (17 lines)
V		src/app/Console/Commands/{Wallet/Mandate.php ← WalletMandate.php} (17 lines)
A	M	src/app/Console/Commands/Wallet/OwnerCommand.php (13 lines)
A	M	src/app/Console/Commands/Wallet/ReadCommand.php (12 lines)
V		src/app/Console/Commands/{Wallet/SetBalance.php ← WalletSetBalance.php} (17 lines)
V		src/app/Console/Commands/{Wallet/SetDiscount.php ← WalletSetDiscount.php} (20 lines)
V		src/app/Console/Commands/{Wallet/Transactions.php ← WalletTransactions.php} (19 lines)
V		src/app/Console/Commands/{Wallet/Until.php ← WalletUntil.php} (17 lines)
V		src/app/Console/Commands/{WalletAddTransaction.php → Wallet/AddTransaction.php}
V		src/app/Console/Commands/{WalletBalances.php → Wallet/Balances.php}
V		src/app/Console/Commands/{WalletCharge.php → Wallet/Charge.php}
D	M	src/app/Console/Commands/WalletDiscount.php (62 lines)
V		src/app/Console/Commands/{WalletExpected.php → Wallet/Expected.php}
V		src/app/Console/Commands/{WalletGetBalance.php → Wallet/GetBalance.php}
V		src/app/Console/Commands/{WalletGetDiscount.php → Wallet/GetDiscount.php}
V		src/app/Console/Commands/{WalletMandate.php → Wallet/Mandate.php}
V		src/app/Console/Commands/{WalletSetBalance.php → Wallet/SetBalance.php}
V		src/app/Console/Commands/{WalletSetDiscount.php → Wallet/SetDiscount.php}
A	M	src/app/Console/Commands/WalletSetting/ReadCommand.php (12 lines)
A	M	src/app/Console/Commands/WalletSettingsCommand.php (12 lines)
V		src/app/Console/Commands/{WalletTransactions.php → Wallet/Transactions.php}
V		src/app/Console/Commands/{WalletUntil.php → Wallet/Until.php}
A	M	src/app/Console/Commands/WalletsCommand.php (12 lines)
D	M	src/app/Console/Development/DomainStatus.php (87 lines)
D	M	src/app/Console/Development/UserStatus.php (86 lines)
A	M	src/app/Console/ObjectCreateCommand.php (81 lines)
A	M	src/app/Console/ObjectListCommand.php (73 lines)
A	M	src/app/Console/ObjectReadCommand.php (56 lines)
A	M	src/app/Console/ObjectRelationListCommand.php (122 lines)
M		src/app/Domain.php (208 lines)
M		src/app/User.php (2 lines)
M		src/app/Utils.php (39 lines)
M		src/database/seeds/local/UserSeeder.php (2 lines)
P		src/tests/Feature/Console/Domain/AddTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
P		src/tests/Feature/Console/Domain/DeleteTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
A	M	src/tests/Feature/Console/Domain/ListDomainsTest.php (19 lines)
P		src/tests/Feature/Console/Domain/ListUsersTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
P		src/tests/Feature/Console/Domain/SetStatusTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
P		src/tests/Feature/Console/Domain/SetWalletTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
A	M	src/tests/Feature/Console/Domain/StatusTest.php (19 lines)
P		src/tests/Feature/Console/Domain/SuspendTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
P		src/tests/Feature/Console/Domain/UnsuspendTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
P		src/tests/Feature/Console/User/AddAliasTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
V		src/tests/Feature/Console/{User/DeleteTest.php ← WalletDiscountTest.php} (7 lines)
P		src/tests/Feature/Console/User/DiscountTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
V		src/tests/Feature/Console/{User/DomainsTest.php ← UserDomainsTest.php} (7 lines)
V		src/tests/Feature/Console/{User/EntitlementsTest.php ← UserEntitlementsTest.php} (7 lines)
V		src/tests/Feature/Console/{User/ForceDeleteTest.php ← UserForceDeleteTest.php} (4 lines)
A	M	src/tests/Feature/Console/User/StatusTest.php (19 lines)
P		src/tests/Feature/Console/User/SuspendTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
P		src/tests/Feature/Console/User/UnsuspendTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
P		src/tests/Feature/Console/User/VerifyTest.php (7 lines) Copied from src/tests/Feature/Console/UserDiscountTest.php
V		src/tests/Feature/Console/{User/WalletsTest.php ← UserWalletsTest.php} (7 lines)
P		src/tests/Feature/Console/UserDiscountTest.php Deleted after being copied to multiple locations: src/tests/Feature/Console/Domain/AddTest.php src/tests/Feature/Console/Domain/DeleteTest.php src/tests/Feature/Console/Domain/ListUsersTest.php src/tests/Feature/Console/Domain/SetStatusTest.php src/tests/Feature/Console/Domain/SetWalletTest.php src/tests/Feature/Console/Domain/SuspendTest.php src/tests/Feature/Console/Domain/UnsuspendTest.php src/tests/Feature/Console/User/AddAliasTest.php src/tests/Feature/Console/User/DiscountTest.php src/tests/Feature/Console/User/SuspendTest.php src/tests/Feature/Console/User/UnsuspendTest.php src/tests/Feature/Console/User/VerifyTest.php src/tests/Feature/Console/Wallet/MandateTest.php
V		src/tests/Feature/Console/{UserDomainsTest.php → User/DomainsTest.php}
V		src/tests/Feature/Console/{UserEntitlementsTest.php → User/EntitlementsTest.php}
V		src/tests/Feature/Console/{UserForceDeleteTest.php → User/ForceDeleteTest.php}
V		src/tests/Feature/Console/{UserWalletsTest.php → User/WalletsTest.php}
A	M	src/tests/Feature/Console/Wallet/AddTransactionTest.php (16 lines)
A	M	src/tests/Feature/Console/Wallet/BalancesTest.php (19 lines)
V		src/tests/Feature/Console/{Wallet/ChargeTest.php ← WalletChargeTest.php} (4 lines)
A	M	src/tests/Feature/Console/Wallet/ExpectedTest.php (19 lines)
A	M	src/tests/Feature/Console/Wallet/GetBalanceTest.php (16 lines)
A	M	src/tests/Feature/Console/Wallet/GetDiscountTest.php (16 lines)
V		src/tests/Feature/Console/{Wallet/MandateTest.php ← UserDiscountTest.php} (7 lines)
A	M	src/tests/Feature/Console/Wallet/SetBalanceTest.php (16 lines)
A	M	src/tests/Feature/Console/Wallet/SetDiscountTest.php (16 lines)
A	M	src/tests/Feature/Console/Wallet/TransactionsTest.php (16 lines)
A	M	src/tests/Feature/Console/Wallet/UntilTest.php (48 lines)
V		src/tests/Feature/Console/{WalletChargeTest.php → Wallet/ChargeTest.php}
V		src/tests/Feature/Console/{WalletDiscountTest.php → User/DeleteTest.php}

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	4522	7fb26db		Mon, Oct 5, 11:45 AM	★	★
Diff 2	4528	7fb26db	- Further reduce duplication	Mon, Oct 5, 11:52 AM	★	★
Diff 3	4534	7fb26db	- show-case implementation of 'relations'	Mon, Oct 5, 1:32 PM	★	★
Diff 4	4540	7fb26db	- Commentary and documentation	Mon, Oct 5, 6:23 PM	★	★
Diff 5	4546	7fb26db	- And... another iteration to experiment with.	Mon, Oct 5, 7:05 PM	★	★
Diff 6	4576	7fb26db	- Some create object logic	Tue, Oct 6, 12:48 PM	★	★

Commit	Tree	Parents	Author	Summary	Date
de5e7a14ee36	7627d0d0a502	bc014ddaf4e7	Jeroen van Meeuwen	Some create object logic	Tue, Oct 6, 12:47 PM
bc014ddaf4e7	8cd3183e5697	078520ab1884	Jeroen van Meeuwen	And... another iteration to experiment with.	Mon, Oct 5, 7:05 PM
078520ab1884	3a672ea61f2e	1577e85404d6	Jeroen van Meeuwen	More magic	Mon, Oct 5, 6:23 PM
1577e85404d6	5f9041a6ab80	61d45c618346	Jeroen van Meeuwen	user:wallets translation to the new system	Mon, Oct 5, 5:12 PM
61d45c618346	cc3661e58ce2	ee83ca0c7c09	Jeroen van Meeuwen	Commentary and documentation	Mon, Oct 5, 5:11 PM
ee83ca0c7c09	26e0d4b73fe0	b0658a786e01	Jeroen van Meeuwen	show-case implementation of 'relations'	Mon, Oct 5, 1:31 PM
b0658a786e01	a9d5dedf7d75	f606c70b4d17	Jeroen van Meeuwen	Further reduce duplication	Mon, Oct 5, 11:52 AM
f606c70b4d17	d4b649a46b4d	7532dfddbfd3	Jeroen van Meeuwen	Better structure for console commands, and unification of code/behavior (Show More…)	Mon, Oct 5, 11:44 AM
7532dfddbfd3	fdeac4c6948b	889b1122a23d	Jeroen van Meeuwen	Standardize downloading a file	Mon, Oct 5, 11:43 AM
889b1122a23d	f68da581b81c	4dff024b79e3 7fb26dbf8cc0	Jeroen van Meeuwen	Merge branch 'master' into arcpatch-D1684	Mon, Oct 5, 9:49 AM
4dff024b79e3	a2047b083270	ef964f4d9f5a	Aleksander Machniak	Better structure for console commands, and unification of code/behavior (Show More…)	Mon, Oct 5, 9:48 AM