Как опять победить URLmanager?

Question

[Сергей Беловенцев]

возник такой вопрос вот правила

'serials/<slug:\w*>' =>  'serial/category/list',
    'serials/<id>'   =>  'serial/category/oneserial',

пишу такие ссылки

Yii::$app->urlManager->createUrl(['/serial/category/oneserial','id'=>$model->slug_serial]) //ссылка формируется такая /serials/serial-nazvanie-seriala
     Yii::$app->urlManager->createUrl(['/serial/category/list','slug'=>$ct->slug_category])//ссылка формируется такая /serials/сategory

но если добавить тире к примеру /serials/nazvanie-сategory то в адресной строке получаю serials?slug=nazvanie-category пытался решить так

'serials/<slug:\w*>' =>  'serial/category/list',
    'serials/<slug:[\w_\-]+>' =>  'serial/category/list',
    'serials/<id>'   =>  'serial/category/oneserial',

теперь вот /serials/nazvanie-сategory это получается хорошо но вот этот адрес /serials/serial-nazvanie-seriala теперь использует тоже второе правило парсит на этот адрес serial/category/list что выбрасывает на главную страницу и если честно 1 я не понимаю почему и 2 как это исправить.

Comments

[Никита]

Вы осознанно на один и тот же путь вешаете разные типы страниц? serials - serials и там и там

[Сергей Беловенцев]

Никита: Сео-оптимизатор требует

[Никита]

Сергей Беловенцев: интересно, но это ведь неверно с точки зрения оптимизации для поисковиков.

[Сергей Беловенцев]

Я честно не сео оптимизатор так, что вопрос не совсем ко мне.

Answer 1

[polar-bear]

Вы пытаетесь открыть и категорию и сам сериал, в одном префиксе 'serials'. При создании url проблем нет, но при парсинге yii берет первое подходящее правило.
Создайте лучше класс правил url Creating Rule Classes , и ищите по БД, есть сериал пробрасываете на сериал, если нет проверяете категорию и если она существует на категорию.

Comments

[Сергей Беловенцев]

А есть ссылки на применение этих правил ? Кроме как в документации ?

[polar-bear]

А чем плох пример из документации? Вроде все понятно расписано, реализуем два метода один для создания ссылок, второй для парсинга.
При создании смотрим роут (тут нам БД не нужна), если serial/category/list и существует ключ slug в массиве $params, создаем и возвращаем ссылку serials/nazvanie-сategory, аналогично и для самого сериала.
При парсинге, хватит регулярки что в доках, в массив $matches попадут части url serials и название категории или сериала. Дальше проверка по БД и возращаем нужный маршрут, если найден иначе false.

еще пример blog.neattutorials.com/yii2-routing-urlmanager

[Сергей Беловенцев]

Вот именно а где можно посмотреть вот эту часть "Дальше проверка по БД и возращаем нужный маршрут, если найден иначе false."

[Сергей Беловенцев]

как найти "если serial/category/list и существует ключ slug" опять же не понятно.

[Сергей Беловенцев]

И зачем мне нужно извлекать данные из базы когда у меня все же статические страницы, а передается значение в первом случае через гет с ключом slug в другом id про url я не говорю? Просто все это не объяснено вообще.

[polar-bear]

"статические страницы" - у вас что сериалы и категории не в базе данных хранятся? А в чем разница между slug и id, если в вашем вопросе они могут содержать любые символы? Slug у вас и является тем идентификатором по которому вы ищете сериал или категорию. Вы понимаете что у вас 'serials/' ведет и на категорию и на сериал, этот шаблон ОДИНАКОВ для сериала и для категории. А по БД значит пишем запрос например $exists = Serial::find()->where(['slug' => $slug])->exists(); Если true возвращаем ['serial/category/oneserial'], а иначе например ['serial/category/list'] (либо аналогично проверяем на существование категории).

[Сергей Беловенцев]

Вы извините если все таки на доедаю просто хочу реально понять как это работает есть этот метод

