Skip to content

Support Laravel 13 and improve basket operations - #6

Merged
Jord-JD merged 5 commits into
masterfrom
agent/laravel-13-basket-improvements
Jul 18, 2026
Merged

Support Laravel 13 and improve basket operations#6
Jord-JD merged 5 commits into
masterfrom
agent/laravel-13-basket-improvements

Conversation

@Jord-JD

@Jord-JD Jord-JD commented Jul 18, 2026

Copy link
Copy Markdown
Owner

What changed

  • support Laravel 11, 12, and 13 while restoring the package runtime floor to its actual PHP 7.1 syntax requirement
  • add explicit compatibility jobs for Laravel 5.5 through 13
  • fix test migration registration so product fixtures exist before tests run
  • automatically persist a new basket when its first item is added and reject unsaved basketable models clearly
  • return the created or merged item from add() and respect Laravel morph-map aliases
  • skip orphaned polymorphic rows while matching additions and provide a clear error when their price is requested
  • add clear() for deleting every item in one operation
  • replace Travis/Coveralls with GitHub Actions and refresh the documentation

Compatibility

Existing APIs remain valid. Laravel 5.5 remains supported, the package PHP floor is lowered from 8.1 to 7.1, and Composer selects the PHP floor required by each Laravel major.

Security triage

The two open Laravel advisories concern signed URL paths and the email validation rule, neither of which this basket library invokes. Keeping older Laravel majors installable is intentional; current applications resolve patched Laravel releases through Composer.

Validation

  • git diff --check
  • composer.json parsed with jq
  • GitHub Actions covers every supported Laravel major from 5.5 through 13

@Jord-JD
Jord-JD merged commit 65d8b5b into master Jul 18, 2026
18 of 20 checks passed
@Jord-JD
Jord-JD deleted the agent/laravel-13-basket-improvements branch July 18, 2026 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant