View Issue Details

IDProjectCategoryView StatusLast Update
0006605OXID eShop (all versions)1.05. Userspublic2024-09-17 10:44
Reporterm0rb Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionno change required 
Product Version4.10.3 / 5.3.3 
Summary0006605: oxbase :: No check for useSkipUpdateFields in _udpate
DescriptionIn the _update()-function in oxbase a query string is build. One part of this query is generated by function $this->_getUpdateFields(). This function has the optional param $blUseSkipSaveFields, which isn't used in this context. Thatswhy the defined skipsaveFields don't work for updates. That's a bug I think.
The fix would be to call the _getUpdateFields the same way like in the _insert-function:
$this->_getUpdateFields($this->getUseSkipSaveFields())
Steps To ReproduceAdd skipsavefields to a model and execute save-function for an existing element (oxaddress).
TagsNo tags attached.
ThemeNot defined
BrowserNot defined
PHP VersionNot defined
Database VersionNot defined

Activities

SvenBrunk

2024-09-17 10:44

administrator   ~0017542

You are wrong. The default is $useSkipSaveFields = true in getUpdateFields which will then by default consider the skipped fields:
if (
                !$useSkipSaveFields
--> || ($useSkipSaveFields && !in_array(strtolower($oneFieldName), $this->_aSkipSaveFields))