idiomatic-ar.php

<?php

# Discussion about idiomatic ActiveRecord usage
## Please help me to determine the idiomatic approach for creating a member and attaching an invoice ONLY IF the member doesn't already have 5 invoices.

class Member extends Eloquent {

}

$member = Member::create([
    'email'    => $email,
    'password' => $password,
]);

// Then later, when they are adding invoices

// my gut says that this isn't idiomatic.. suggestions?
if ($member->invoices()->count() >= 5) {
    throw new MemberCannotHaveMoreThanFiveInvoices($member->id);
}

$member->invoices()->create([
    // ...
]);

@zackkitzmiller I usually keep the happy path outdented and guard the less common cases up front, but not religious about it 👍