CRUD YII2 проблема при удалении записи из таблицы?

Question

[OlegBas1996]

Здравствуйте! Создаю админку , сгенировал CRUD через gii для таблицы users, прописал маршруты в файле web.php. При удалении записи из таблицы users осуществляет редирект на несуществующую страницу. Что интересно, для страниц добавления и обновления записи я не прописывал маршруты,и они работают. Помогите решить проблему.

Comments

[Дмитрий]

Что за идиотская привычка у многих вставлять код картинкой?!!

[Дмитрий]

Так же, как в и предыдущем Вашем вопросе, показывайте модель, контроллер, ссылку на удаление.
Последнее правило работать не будет. Да и остальные правила что-то странные какие-то...

[OlegBas1996]

Контроллер https://github.com/OlegBas1996/news-site.ru/blob/m...
Модель https://github.com/OlegBas1996/news-site.ru/blob/m...
Вид, где генерируется ссылка https://github.com/OlegBas1996/news-site.ru/blob/m...

[dtBlack]

Уточни какую конкретно тебе ошибку выдает, и дай ссылку куда тебя редиректит?

[OlegBas1996]

dtBlack, редирект идёт на страницу 404

[dtBlack]

OlegBas1996, а у страницы 404 в адресной строке нет url случаем? Как можно, что то подсказать, если информации нет от слова совсем? Еще и в тексте упомянута таблица users, а приводятся ссылки для news?

[Дмитрий]

OlegBas1996, код в обязательном порядке надо добавить в вопрос!!!
Потрудитесь отредактировать!

[Дмитрий]

OlegBas1996, когда добавите необходимый код в вопрос, вот тогда можно будет продолжать. С модулем admin у Вас тоже не всё хорошо, так же как и с правилами urlManager

[OlegBas1996]

slo_nik, Хорошо, добавлю.

[OlegBas1996]

dtBlack, Ошибка 400, редирект на news-site.ru/admin/section/delete?id=9

[OlegBas1996]

slo_nik, мне нужно сделать управление пользователями в админке. Для этого я создал модуль admin. Далее через gii, используя генератор CRUD я сгенерировал множество файлов: контроллер UserController, модели UserSearch и Users, директорию видов user.
Cссылка на удалении выглядит на скриншоте как пиктограмма корзины, генерируется через класс ['class' => 'yii\grid\ActionColumn'], прописанный внутри виджета GridView, через который выводится таблицы с пользователями.
При нажатии на кнопку удаления должен отработать метод

public function actionDelete($id)
    {
       $this->findModel($id)->delete();

        return $this->redirect(['index']);
    }

Но в него интерпретатор даже не заходит, следовательно исключение о ошибке 400 выбрасывается раньше, причем url в браузере после отображения 400 ошибке не изменяется news-site.ru/admin/user/delete?id=4.
Маршруты, прописанный в web.php определяют работу главных страницы админки

'rules' => [ // Описываем шаблоны url и соответсвующие им контроллеры и действия
            // Массив маршрутов
                '/' => 'site/index', 
/*               "admin/tizer/delete/<id:\d+>" => "admin/tizer/delete",
*/                'article/short/<id:\d+>' => 'site/article/',
                'article/full/<id:\d+>' => 'site/fullarticle/',
                'addClick/<idClickBlock:\d+>' => 'site/addclick',
                'admin' => 'admin/default',
                'admin/news' => 'admin/news',
                'admin/tizers' => 'admin/tizer',
                'admin/sections' => 'admin/section',
                'admin/users' => 'admin/user',
            ],

Обратите внимание:
Для страниц добавления(news-site.ru/admin/user/create) и изменения(news-site.ru/admin/user/update?id=3) я не прописывал маршруты,хотя они открываются

Модель Users представлена ниже

<?php

namespace app\modules\admin\models;

use Yii;

/**
 * This is the model class for table "users".
 *
 * @property string $id
 * @property string $username
 * @property string $email
 * @property string $password
 * @property integer $created_at
 */
class Users extends \yii\db\ActiveRecord
{
    /**
     * @inheritdoc
     */
    public static function tableName()
    {
        return 'users';
    }

