Hi – in the end I have left my routes as they were originally and I have extended the paginator helper.

In the extended paginator helper I’ve over-ridden the link() method and am building the url directly using the data that was passed to the view in $this->passedArgs as a basis.

Thanks for your ideas.

It’s just an idea, I don’t know whether that’s practical in your specific situation…

I think I may just leave it with my original routes solution and just write a custom pagination element for the single instance where the problem occurs.

Perhaps this is just one of those small annoyances I’ll just have to live with, sometimes workarounds are where its at.

John

Maybe it is worth an enhancement ticket?

Hope that helps!

I’ve been using the variant below (first posted here by panther40k I think) (different controller / action obviously)
$Route->connect(’(?!admin|function)(.*)’, array(’controller’ => ‘nodes’, ‘action’ => ‘view’));
for ages in 1.2 without any problems… until now. The problem is a bit long winded and is described on groups here: http://groups.google.com/group/cake-php/browse_thread/thread/9e295aefeeeb3eb8?hl=en

Anyway what I am trying to do is rewrite the above route in a proper Cake 1.2 way e.g.

Router::connect(
'/*',
array(
'controller' => 'nodes',
'action' => 'view',

What I am trying to do is have * parsed as a named parameter. Any ideas anyone?

Thanks for the snippet, its very elegant! The only problem for me was the fact that I made heavy use of the theme-option in cakephp. 3 more lines and thats working also though:

<?php 

class PagesController extends AppController
{

    public $uses = array();

    public function display()
    {

        $view_filename = array_pop($this->params['pass']) . '.ctp';
        $view_filename = str_replace('-', '_', $view_filename);
        $view_path =  DS . $view_filename;

        if ($this->params['pass'])
        {

            $view_path = DS . implode(DS, $this->params['pass']) . $view_path;

        }

        if (isset($this->theme) && file_exists(VIEWS . 'themed' . DS . $this->theme . DS . 'pages' . $view_path))
        {

            $this->render(null, null, VIEWS . 'themed' . DS . $this->theme . DS . 'pages' . $view_path);

        }
        elseif (file_exists(VIEWS . 'pages' . $view_path))
        {

            $this->render(null, null, VIEWS . 'pages' . $view_path);

        }
        else
        {

            $this->cakeError('error404');

        }

    }

}

?>

Greetings!

Which CakePHP version do you use? I just copied your route and your $html->link() statement into my test application (which uses the latest cake version from the repository), and there the reverse routing worked fine, i.e. the generated link points to “/contact”. Is it possible that you added your route after the default pages route (Router::connect(‘/pages/*’, array(‘controller’ => ‘pages’, ‘action’ => ‘display’));)?

Router::connect(‘/contact’, array(‘controller’ => ‘pages’, ‘action’ => ‘display’, ‘contact’));

… and I create a link like this:

$html->link(
‘Contact’,
array(‘controller’ => ‘pages’, ‘action’ => ‘display’, ‘contact’),
array(‘title’ => ‘How to contact us’)
)

.. I get a link like “/pages/contact”. If I can use custom routes for other controllers, it should be possible here, also. Very strange.

With your technique, using reverse routing is again out of the question, but it’s cleaner than what I’ve been using, so I think I’ll upgrade.

I made a change to display() in order to have deeper levels without having to specify “index’ for the default page.

eg. ‘/about’ or ‘/about/’ will display ‘views/pages/about/index.ctp’

public function display()
{
$viewFilename = str_replace(‘-’, ‘_’, array_pop($this->params['pass']));

$viewPath = DS . $viewFilename;
if ($this->params['pass'])
{
$viewPath = DS . implode(DS, $this->params['pass']) . $viewPath;
}

if (file_exists(VIEWS.’pages’.$viewPath.’.ctp’))
{
$this->render(null, null, VIEWS.’pages’.$viewPath . ‘.ctp’);
}
else if (file_exists(VIEWS.’pages’.$viewPath.DS.’index.ctp’))
{
$this->render(null, null, VIEWS.’pages’.$viewPath.DS.’index.ctp’);
}
else
{
$this->cakeError(‘error404′);
}
}

I removed the ‘.ctp’ append to $viewPath so that it isn’t in the way in the “else if” test.

Well, the DS is required so that $viewPath always starts with a DS (which is needed for the file_exists check I used in the article). But if you remove the “pages” from the file_exists call, then you are right, and the DS is not needed.