minor #7985 Use of Setters and Getters (martin05)

weaverryan · weaverryan · commit 83ef27debfa5 · 2017-07-06T06:36:14.000-04:00
This PR was submitted for the 3.3 branch but it was merged into the 2.7 branch instead (closes #7985). Discussion ---------- Use of Setters and Getters If I have already define getter for the $targetDir, I should use it. Commits ------- 789ed43 Use of Setters and Getters
diff --git a/controller/upload_file.rst b/controller/upload_file.rst
@@ -243,7 +243,7 @@ logic to a separate service::
         {
             $fileName = md5(uniqid()).'.'.$file->guessExtension();
 
-            $file->move($this->targetDir, $fileName);
+            $file->move($this->getTargetDir(), $fileName);
 
             return $fileName;
         }

Original file line number	Diff line number	Diff line change
`@@ -243,7 +243,7 @@ logic to a separate service::`
`243`	`243`	`{`
`244`	`244`	`$fileName = md5(uniqid()).'.'.$file->guessExtension();`
`245`	`245`
`246`		`- $file->move($this->targetDir, $fileName);`
	`246`	`+ $file->move($this->getTargetDir(), $fileName);`
`247`	`247`
`248`	`248`	`return $fileName;`
`249`	`249`	`}`