[Routing] moved protected to private

symfony · Mar 23, 2011 · b585752 · b585752
1 parent 9595963
commit b585752
Show file tree

Hide file tree

Showing 21 changed files with 88 additions and 87 deletions.
diff --git a/src/Symfony/Component/Routing/Annotation/Route.php b/src/Symfony/Component/Routing/Annotation/Route.php
@@ -18,11 +18,11 @@
  */
 class Route
 {
-    protected $pattern;
-    protected $name;
-    protected $requirements;
-    protected $options;
-    protected $defaults;
+    private $pattern;
+    private $name;
+    private $requirements;
+    private $options;
+    private $defaults;
 
     /**
      * Constructor.

diff --git a/src/Symfony/Component/Routing/Annotation/Routes.php b/src/Symfony/Component/Routing/Annotation/Routes.php
@@ -18,7 +18,7 @@
  */
 class Routes
 {
-    protected $routes;
+    private $routes;
 
     /**
      * Constructor.

diff --git a/src/Symfony/Component/Routing/CompiledRoute.php b/src/Symfony/Component/Routing/CompiledRoute.php
@@ -18,11 +18,11 @@
  */
 class CompiledRoute
 {
-    protected $route;
-    protected $variables;
-    protected $tokens;
-    protected $staticPrefix;
-    protected $regex;
+    private $route;
+    private $variables;
+    private $tokens;
+    private $staticPrefix;
+    private $regex;
 
     /**
      * Constructor.

diff --git a/src/Symfony/Component/Routing/Generator/Dumper/GeneratorDumper.php b/src/Symfony/Component/Routing/Generator/Dumper/GeneratorDumper.php
@@ -21,7 +21,7 @@
  */
 abstract class GeneratorDumper implements GeneratorDumperInterface
 {
-    protected $routes;
+    private $routes;
 
     /**
      * Constructor.
@@ -32,4 +32,9 @@ public function __construct(RouteCollection $routes)
     {
         $this->routes = $routes;
     }
+
+    public function getRoutes()
+    {
+        return $this->routes;
+    }
 }
diff --git a/src/Symfony/Component/Routing/Generator/Dumper/GeneratorDumperInterface.php b/src/Symfony/Component/Routing/Generator/Dumper/GeneratorDumperInterface.php
@@ -11,6 +11,8 @@
 
 namespace Symfony\Component\Routing\Generator\Dumper;
 
+use Symfony\Component\Routing\RouteCollection;
+
 /**
  * GeneratorDumperInterface is the interface that all generator dumper classes must implement.
  *
@@ -31,4 +33,11 @@ interface GeneratorDumperInterface
      * @return string A PHP class representing the generator class
      */
     function dump(array $options = array());
+
+    /**
+     * Gets the routes to dump.
+     *
+     * @return RouteCollection A RouteCollection instance
+     */
+    function getRoutes();
 }
diff --git a/src/Symfony/Component/Routing/Generator/Dumper/PhpGeneratorDumper.php b/src/Symfony/Component/Routing/Generator/Dumper/PhpGeneratorDumper.php
@@ -47,10 +47,10 @@ public function dump(array $options = array())
         ;
     }
 
-    protected function addGenerator()
+    private function addGenerator()
     {
         $methods = array();
-        foreach ($this->routes->all() as $name => $route) {
+        foreach ($this->getRoutes()->all() as $name => $route) {
             $compiledRoute = $route->compile();
 
             $variables = str_replace("\n", '', var_export($compiledRoute->getVariables(), true));
@@ -64,7 +64,7 @@ protected function addGenerator()
             $escapedName = str_replace('.', '__', $name);
 
             $methods[] = <<<EOF
-    protected function get{$escapedName}RouteInfo()
+    private function get{$escapedName}RouteInfo()
     {
         \$defaults = \$this->defaults;
 $defaultsMerge
@@ -96,10 +96,10 @@ public function generate(\$name, array \$parameters, \$absolute = false)
 EOF;
     }
 
-    protected function startClass($class, $baseClass)
+    private function startClass($class, $baseClass)
     {
         $routes = array();
-        foreach ($this->routes->all() as $name => $route) {
+        foreach ($this->getRoutes()->all() as $name => $route) {
             $routes[] = "       '$name' => true,";
         }
         $routes  = implode("\n", $routes);
@@ -115,15 +115,15 @@ protected function startClass($class, $baseClass)
  */
 class $class extends $baseClass
 {
-    static protected \$declaredRouteNames = array(
+    static private \$declaredRouteNames = array(
 $routes
     );
 
 
 EOF;
     }
 
-    protected function addConstructor()
+    private function addConstructor()
     {
         return <<<EOF
     /**
@@ -138,7 +138,7 @@ public function __construct(array \$context = array(), array \$defaults = array(
 EOF;
     }
 
-    protected function endClass()
+    private function endClass()
     {
         return <<<EOF
 }

diff --git a/src/Symfony/Component/Routing/Generator/UrlGenerator.php b/src/Symfony/Component/Routing/Generator/UrlGenerator.php
@@ -21,10 +21,11 @@
  */
 class UrlGenerator implements UrlGeneratorInterface
 {
-    protected $routes;
     protected $defaults;
     protected $context;
-    protected $cache;
+
+    private $routes;
+    private $cache;
 
     /**
      * Constructor.

diff --git a/src/Symfony/Component/Routing/Loader/AnnotationFileLoader.php b/src/Symfony/Component/Routing/Loader/AnnotationFileLoader.php
@@ -24,7 +24,7 @@
  */
 class AnnotationFileLoader extends FileLoader
 {
-    protected $loader;
+    private $loader;
 
     /**
      * Constructor.
@@ -86,7 +86,7 @@ public function supports($resource, $type = null)
      *
      * @return string|false Full class name if found, false otherwise 
      */
-    protected function findClass($file)
+    private function findClass($file)
     {
         $class = false;
         $namespace = false;

diff --git a/src/Symfony/Component/Routing/Loader/AnnotationGlobLoader.php b/src/Symfony/Component/Routing/Loader/AnnotationGlobLoader.php
@@ -62,7 +62,7 @@ public function supports($resource, $type = null)
      *
      * @return array An array of paths matching the glob pattern
      */
-    protected function getAbsolutePaths($glob)
+    private function getAbsolutePaths($glob)
     {
         $dirs = array();
         foreach ($this->paths as $path) {

diff --git a/src/Symfony/Component/Routing/Loader/XmlFileLoader.php b/src/Symfony/Component/Routing/Loader/XmlFileLoader.php
@@ -167,7 +167,7 @@ protected function validate(\DOMDocument $dom)
      *
      * @return array An array of libxml error strings
      */
-    protected function getXmlErrors()
+    private function getXmlErrors()
     {
         $errors = array();
         foreach (libxml_get_errors() as $error) {

diff --git a/src/Symfony/Component/Routing/Loader/YamlFileLoader.php b/src/Symfony/Component/Routing/Loader/YamlFileLoader.php
@@ -42,7 +42,7 @@ public function load($file, $type = null)
     {
         $path = $this->locator->locate($file);
 
-        $config = $this->loadFile($path);
+        $config = Yaml::load($path);
 
         $collection = new RouteCollection();
         $collection->addResource(new FileResource($path));
@@ -113,18 +113,6 @@ protected function parseRoute(RouteCollection $collection, $name, $config, $file
         $collection->add($name, $route);
     }
 
-    /**
-     * Loads a Yaml file.
-     *
-     * @param string $file A Yaml file path
-     *
-     * @return array
-     */
-    protected function loadFile($file)
-    {
-        return Yaml::load($file);
-    }
-
     /**
      * Normalize route configuration.
      *

diff --git a/src/Symfony/Component/Routing/Matcher/Dumper/ApacheMatcherDumper.php b/src/Symfony/Component/Routing/Matcher/Dumper/ApacheMatcherDumper.php
@@ -43,7 +43,7 @@ public function dump(array $options = array())
         $rules = array("# skip \"real\" requests\nRewriteCond %{REQUEST_FILENAME} -f\nRewriteRule .* - [QSA,L]");
         $methodVars = array();
 
-        foreach ($this->routes->all() as $name => $route) {
+        foreach ($this->getRoutes()->all() as $name => $route) {
             $compiledRoute = $route->compile();
 
             // prepare the apache regex

diff --git a/src/Symfony/Component/Routing/Matcher/Dumper/MatcherDumper.php b/src/Symfony/Component/Routing/Matcher/Dumper/MatcherDumper.php
@@ -21,7 +21,7 @@
  */
 abstract class MatcherDumper implements MatcherDumperInterface
 {
-    protected $routes;
+    private $routes;
 
     /**
      * Constructor.
@@ -32,4 +32,14 @@ public function __construct(RouteCollection $routes)
     {
         $this->routes = $routes;
     }
+
+    /**
+     * Gets the routes to dump.
+     *
+     * @return RouteCollection A RouteCollection instance
+     */
+    public function getRoutes()
+    {
+        return $this->routes;
+    }
 }
diff --git a/src/Symfony/Component/Routing/Matcher/Dumper/MatcherDumperInterface.php b/src/Symfony/Component/Routing/Matcher/Dumper/MatcherDumperInterface.php
@@ -31,4 +31,11 @@ interface MatcherDumperInterface
      * @return string A PHP class representing the matcher class
      */
     function dump(array $options = array());
+
+    /**
+     * Gets the routes to match.
+     *
+     * @return RouteCollection A RouteCollection instance
+     */
+    function getRoutes();
 }
diff --git a/src/Symfony/Component/Routing/Matcher/Dumper/PhpMatcherDumper.php b/src/Symfony/Component/Routing/Matcher/Dumper/PhpMatcherDumper.php
@@ -47,11 +47,11 @@ public function dump(array $options = array())
         ;
     }
 
-    protected function addMatcher()
+    private function addMatcher()
     {
         $code = array();
 
-        foreach ($this->routes->all() as $name => $route) {
+        foreach ($this->getRoutes()->all() as $name => $route) {
             $compiledRoute = $route->compile();
 
             $conditions = array();
@@ -139,7 +139,7 @@ public function match(\$pathinfo)
 EOF;
     }
 
-    protected function startClass($class, $baseClass)
+    private function startClass($class, $baseClass)
     {
         return <<<EOF
 <?php
@@ -159,7 +159,7 @@ class $class extends $baseClass
 EOF;
     }
 
-    protected function addConstructor()
+    private function addConstructor()
     {
         return <<<EOF
     /**
@@ -174,7 +174,7 @@ public function __construct(array \$context = array(), array \$defaults = array(
 EOF;
     }
 
-    protected function endClass()
+    private function endClass()
     {
         return <<<EOF
 }

diff --git a/src/Symfony/Component/Routing/Matcher/UrlMatcher.php b/src/Symfony/Component/Routing/Matcher/UrlMatcher.php
@@ -23,10 +23,11 @@
  */
 class UrlMatcher implements UrlMatcherInterface
 {
-    protected $routes;
     protected $defaults;
     protected $context;
 
+    private $routes;
+
     /**
      * Constructor.
      *

diff --git a/src/Symfony/Component/Routing/Route.php b/src/Symfony/Component/Routing/Route.php
@@ -18,13 +18,13 @@
  */
 class Route
 {
-    protected $pattern;
-    protected $defaults;
-    protected $requirements;
-    protected $options;
-    protected $compiled;
+    private $pattern;
+    private $defaults;
+    private $requirements;
+    private $options;
+    private $compiled;
 
-    static protected $compilers = array();
+    static private $compilers = array();
 
     /**
      * Constructor.
@@ -240,7 +240,7 @@ public function compile()
         return $this->compiled = static::$compilers[$class]->compile($this);
     }
 
-    protected function sanitizeRequirement($key, $regex)
+    private function sanitizeRequirement($key, $regex)
     {
         if (is_array($regex)) {
             throw new \InvalidArgumentException(sprintf('Routing requirements must be a string, array given for "%s"', $key));

diff --git a/src/Symfony/Component/Routing/RouteCollection.php b/src/Symfony/Component/Routing/RouteCollection.php
@@ -20,8 +20,8 @@
  */
 class RouteCollection
 {
-    protected $routes;
-    protected $resources;
+    private $routes;
+    private $resources;
 
     /**
      * Constructor.

diff --git a/src/Symfony/Component/Routing/RouteCompiler.php b/src/Symfony/Component/Routing/RouteCompiler.php
@@ -119,7 +119,7 @@ protected function postCompile()
      *
      * @throws \InvalidArgumentException When route can't be parsed
      */
-    protected function tokenize()
+    private function tokenize()
     {
         $this->tokens = array();
         $buffer = $this->route->getPattern();
@@ -221,7 +221,7 @@ protected function compileForSeparator($separator, $regexSeparator)
     {
     }
 
-    protected function getOptions()
+    private function getOptions()
     {
         $options = $this->route->getOptions();