Skip to content

Fix a bug when related fields were not filled in BelongsToMorphed#550

Merged
roxblnfk merged 1 commit into2.xfrom
notfix/belongs-to-morphed
Feb 10, 2026
Merged

Fix a bug when related fields were not filled in BelongsToMorphed#550
roxblnfk merged 1 commit into2.xfrom
notfix/belongs-to-morphed

Conversation

@roxblnfk
Copy link
Member

🔍 What was changed

Fixed a bug where the linking fields were not mapped when saving a BelongsToMorphed relation by assigning a related entity only.

Can't reproduce in tests

📝 Checklist

  • Closes #
  • How was this tested:
    • Tested manually
    • Unit tests added

@codecov
Copy link

codecov bot commented Feb 10, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 91.73%. Comparing base (72ee07e) to head (ca53d95).
⚠️ Report is 1 commits behind head on 2.x.

Files with missing lines Patch % Lines
src/Relation/BelongsTo.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##                2.x     #550   +/-   ##
=========================================
  Coverage     91.73%   91.73%           
  Complexity     1924     1924           
=========================================
  Files           123      123           
  Lines          4958     4958           
=========================================
  Hits           4548     4548           
  Misses          410      410           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@roxblnfk roxblnfk merged commit 4be87c3 into 2.x Feb 10, 2026
31 of 34 checks passed
@roxblnfk roxblnfk deleted the notfix/belongs-to-morphed branch February 10, 2026 08:37
@roxblnfk roxblnfk changed the title Fix a bug when related fields were not filled Fix a bug when related fields were not filled in BelongsToMorphed Feb 10, 2026
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