fix: trim whitespace from string attributes on BaseModel

[bpotmalnik]

Laravel's TrimStrings middleware is intentionally skipped for Livewire requests (Livewire\Mechanisms\HandleRequests\HandleRequests calls TrimStrings::skipWhen). This means leading/trailing whitespace entered via Livewire forms (e.g. the admin panel) is never stripped before reaching the model, allowing values like " size " to be persisted.

Override setAttribute() on BaseModel to trim all incoming string values before delegating to the parent. This covers every model in Lunar (handle, sku, slug, coupon, code, reference, postcode, etc.)

Fixes #1664

[bpotmalnik]

I was considering if it is breaking change or not, but it would break things only if people would assert things agains hardcoded strings (which likelihood is pretty low) so I think it can got to v1.

[wychoong]

I don’t think this is a suitable fix

• it’s too heavy handed (a Model::update([…]) will be affected too)
• Not the common livewire/filament form expectation
• Probably more suitable to implement Input::configureUsing( ->dehydrateStateUsing ) in your app to trim the fields

[bpotmalnik]

@wychoong I hear you. I still think that reinstating default Laravel behavior would be handy for users (developers) using the package.

As it is not indeed default filament behavior, but this package is kind of special case where lunar is in control of both core and admin panel itself (filament) which can drive handy behaviors.