Merge pull request #16499 from niden/T16498-model-toarray-segfault

reworking the toarray conditionals
phalcon · Jan 9, 2024 · d9be887 · d9be887
2 parents 9bb774a + 379040f
commit d9be887
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/phalcon/Mvc/Model.zep b/phalcon/Mvc/Model.zep
@@ -3284,7 +3284,7 @@ abstract class Model extends AbstractInjectionAware implements EntityInterface,
      */
     public function toArray(columns = null, useGetter = true) -> array
     {
-        var attribute, attributeField, columnMap, metaData, method, value;
+        var attribute, attributeField, columnMap, metaData, method;
         array data;
 
         let data = [],
@@ -3330,8 +3330,8 @@ abstract class Model extends AbstractInjectionAware implements EntityInterface,
 
             if true === useGetter && method_exists(this, method) {
                 let data[attributeField] = this->{method}();
-            } elseif fetch value, this->{attributeField} {
-                let data[attributeField] = value;
+            } elseif isset(this->{attributeField}) {
+                let data[attributeField] = this->{attributeField};
             } else {
                 let data[attributeField] = null;
             }