[fix] introduce AbstractScope::setSystem

src/Model/Scope.php

@@ -75,6 +75,13 @@ public function addCondition($field, $operator = null, $value = null)
         return $this;
     }
 
+    protected function setSystem($system = true)
+    {
+        if ($this->isAnd()) {
+            foreach ($this->elements as $nestedCondition) {
+                $nestedCondition->setSystem($system);
+            }
+        }
 
         return $this;
     }

src/Model/Scope/AbstractScope.php

@@ -31,11 +31,16 @@ public function init(): void
 
         $this->_init();
 
+        // always set system flag if condition added to another condition
+        $this->setSystem($this->owner instanceof RootScope);
+
         $this->onChangeModel();
     }
 
     abstract protected function onChangeModel();
 
+    abstract protected function setSystem($system = true);
+
     /**
      * Get the model this condition is associated with.
      */

src/Model/Scope/Condition.php

@@ -34,6 +34,8 @@ class Condition extends AbstractScope
      */
     public $value;
 
+    protected $system = false;
+
     public const OPERATOR_EQUALS = '=';
     public const OPERATOR_DOESNOT_EQUAL = '!=';
     public const OPERATOR_GREATER = '>';
@@ -137,14 +139,21 @@ public function __construct($key, $operator = null, $value = null)
         }
     }
 
+    protected function setSystem($system = true)
+    {
+        $this->system = $system;
+
+        return $this;
+    }
+
     protected function onChangeModel(): void
     {
         if ($model = $this->getModel()) {
             // if we have a definitive scalar value for a field
             // sets it as default value for field and locks it
             // new records will automatically get this value assigned for the field
             // @todo: consider this when condition is part of OR scope
-            if ($this->operator === self::OPERATOR_EQUALS && !is_object($this->value) && !is_array($this->value)) {
+            if ($this->system && $this->setsDefiniteValue()) {
                 // key containing '/' means chained references and it is handled in toQueryArguments method
                 if (is_string($field = $this->key) && !$this->isTraversing()) {
                     $field = $model->getField($field);
@@ -229,6 +238,14 @@ public function isEmpty(): bool
         return array_filter([$this->key, $this->operator, $this->value]) ? false : true;
     }
 
+    /**
+     * Checks if condition sets a definitive scalar value for a field.
+     */
+    protected function setsDefiniteValue(): bool
+    {
+        return $this->operator === self::OPERATOR_EQUALS && !is_object($this->value) && !is_array($this->value);
+    }
+
     public function clear()
     {
         $this->key = $this->operator = $this->value = null;