public function parseRequest($manager, $request)
    {
        $pathInfo = $request->getPathInfo();
        if (preg_match('%^(\w+)(/(\w+))?$%', $pathInfo, $matches)) {

как мне узнать в нем $slug в вашем примере? Можно ли как то получить сам путь и к примеру делать выборку к примеру

if($path=='serial/category/oneserial'){
     $exists = Serial::find()->where(['slug' => $id])->exists()
        if($exists){
       return  ['serial/category/oneserial',params['id]];
       }
}
if($path=='serial/category/list'){
     $exists = Category::find()->where(['slug' => $slug])->exists()
        if($exists){
       return  ['serial/category/list',params['slug]];
       }
}

[polar-bear]

Метод parseRequest:

preg_match('%^([\w-_]+)(/([\w-_]+))?$%', $pathInfo, $matches);

в массиве $matches будет нужный slug, а вот к сериалу он или категории нужно узнать через БД. А в переменной $pathInfo запрашиваемый путь.

В методе parseRequest не нужно проверять по БД, можно даже как-то так:

if (($route === 'serial/category/oneserial' || $route === 'serial/category/list') && isset($params['slug'])) {
    $slug = $params['slug'];
    unset($params['slug']);
    return 'serials/' . http_build_query($params);
}
return false;

[Сергей Беловенцев]

просто если бы понять что находится вот здесь $matches ?

[polar-bear]

return 'serials/' . $slug . '?' . http_build_query($params);

[polar-bear]

Сергей Беловенцев: массив, под 3 ключом должен быть slug

[Сергей Беловенцев]

polar-bear: Извините но вы меня совсем запутали откуда $route, $params['slug']; берется в методе parseRequest ? "в массиве $matches будет нужный slug" у меня или slug или id вот я попробоывал реализовать так

public function createUrl($manager, $route, $params)
    {
        if ($route === 'serial/category/oneserial') {
            if (isset($params['id'])) {
                return 'serials/'.$params['id'];
            }
        }elseif($route === 'serial/category/list'){
            if (isset($params['slug'])) {
                return 'serials/'.$params['slug'];
        }
    }
        return false;  // данное правило не применимо
    }

    public function parseRequest($manager, $request)
    {
        $pathInfo = $request->getPathInfo();
        if (preg_match('%^(\w+)(/(\w+))?$%', $pathInfo, $matches)) {
            $path=$matches[0];
            if($path=='serial/category/oneserial'){
                $exists = Serial::find()->where(['slug_serial' => $matches[1]])->exists();
                if($exists){
                    return  ['serial/category/oneserial','id'=>$matches[1]];
               }
            }
            if($path=='serial/category/list'){
                $exists = Category::find()->where(['slug_category' => $matches[1]])->exists();
                if($exists){
                    return  ['serial/category/list','slug'=>$matches[1]];
               }
            }
        }
        return false;  // данное правило не применимо
    }

теперь получаю так
<?= Yii::$app->urlManager->createUrl(['/serial/category/oneserial','id'=>$model->slug_serial])?> парсится такая ссылка serials/serial-Perevozcik но при этом ошибка 404
Yii::$app->urlManager->createUrl(['/serial/category/list','slug'=>$ct->slug_category]) парсит такую ссылку
film.lc/serials?slug=boevik и на ходит страницу но это не комельфо как понимаете

[Сергей Беловенцев]

polar-bear: это теперь делает тоже самое что и первое правилопарсится такая ссылка serials/boevik но при этом ошибка 404

[polar-bear]

Извините но вы меня совсем запутали откуда $route, $params['slug']; берется в методе parseRequest ? в CreateUrl, ошибся.

<?= Yii::$app->urlManager->createUrl(['/serial/category/oneserial','id'=>$model->slug_serial])?> парсится такая ссылка serials/serial-Perevozcik но при этом ошибка 404
В регулярке не разрешены тире. Попробуйте эту '%^([\w-_]+)(/([\w-_]+))?$%'

Yii::$app->urlManager->createUrl(['/serial/category/list','slug'=>$ct->slug_category]) парсит такую ссылку
film.lc/serials?slug=boevik и на ходит страницу но это не комельфо как понимаете
Попробуйте в createUrl сделать $route = trim($route, '/');
А вы удалили роуты которые упомянули в вопросе?

parseRequest
'serial/category/list' - это маршрут его нужно определить, и его нет в $matches[0]

[polar-bear]

public function parseRequest($manager, $request)
{
    $pathInfo = $request->getPathInfo();
    if (preg_match('%^([\w-_]+)(/([\w-_]+))?$%', $pathInfo, $matches)) {
        if ($matches[1] === 'serials') {
            $exists = Serial::find()->where(['slug_serial' => $matches[3]])->exists();
            if($exists){
                return ['serial/category/oneserial', 'id'=>$matches[3]];
            } else {
                $exists = Category::find()->where(['slug_category' => $matches[3]])->exists();
                if($exists){
                    return  ['serial/category/list', 'slug'=>$matches[3]];
                }
            }
        }
    }
    return false;
}