[alenarmona]

Accept array callables to non-public methods in getBehavior() again

6.0.0 typed Builder::getBehavior()'s first parameter as string|callable. defaultBehavior() and Manager::invoke() pass their strategy through that parameter, so a [$this, 'protectedMethod'] array is rejected by PHP before it reaches the branch that binds it into a closure (callable is scope-sensitive: a protected method is not callable from the Builder's scope). Swivel 4 accepted it, this repo's README documents it (defaultBehavior([$this, 'renderOldView']) with a protected renderOldView), and admin, public and service all use it.

TypeError: Zumba\Swivel\Builder::getBehavior(): Argument #1 ($slug) must be of type callable|string, array given, called in src/Builder.php on line 102

Change

• Builder::getBehavior() and BuilderInterface::getBehavior(): first parameter is string|callable|array, so the existing bindTo() branch handles the array as in 4.x. No other behavior change.
• BuilderTest: two regression tests (getBehavior() with a protected array callable as the strategy; defaultBehavior() with a protected target through execute()). Both fail on 6.0.1 and pass here. Suite: 173 tests, PSR-2 clean.

Who is affected on 6.0.1

• admin (swivel-cake 3.0.0 → swivel 6.0.1): TrainingsController::membershipFees() is private and is the default of ZinRefactor.Benefit.ReplaceRole; it throws whenever that flag is off for the bucket.
• public (same versions, deploying): its 27 array-callable defaults all target public methods, so no failure today, but the README pattern would break on first use.
• service via zumba/primer: 17 production sites target protected methods, which is what blocks primer from requiring swivel 6 (zumba/primer#349).

Suggested tag: v6.0.2. swivel-cake already allows it (^6.0.1); admin and public pick it up with composer update zumba/swivel.

🤖 Generated with Claude Code