Update readme.md

burzum · web-flow · commit 58feabe66bdf · 2019-03-13T07:44:05.000+01:00
diff --git a/readme.md b/readme.md
@@ -63,14 +63,16 @@ class FooController extends AppController
     {
         $this->set('results', $this->Articles->getListingForUser(
             $this->Auth->user('id')
-            $this->request
+            $this->getRequest()->getQueryParams()
         ));
     }
 }
 ```
 
 If there is already a property with the name of the service used in the controller a warning will be thrown. In an ideal case your controller won't have to use any table instances anyway when using services. The tables are not a concern of the controller.
 
+The advantage of the above code is that the args passed to the service could come from shell input or any other source. The logic isn't tied to the controller nor the model. Using proper abstraction, the underlying data source, a repository that is used by the service, should be transparently replaceable with any interface that matches the required implementation.
+
 For details see **[docs](/docs)**.
 
 ## Why no DI container?

Original file line number	Diff line number	Diff line change
`@@ -63,14 +63,16 @@ class FooController extends AppController`
`63`	`63`	`{`
`64`	`64`	`$this->set('results', $this->Articles->getListingForUser(`
`65`	`65`	`$this->Auth->user('id')`
`66`		`- $this->request`
	`66`	`+ $this->getRequest()->getQueryParams()`
`67`	`67`	`));`
`68`	`68`	`}`
`69`	`69`	`}`
`70`	`70`	```
`71`	`71`
`72`	`72`	`If there is already a property with the name of the service used in the controller a warning will be thrown. In an ideal case your controller won't have to use any table instances anyway when using services. The tables are not a concern of the controller.`
`73`	`73`
	`74`	`+The advantage of the above code is that the args passed to the service could come from shell input or any other source. The logic isn't tied to the controller nor the model. Using proper abstraction, the underlying data source, a repository that is used by the service, should be transparently replaceable with any interface that matches the required implementation.`
	`75`	`+`
`74`	`76`	`For details see [docs](/docs).`
`75`	`77`
`76`	`78`	`## Why no DI container?`