Sta illuminate/support ^13.0 toe voor Laravel 13-compatibiliteit - #1
Open
janalwin wants to merge 1 commit into
Open
Sta illuminate/support ^13.0 toe voor Laravel 13-compatibiliteit#1janalwin wants to merge 1 commit into
janalwin wants to merge 1 commit into
Conversation
Overgenomen uit saintsystems#196 (Laravel Shift). Alleen de constraint verruimd; het pakket gebruikt van het framework enkel Arr, Str, Collection, LazyCollection, Arrayable, de Date-facade en Carbon, die alle ongewijzigd zijn in Laravel 13. De Shift-waarschuwing over src/Entity.php (LogicException bij het booten van models) is niet van toepassing: SaintSystems\OData\Entity is geen Eloquent-model maar een standalone klasse met eigen boot-logica. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Verruimt
illuminate/supportnaar^13.0zodat dit pakket geen blokkade meer vormt voor de Laravel 13-upgrade van de matching-API (applicant_profiles, branchJA/laravel-13).Overgenomen uit de Laravel Shift-PR upstream: saintsystems#196. Die staat daar open sinds 23-02-2026 zonder review; upstream lijkt dormant (laatste release 1.0.1), dus we nemen de wijziging in eigen beheer zoals bij onze andere forks.
Waarom er geen codewijziging nodig is
Het pakket gebruikt van het framework alleen
Arr,Str,Collection,LazyCollection,Arrayable, deDate-facade en Carbon. Die zijn allemaal ongewijzigd in Laravel 13.De Shift-bot waarschuwt over
src/Entity.php(Laravel 13 gooit eenLogicExceptionals je tijdens het booten van een model instances aanmaakt of observers registreert). Dat is een false positive:SaintSystems\OData\Entityis geen Eloquent-model maar een standaloneclass Entity implements ArrayAccess, Arrayablemet een eigenboot()/bootTraits()-implementatie. De nieuwe exception komt uit Eloquents boot-pad en raakt dit niet.PHP 8.5 vroeg ook geen wijziging: de bestaande constraint
^8.4betekent>=8.4 <9.0.Basis
Deze branch staat op upstream
master, wat dezelfde commit is als tag1.0.1(fdb08b2) — dus geen los ontwikkelwerk. We stonden gelockt op 1.0.0, dus we nemen de 1.0.0 → 1.0.1-diff mee: eenContent-Type-fix voor batch-requests en een ontbrekendeuseinPsr17HttpProvider. We gebruiken geen batch-requests.Getest
In
applicant_profilesopJA/laravel-13metsaintsystems/odata-client: dev-JA/laravel-13-support:./start-unit-test.sh --filter SK123→ 147 tests, 543 assertions, OK (1 skipped)./start-unit-test.sh --filter DwEducation→ 13 tests, 37 assertions, OK🤖 Generated with Claude Code