    /**
     * @inheritdoc
     */
    public function rules()
    {
        return [
            [['username', 'email', 'password', 'created_at'], 'required'],
            [['created_at'], 'integer'],
            ['email','email'],
            [['username', 'password'], 'string', 'max' => 50],
            [['email'], 'string', 'max' => 50],
        ];
    }

    /**
     * @inheritdoc
     */
    public function attributeLabels()
    {
        return [
            'id' => 'ID',
            'username' => 'Имя пользователя',
            'email' => 'Email',
            'password' => 'Пароль',
            'created_at' => 'Дата регистрации',
        ];
    }
}

[Дмитрий]

Почитайте здесь, как правильно сделать модуль и как настроить правила urlManger для модуля.
Ну и остальные статьи читайте, много полезного для себя найдёте.

[OlegBas1996]

slo_nik, ну а по дефолту почему. правила не работают? Я уже создавал пару проектов, где тоже не прописывал правила для страниц удаления, добавления, редактирования и все прекрасно работало.

[Дмитрий]

OlegBas1996, потому, что не работают. Для модуля надо свои правила писать.

[Дмитрий]

OlegBas1996, например правило
'<_c:\w+>/<index>' => '<_c>/<index>
Ищет какой-то контроллер, а в нём какое-то действие, будет работать для "site/index", "profile/index", "articles/index", где "site", "profile" и "articles" являются контроллерами.
Теперь Ваш вариант, с модулем админ
'<_m:\w+>/<_c:\w+>/<_a:\w+>' => '<_m>/<_c>/<_a>
В этом случае сначала будет искаться модуль с определённым названием, потом контроллер этого модуля и уж потом действие в контроллере модуля.
Читайте статью по ссылке, которую я дал. Для модуля admin там создаются отдельные правила, которые прекрасно работают.
Так же имейте ввиду, что порядок следования правил в конфигурации urlManager тоже имеет значение.
На том же сайте найдёте видео о маршрутизации в yii2.

[OlegBas1996]

slo_nik, я определил в классе UserController метод beforeAction(), чтобы понять на каком этапе выбрасывается исключение о 400 ошибке

public function beforeAction($action)
        {
             print_r($action);
            return false;
        }

Функция beforeAction отработала, значит, ошибка выбрасывается после вызова этого метода, причем если комментировать метод

/*public function actionDelete($id)
    {
        $this->findModel($id)->delete();
        return $this->redirect(['index']);
    }*/

То ошибка уже будет 404, т.к действия нужно не найдено.
Если следовать Вашему совету, то при отсутствии маршрута в web.php каким образом фреймворк тогда запускает объект UserController и ищет в нем метод actionDelete?

[OlegBas1996]

slo_nik, прописал это правило в самый конец
'<_m:\w+>/<_c:\w+>/<_a:\w+>' => '<_m>/<_c>/<_a>
и все работает
Я не понял почему, в одном случае я ничего не прописывал и все работало, а данном нужно было прописать?

[Дмитрий]

OlegBas1996, это не мой совет, это написано в документации. Читайте внимательно статью в блоге.
Так же есть такая статья(начинать читать с самого начала).
Ещё есть видео.

каким образом фреймворк тогда запускает объект UserController и ищет в нем метод actionDelete

Отработает маршруты по умолчанию. Но если такого маршрута нет, то надо писать правило.
В Вашем случае для модуля "admin" надо писать правила, так просто они не отработают.

[Дмитрий]

Я не понял почему, в одном случае я ничего не прописывал и все работало, а данном нужно было прописать?

OlegBas1996, в каком случае?

[OlegBas1996]

slo_nik, еще я добавил данный метод

public function beforeAction($action)
        {
            return true;
        }

Без него также не запускается метод actionDelete

Теперь скажите, у меня все заработало без написания маршрутов? Каким образом? Или это баги во фреймворки?

[Дмитрий]

OlegBas1996, зачем Вам метод beforeAction()? Всё должно работать и работает без него. Просто изучите материал по ссылкам.

Или это баги во фреймворки?

Нет, это Ваше незнание как писать правила в urlManager.
Читайте документацию.
Ну и покажите, что Вы наворотили в правилах urlManager()?

[OlegBas1996]

slo_nik, я думал если работает, значит правильно пишу
Я методом эксперимента добавил метод beforeAction(), чтобы понять где происходит выброс ошибки
Вот мои правила

