Support full REST relationship

README.md

@@ -192,8 +192,8 @@ The following RESTful controller methods could be add by your need. which each m
 public function index() {}
 protected function store($requestData=null) {}
 protected function show($resourceID) {}
-protected function update($resourceID, $requestData=null) {}
-protected function delete($resourceID=null) {}
+protected function update($resourceID=null, $requestData=null) {}
+protected function delete($resourceID=null, $requestData=null) {}
 ```
 
 > `$requestData` (array) is the raw body from request
@@ -371,3 +371,5 @@ REFERENCE
 
 - [RFC6750 - The OAuth 2.0 Authorization Framework: Bearer Token Usage](https://tools.ietf.org/html/rfc6750)
 
+- [REST Relationship between URL and HTTP methods](https://en.wikipedia.org/wiki/Representational_state_transfer#Relationship_between_URL_and_HTTP_methods)
+

examples/RestController.php

@@ -29,12 +29,13 @@ protected function show($resourceID) {}
  * @param int|string $resourceID
  * @param array $requestData
  */
- protected function update($resourceID, $requestData=null) {}
+ protected function update($resourceID=null, $requestData=null) {}
 
  /**
  * Action: Delete
  * 
- * @param int|string $resourceID
+ * @param int|string $resourceID Support single resource delete
+ * @param array $requestData Support delete parameters
  */
- protected function delete($resourceID) {} 
+ protected function delete($resourceID=null, $requestData=null) {} 
 }

src/rest/Controller.php

@@ -12,6 +12,7 @@
  * @version 1.4.0
  * @link https://github.com/yidas/codeigniter-rest/
  * @see https://github.com/yidas/codeigniter-rest/blob/master/examples/RestController.php
+ * @see https://en.wikipedia.org/wiki/Representational_state_transfer#Relationship_between_URL_and_HTTP_methods
  * 
  * Controller extending:
  * ```php
@@ -103,18 +104,16 @@ public function route($resourceID=NULL)
  return $this->_action(['store', $this->request->getBodyParams()]);
  }
  break;
- case 'PUT':
  case 'PATCH':
- if ($resourceID) {
- return $this->_action(['update', $resourceID, $this->request->getBodyParams()]);
+ // PATCH could only allow single element
+ if (!$resourceID) {
+ return $this->_defaultAction();
  }
+ case 'PUT':
+ return $this->_action(['update', $resourceID, $this->request->getBodyParams()]);
  break;
  case 'DELETE':
- if ($resourceID) {
- return $this->_action(['delete', $resourceID]);
- } else {
- return $this->_action(['delete']);
- }
+ return $this->_action(['delete', $resourceID, $this->request->getBodyParams()]);
  break;
  case 'GET':
  default: