]> source.dussan.org Git - nextcloud-server.git/commitdiff
Make abstract Middleware class public
authorThomas Tanghus <thomas@tanghus.net>
Sat, 5 Oct 2013 14:59:06 +0000 (16:59 +0200)
committerThomas Tanghus <thomas@tanghus.net>
Sat, 5 Oct 2013 14:59:06 +0000 (16:59 +0200)
It doesn't make sense for subclasses to have to implement
all methods.

lib/private/appframework/middleware/middleware.php [deleted file]
lib/private/appframework/middleware/middlewaredispatcher.php
lib/private/appframework/middleware/security/securitymiddleware.php
lib/public/appframework/middleware.php [new file with mode: 0644]
tests/lib/appframework/middleware/MiddlewareDispatcherTest.php
tests/lib/appframework/middleware/MiddlewareTest.php

diff --git a/lib/private/appframework/middleware/middleware.php b/lib/private/appframework/middleware/middleware.php
deleted file mode 100644 (file)
index b12c03c..0000000
+++ /dev/null
@@ -1,100 +0,0 @@
-<?php
-
-/**
- * ownCloud - App Framework
- *
- * @author Bernhard Posselt
- * @copyright 2012 Bernhard Posselt nukeawhale@gmail.com
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE
- * License as published by the Free Software Foundation; either
- * version 3 of the License, or any later version.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU AFFERO GENERAL PUBLIC LICENSE for more details.
- *
- * You should have received a copy of the GNU Affero General Public
- * License along with this library.  If not, see <http://www.gnu.org/licenses/>.
- *
- */
-
-
-namespace OC\AppFramework\Middleware;
-
-use OCP\AppFramework\Http\Response;
-
-
-/**
- * Middleware is used to provide hooks before or after controller methods and
- * deal with possible exceptions raised in the controller methods.
- * They're modeled after Django's middleware system:
- * https://docs.djangoproject.com/en/dev/topics/http/middleware/
- */
-abstract class Middleware {
-
-
-       /**
-        * This is being run in normal order before the controller is being
-        * called which allows several modifications and checks
-        *
-        * @param Controller $controller the controller that is being called
-        * @param string $methodName the name of the method that will be called on
-        *                           the controller
-        */
-       public function beforeController($controller, $methodName){
-
-       }
-
-
-       /**
-        * This is being run when either the beforeController method or the
-        * controller method itself is throwing an exception. The middleware is
-        * asked in reverse order to handle the exception and to return a response.
-        * If the response is null, it is assumed that the exception could not be
-        * handled and the error will be thrown again
-        *
-        * @param Controller $controller the controller that is being called
-        * @param string $methodName the name of the method that will be called on
-        *                           the controller
-        * @param \Exception $exception the thrown exception
-        * @throws \Exception the passed in exception if it cant handle it
-        * @return Response a Response object in case that the exception was handled
-        */
-       public function afterException($controller, $methodName, \Exception $exception){
-               throw $exception;
-       }
-
-
-       /**
-        * This is being run after a successful controllermethod call and allows
-        * the manipulation of a Response object. The middleware is run in reverse order
-        *
-        * @param Controller $controller the controller that is being called
-        * @param string $methodName the name of the method that will be called on
-        *                           the controller
-        * @param Response $response the generated response from the controller
-        * @return Response a Response object
-        */
-       public function afterController($controller, $methodName, Response $response){
-               return $response;
-       }
-
-
-       /**
-        * This is being run after the response object has been rendered and
-        * allows the manipulation of the output. The middleware is run in reverse order
-        *
-        * @param Controller $controller the controller that is being called
-        * @param string $methodName the name of the method that will be called on
-        *                           the controller
-        * @param string $output the generated output from a response
-        * @return string the output that should be printed
-        */
-       public function beforeOutput($controller, $methodName, $output){
-               return $output;
-       }
-
-}
index 70ab108e6b884d8404504a2db5bc246c3c2aa578..5cce97b216773d4d7ec3b6e0b24354be01cd2161 100644 (file)
@@ -26,7 +26,7 @@ namespace OC\AppFramework\Middleware;
 
 use OC\AppFramework\Controller\Controller;
 use OCP\AppFramework\Http\Response;
-
+use OCP\AppFramework\IMiddleWare;
 
 /**
  * This class is used to store and run all the middleware in correct order
@@ -58,7 +58,7 @@ class MiddlewareDispatcher {
         * Adds a new middleware
         * @param Middleware $middleware the middleware which will be added
         */
-       public function registerMiddleware(Middleware $middleWare){
+       public function registerMiddleware(IMiddleware $middleWare){
                array_push($this->middlewares, $middleWare);
        }
 
index 4f1447e1afba11ffea26f40792be80c392b053b3..d6daf737bb4f48a0125b49aec09a68ec1febf07f 100644 (file)
@@ -29,8 +29,8 @@ use OC\AppFramework\Http\Http;
 use OC\AppFramework\Http\Request;
 use OC\AppFramework\Http\RedirectResponse;
 use OC\AppFramework\Utility\MethodAnnotationReader;
-use OC\AppFramework\Middleware\Middleware;
 use OC\AppFramework\Core\API;
+use OCP\AppFramework\Middleware;
 use OCP\AppFramework\Http\Response;
 use OCP\AppFramework\Http\JSONResponse;
 
diff --git a/lib/public/appframework/middleware.php b/lib/public/appframework/middleware.php
new file mode 100644 (file)
index 0000000..0f08eae
--- /dev/null
@@ -0,0 +1,100 @@
+<?php
+
+/**
+ * ownCloud - App Framework
+ *
+ * @author Bernhard Posselt
+ * @copyright 2012 Bernhard Posselt nukeawhale@gmail.com
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE
+ * License as published by the Free Software Foundation; either
+ * version 3 of the License, or any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU AFFERO GENERAL PUBLIC LICENSE for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public
+ * License along with this library.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+
+namespace OCP\AppFramework;
+
+use OCP\AppFramework\Http\Response;
+
+
+/**
+ * Middleware is used to provide hooks before or after controller methods and
+ * deal with possible exceptions raised in the controller methods.
+ * They're modeled after Django's middleware system:
+ * https://docs.djangoproject.com/en/dev/topics/http/middleware/
+ */
+abstract class Middleware implements IMiddleWare {
+
+
+       /**
+        * This is being run in normal order before the controller is being
+        * called which allows several modifications and checks
+        *
+        * @param Controller $controller the controller that is being called
+        * @param string $methodName the name of the method that will be called on
+        *                           the controller
+        */
+       public function beforeController($controller, $methodName){
+
+       }
+
+
+       /**
+        * This is being run when either the beforeController method or the
+        * controller method itself is throwing an exception. The middleware is
+        * asked in reverse order to handle the exception and to return a response.
+        * If the response is null, it is assumed that the exception could not be
+        * handled and the error will be thrown again
+        *
+        * @param Controller $controller the controller that is being called
+        * @param string $methodName the name of the method that will be called on
+        *                           the controller
+        * @param \Exception $exception the thrown exception
+        * @throws \Exception the passed in exception if it cant handle it
+        * @return Response a Response object in case that the exception was handled
+        */
+       public function afterException($controller, $methodName, \Exception $exception){
+               throw $exception;
+       }
+
+
+       /**
+        * This is being run after a successful controllermethod call and allows
+        * the manipulation of a Response object. The middleware is run in reverse order
+        *
+        * @param Controller $controller the controller that is being called
+        * @param string $methodName the name of the method that will be called on
+        *                           the controller
+        * @param Response $response the generated response from the controller
+        * @return Response a Response object
+        */
+       public function afterController($controller, $methodName, Response $response){
+               return $response;
+       }
+
+
+       /**
+        * This is being run after the response object has been rendered and
+        * allows the manipulation of the output. The middleware is run in reverse order
+        *
+        * @param Controller $controller the controller that is being called
+        * @param string $methodName the name of the method that will be called on
+        *                           the controller
+        * @param string $output the generated output from a response
+        * @return string the output that should be printed
+        */
+       public function beforeOutput($controller, $methodName, $output){
+               return $output;
+       }
+
+}
index 43727846dcf3863af85aa4b14f272d060b805ec6..dd85a9ad52f3c822c332f71cd291d0fc46af6497 100644 (file)
@@ -25,8 +25,8 @@
 namespace OC\AppFramework;
 
 use OC\AppFramework\Http\Request;
-use OC\AppFramework\Middleware\Middleware;
 use OC\AppFramework\Middleware\MiddlewareDispatcher;
+use OCP\AppFramework\Middleware;
 use OCP\AppFramework\Http\Response;
 
 
@@ -142,12 +142,12 @@ class MiddlewareDispatcherTest extends \PHPUnit_Framework_TestCase {
 
        public function testAfterExceptionShouldReturnResponseOfMiddleware(){
                $response = new Response();
-               $m1 = $this->getMock('\OC\AppFramework\Middleware\Middleware',
+               $m1 = $this->getMock('\OCP\AppFramework\Middleware',
                                array('afterException', 'beforeController'));
                $m1->expects($this->never())
                                ->method('afterException');
 
-               $m2 = $this->getMock('OC\AppFramework\Middleware\Middleware',
+               $m2 = $this->getMock('OCP\AppFramework\Middleware',
                                array('afterException', 'beforeController'));
                $m2->expects($this->once())
                                ->method('afterException')
@@ -267,7 +267,7 @@ class MiddlewareDispatcherTest extends \PHPUnit_Framework_TestCase {
        public function testExceptionShouldRunAfterExceptionOfOnlyPreviouslyExecutedMiddlewares(){
                $m1 = $this->getMiddleware();
                $m2 = $this->getMiddleware(true);
-               $m3 = $this->getMock('\OC\AppFramework\Middleware\Middleware');
+               $m3 = $this->getMock('\OCP\AppFramework\Middleware');
                $m3->expects($this->never())
                                ->method('afterException');
                $m3->expects($this->never())
index 5e2930ac6a36a426127e164286ae1093a8006680..d0be7f7ca74994f63f3f6453bff3fa4c0c039877 100644 (file)
@@ -25,7 +25,7 @@
 namespace OC\AppFramework;
 
 use OC\AppFramework\Http\Request;
-use OC\AppFramework\Middleware\Middleware;
+use OCP\AppFramework\Middleware;
 
 
 class ChildMiddleware extends Middleware {};