yii2 icon indicating copy to clipboard operation
yii2 copied to clipboard

Remove magic synchronization for relations (issue #19788).

Open PowerGamer1 opened this issue 1 year ago • 2 comments

Q A
Is bugfix? ✔️
New feature?
Breaks BC? ✔️
Fixed issues #19788

PowerGamer1 avatar Jul 27 '23 15:07 PowerGamer1

PR Summary

  • Removal of Non-Functional Synchronization Attempts The system will no longer make non-functional attempts to automatically synchronize ActiveRecord relations with corresponding foreign key values. This change affects framework/CHANGELOG.md and framework/UPGRADE.md.

  • GridView Improvements There's an improvement on GridView; it no longer calls Model::generateAttributeLabel() unnecessarily to generate label values that are not being used. This change is denoted in framework/CHANGELOG.md.

  • FileTarget Log Update The logging process for yii\log\FileTarget has been amended, and it will no longer export empty messages - as detailed in framework/CHANGELOG.md.

  • Database Query Caching Issues surrounding the caching of a MSSQL query with BLOB data type has been fixed in framework/CHANGELOG.md.

  • Removal of Unnecessary Database Calls In framework/db/BaseActiveRecord.php, redundant calls to $this->resetDependentRelations() and $this->setRelationDependencies() have been removed.

  • New Column Addition A new column profile_id has been included in tests/data/ar/Customer.php.

  • Removal of Superfluous Tables and Views Unnecessary tables and views in all files located at tests/data/ have been removed.

  • Manual Unsetting of Relations A note has been added in framework/UPGRADE.md about the requirement to manually unset relations before using them if they are anticipated to change.

what-the-diff[bot] avatar Jul 27 '23 15:07 what-the-diff[bot]

Codecov Report

Patch coverage has no change and project coverage change: -0.04% :warning:

Comparison is base (c8c0ea9) 48.89% compared to head (bfe2ef9) 48.85%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #19918      +/-   ##
==========================================
- Coverage   48.89%   48.85%   -0.04%     
==========================================
  Files         445      445              
  Lines       42786    42754      -32     
==========================================
- Hits        20919    20887      -32     
  Misses      21867    21867              
Files Changed Coverage Δ
framework/db/BaseActiveRecord.php 72.40% <ø> (-1.67%) :arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jul 27 '23 16:07 codecov[bot]