Yii Framework

GHopper писал(а): ↑2018.07.27, 23:34 1. Хранить историю для каждого slug и обрабатывать все запросы (по уже не используемым slug и по текущим)

andku83 писал(а): ↑2018.07.28, 01:10

GHopper писал(а): ↑2018.07.27, 23:34 1. Хранить историю для каждого slug и обрабатывать все запросы (по уже не используемым slug и по текущим)

А разве для СЕО дубли страниц доступных по разным урлам не хуже отсутствия чпу и slug?

Loveorigami писал(а): ↑2018.07.30, 09:39 Пишите небольшое поведение. При изменении slug-a, заносите старое название, как url, в таблицу Redirects (old_url, post_id).

Loveorigami писал(а): ↑2018.07.30, 09:39 Направляете обработку ошибок, в случае неверного url-a, на свой компонент

Код: Выделить всё

    'components' => [
        'errorHandler' => [
            'class' => modules\seo\components\Redirect::class,
            'errorAction' => 'site/error',
        ],
    ],

yan писал(а): ↑2018.07.30, 13:45 зачем? с точки зрения сео тот слаг которой уже был, не отличается от произвольного неверного слага - на обоих д.б. переадресация на верный слаг

site.ru/first-slug
site.ru/first-slug-for-post1
site.ru/new-slug
site.ru/last-slug

с чего бы вдруг все ошибки на сайте обслуживал компонент отвечающий только за переадресацию одного типа урлов?

Loveorigami писал(а): ↑2018.07.30, 14:00

с первым согласен - не учел, отчасти поэтому не делают подобные слаги, что со временем слаг одной страницы может перекочевать под другую, что совсем не хорошо с точки зрения сео

Так уж получилось, что в проекте нужно использовать SLUG. Причем это поле пользователь должен задавать руками. Отсюда вытекат большая вероятность отшибок/опечаток и желание изменить slug после его создания. Знаю два варианта решения:

по второму это слишком не правильно, чтобы приводить даже для примера, общий обработчик ошибок не должен быть заточен под такую узкую задачу - через него же все ошибки проходят, почему он вдруг только этими редиректами занимается?

class PageController extends Controller
{
    public function actionView($slug)
    {
        $model = PageItem::find()->where(['slug' => $slug])->published()->one();

        if (!$model) {
            throw new NotFoundHttpException(\Yii::t('frontend', 'Page not found'));
        }
    }
}

class Redirect extends ErrorHandler
{
    /**
     * @param \Exception $exception
     */
    public function handleException($exception): void
    {
    
        if ($exception instanceof NotFoundHttpException) {
            /** @var SeoRedirect $redirectModel */
            $redirectModel = SeoRedirect::find()
                ->where(['old_url' => Yii::$app->request->url])
                ->asArray()
                ->one();

            if (!$redirectModel) {
                $redirectStatus = $redirectModel['code'] ?: 301;
                header('Location: ' . $redirectModel['new_url'], true, $redirectStatus);
                exit;
            }
        }

        parent::handleException($exception);
    }

}

yan писал(а): ↑2018.07.29, 22:47 вариант "<post_id>/<slug>" портит структуру (иерархию) урлов

GHopper писал(а): ↑2018.07.30, 17:14

yan писал(а): ↑2018.07.29, 22:47 вариант "<post_id>/<slug>" портит структуру (иерархию) урлов

насколько это критично?

Loveorigami писал(а): ↑2018.07.30, 16:07 У Вас есть другое решение?

3 вариант https://site/<slug>-<post_id> и для устаревших слагов - переадресация с любых неверных слагов на верный

'shop/<category:[\w_-]+>/<slug:[\w_-]+>'=>'shop/product',
'shop/<category:[\w_-]+>'=>'shop/category',
'shop'=>'shop/index',

'news/<category:[\w_-]+>/<slug:[\w_-]+>'=>'news/view',
'news/<category:[\w_-]+>'=>'news/category',
'news'=>'news/index',

'<slug:[\w_-]+>'=>'page/view',

Loveorigami писал(а): ↑2018.07.30, 20:43 С не "непроизвольными", а с теми, у которых изменился slug и страница стала недоступна по старому slug-у.

Loveorigami писал(а): ↑2018.07.30, 20:43 Как будет выглядеть Ваш Url-manager?
А если у меня пару десятков модулей?

На каждый модуль писать свое правило? Делать запрос на существование слуга. Если нет - тогда запрос в редирект. Если есть - тогда перенаправляем на нужный action, и там же еще раз делаем этот же запрос.

Т.е. на любой существующий url нужно делать 2 запроса и писать свои классы правил?

Loveorigami писал(а): ↑2018.07.30, 20:54 А, кстати, как из правила делать редирект?

'shop/<category:[\w_-]+>/<slug:[\w_-]+>'=>'shop/product',

На каждый модуль писать свое правило? Делать запрос на существование слуга. Если нет - тогда запрос в редирект. Если есть - тогда перенаправляем на нужный action, и там же еще раз делаем этот же запрос.

а как в вашем варианте будут обрабатываться актуальные, не устаревшие слаги?

Loveorigami писал(а): ↑2018.07.30, 21:13 3. Я в контролере с продуктом словил NotFoundHttpException и в ErrorHandler-е сделал 1 запрос и редирект. Все.

Loveorigami писал(а): ↑2018.07.30, 21:22

а как в вашем варианте будут обрабатываться актуальные, не устаревшие слаги?

У меня нет понятия устаревший или нет...

В который раз пишу одно и то же...
1. Зашел по урлу (slug - как часть урла).
2. UrlManger перенаправил на нужный модуль/контроллер/экшн.
3. Если модель найдена - по вашему - урл актуальный. 200
4. Если нет - NotFoundHttpException. Ищу запись в редиректе.
5. Если есть - значит урл, устаревший. 301
6. Если нет - урл не существует. 404