Page MenuHomekolab.org

BelongsToTenantTrait
ClosedPublic

Authored by mollekopf on Aug 24 2021, 9:06 AM.

Details

Reviewers
machniak
Group Reviewers
Restricted Project
Commits
rK098289d3f9b1: BelongsToTenantTrait

Diff Detail

Repository
rK kolab
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 36430
Build 14446: arc lint + arc unit

Event Timeline

mollekopf requested review of this revision.Aug 24 2021, 9:06 AM
mollekopf created this revision.
mollekopf added a reviewer: Restricted Project.Aug 24 2021, 9:08 AM
mollekopf updated this revision to Diff 8050.Aug 24 2021, 9:37 AM

Whitespace

machniak requested changes to this revision.Aug 25 2021, 12:56 PM
machniak added a subscriber: machniak.
machniak added inline comments.
src/app/BelongsToTentantTrait.php
3 ↗(On Diff #8050)

It should be app/Traits/BelongsToTentantTrait.php

10 ↗(On Diff #8050)

I'm pretty sure this is not going to fly if there are two traits with boot() method, i.e. the uuid trait. Am I wrong?

This revision now requires changes to proceed.Aug 25 2021, 12:56 PM
mollekopf updated this revision to Diff 8104.Aug 26 2021, 1:53 PM

Use the trait boot method so we can have multiple traits with boot methods

mollekopf updated this revision to Diff 8116.Aug 26 2021, 2:24 PM

Rebased on uuidtraitpatch

machniak accepted this revision.Aug 27 2021, 9:00 AM
This revision is now accepted and ready to land.Aug 27 2021, 9:00 AM
This revision was automatically updated to reflect the committed changes.