refactoring

arojunior · arojunior · commit c0218957d261 · 2017-02-03T10:43:28.000-02:00
diff --git a/core/model/Model.php b/core/model/Model.php
@@ -35,7 +35,7 @@ private function fields($dados = null)
         foreach ($dados as $k => $v) {
             $fields[] = $k;
         }
-        $fields = implode(',', $fields);        
+        $fields = implode(',', $fields);
 
         return $fields;
     }
@@ -113,12 +113,13 @@ public function save($dados)
 
         if (isset($dados[$this->pk])) {
             $this->find([$this->pk => $dados[$this->pk]]);
-            if ($this->count > 0) {
-                $this->update($dados);
-            } else {
-                $this->create($dados);
-            }
         }
+
+        if ($this->count > 0) {
+            return $this->update($dados);
+        }
+
+        return $this->create($dados);
     }
 
     public function update($dados)

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ private function fields($dados = null)`
`35`	`35`	`foreach ($dados as $k => $v) {`
`36`	`36`	`$fields[] = $k;`
`37`	`37`	`}`
`38`		`- $fields = implode(',', $fields);`
	`38`	`+ $fields = implode(',', $fields);`
`39`	`39`
`40`	`40`	`return $fields;`
`41`	`41`	`}`
`@@ -113,12 +113,13 @@ public function save($dados)`
`113`	`113`
`114`	`114`	`if (isset($dados[$this->pk])) {`
`115`	`115`	`$this->find([$this->pk => $dados[$this->pk]]);`
`116`		`- if ($this->count > 0) {`
`117`		`- $this->update($dados);`
`118`		`- } else {`
`119`		`- $this->create($dados);`
`120`		`- }`
`121`	`116`	`}`
	`117`	`+`
	`118`	`+ if ($this->count > 0) {`
	`119`	`+ return $this->update($dados);`
	`120`	`+ }`
	`121`	`+`
	`122`	`+ return $this->create($dados);`
`122`	`123`	`}`
`123`	`124`
`124`	`125`	`public function update($dados)`