New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Data objects: saving relations, exclude non-relational fields from condition #16598
base: 11.2
Are you sure you want to change the base?
Conversation
Review Checklist
|
This comment was marked as resolved.
This comment was marked as resolved.
Quality Gate passedIssues Measures |
if (!$this->model->isLazyKeyLoaded($fieldName) || $fd instanceof DataObject\ClassDefinition\Data\ReverseObjectRelation) { | ||
//this is a relation subject to lazy loading - it has not been loaded | ||
$untouchable[] = $fieldName; | ||
if($fd instanceof DataObject\ClassDefinition\Data\Relations\AbstractRelations) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please share some insights behind this change?
By following the $untouchable
var usages, ended up in this case in URL Slug data type
pimcore/models/DataObject/ClassDefinition/Data/UrlSlug.php
Lines 176 to 178 in 8d64417
if (isset($params['isUntouchable']) && $params['isUntouchable']) { | |
return; | |
} |
Before PR it's true
, after PR is always false
, not sure if there was something legacy leftover but looks like a potential behavior change
No description provided.