'urlManager' => [
            'enablePrettyUrl' => true, // Включаем компонент
            'showScriptName' => false,// Исключаем точку входа (index.php) из ссылок
            'rules' => [ // Описываем шаблоны url и соответсвующие им контроллеры и действия
            // Массив маршрутов
                '/' => 'site/index', 
/*               "admin/tizer/delete/<id:\d+>" => "admin/tizer/delete",
*/                'article/short/<id:\d+>' => 'site/article/',
                'article/full/<id:\d+>' => 'site/fullarticle/',
                'addClick/<idClickBlock:\d+>' => 'site/addclick',
                'admin' => 'admin/default',
                'admin/news' => 'admin/news',
                'admin/tizers' => 'admin/tizer',
                'admin/sections' => 'admin/section',
                'admin/users' => 'admin/user',
                '<_m:\w+>/<_c:\w+>/<_a:\w+>' => '<_m>/<_c>/<_a>'
            ],
        ],

Спасибо Вам за совет! Буду читать

[Дмитрий]

я думал если работает, значит правильно пишу

OlegBas1996, не всегда это так.
Вы написали жёсткие правила, а если Вы добавите ещё один модуль или ещё один контроллер, то что, будете продлевать "портянку" правил? Например Вам надо будет добавить контроллер CommentsController, получается, что для этого контроллера Вам надо будет прописать ещё ряд правил.
На какое действие ведут эти правила?

'article/short/<id:\d+>' => 'site/article/',
 'article/full/<id:\d+>' => 'site/fullarticle/',

А эти?

'admin/news' => 'admin/news',
'admin/tizers' => 'admin/tizer',

Ведь в контроллерах news и tizer у Вас не только действие index, а есть ещё другие правила.
Так что Ваши правила будут работать неоднозначно, если можно так выразится и сбои будут происходить постоянно.

Answer 1

[Максим Тимофеев]

Вы делаете return $this->redirect(['index']);
при этом правила url manager

'/' => 'site/index', 
                'article/short/<id:\d+>' => 'site/article/',
                'article/full/<id:\d+>' => 'site/fullarticle/',
                'addClick/<idClickBlock:\d+>' => 'site/addclick',
                'admin' => 'admin/default/',
                'admin/news' => 'admin/news/',
                'admin/tizers' => 'admin/tizer/',
                'admin/sections' => 'admin/section/',
                'admin/users' => 'admin/user/',

Какое правило по Вашему должно сработать, что бы отработал роут ['index']
При вашем конфиге, я так понимаю редирект должен выглядеть так:

public function actionDelete($id)
    {
        $this->findModel($id)->delete();
        return $this->redirect(['admin/news/]);
    }

Но возможно у Вас 404 еще до этого, еще на этапе поиска экшена удаления

Comments

[OlegBas1996]

Максим, дело не в
return $this->redirect(['index']);
Я пробовал комментировать код внутри функции удаления, но несмотря на это, данная функция даже не вызывается, т.к исключение о 400 ошибке выбрасывается раньше
Ниже Вы можете увидеть панель информацию из отладчика

[Максим Тимофеев]

OlegBas1996, я вам так и написал

Но возможно у Вас 404 еще до этого, еще на этапе поиска экшена удаления

посмотрите свой конфиг и поймите, что ни одно правило не приведет запрос с роутом admin/section/delete туда куда Вы хотите

[OlegBas1996]

Максим Тимофеев, а как работают страницы news-site.ru/admin/user/update?id=3,http://news-si...
Для них также не прописаны правила
Я так понимаю при там встроенная маршрутизация

Answer 2

[Евгений Бухарев]

'enableStrictParsing' => true, // чтобы "я не прописывал маршруты,и они работают."

Как вариант добавить базовые маршруты в самый конец rules, а более кастомные маршруты прописывать выше

// Base
'<action>' => 'site/<action>',
'<controller>/<action>' => '<controller>/<action>',
'<module>/<controller>/<action>/<id:\w+>' => '<module>/<controller>/<action>',
'<module>/<controller>' => '<module>/<controller>/index',
'<module>/<controller>/<action>' => '<module>/<controller>/<action>',

Comments

[OlegBas1996]

Тем не менее, не работает!!!

[Евгений Бухарев]

OlegBas1996, Каким образом генерируете урл? до контроллера запрос доходит? как выглядит экшн?