Merge pull request #4090 from najdanovicivan/nosql/base-model-fix

BaseModel/Model - Removed $escape from doUpdate

Author: michalsn

system/BaseModel.php

@@ -377,13 +377,12 @@ abstract protected function doInsertBatch(?array $set = null, ?bool $escape = nu
 	 * Updates a single record in the database.
 	 * This methods works only with dbCalls
 	 *
-	 * @param integer|array|string|null $id     ID
-	 * @param array|null                $data   Data
-	 * @param boolean|null              $escape Escape
+	 * @param integer|array|string|null $id   ID
+	 * @param array|null                $data Data
 	 *
 	 * @return boolean
 	 */
-	abstract protected function doUpdate($id = null, $data = null, ?bool $escape = null): bool;
+	abstract protected function doUpdate($id = null, $data = null): bool;
 
 	/**
 	 * Compiles an update and runs the query

system/Model.php

@@ -321,13 +321,12 @@ protected function doInsertBatch(?array $set = null, ?bool $escape = null, int $
 	 * Updates a single record in $this->table.
 	 * This methods works only with dbCalls
 	 *
-	 * @param integer|array|string|null $id     ID
-	 * @param array|null                $data   Data
-	 * @param boolean|null              $escape Escape
+	 * @param integer|array|string|null $id   ID
+	 * @param array|null                $data Data
 	 *
 	 * @return boolean
 	 */
-	protected function doUpdate($id = null, $data = null, ?bool $escape = null): bool
+	protected function doUpdate($id = null, $data = null): bool
 	{
 		$escape       = $this->escape;
 		$this->escape = null;