This doesn't change any of the existing commands, making the overall approach more dense for clarity.
Details
Details
None
Diff Detail
Diff Detail
- Repository
- rK kolab
- Branch
- arcpatch-D1996
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 32359 Build 12475: arc lint + arc unit
Event Timeline
Comment Actions
- Remove redundant object retrieval
- Modify how the signature for ObjectListCommand is built
| src/app/Console/Commands/DomainsCommand.php | ||
|---|---|---|
| 24 | How to get multiple attributes might be not obvious for outsiders. The example should use two attributes. | |
| src/app/Console/ObjectReadCommand.php | ||
| 38 | How about an error message here? | |
| src/app/Console/ObjectUpdateCommand.php | ||
| 76 | An error message here. | |
| 87 | We should make sure this timestamps=false really works. | |
| src/app/Console/ObjectUpdateCommand.php | ||
|---|---|---|
| 87 | It doesn't :/ It's a laravel 8.x-ism i think. | |