В базе есть колонка с номером года?

нет, к сожалению, там только cdate_int где просто хранится время в timestamp

В базе есть колонка с номером года?

если создать такую колонку и использовать её для удаления, то это вероятно решит эту проблему?

Как раз не надо такую колонку делать

Хорошо)

есть отдельные колонки для дня и месяца, но они повсюду используются и очень нужны

Просто в метод передашь примерно так 2018 => 2018

Первый параметр зачем там нужен? У него ключом является не колонка, а ее значение

Первый параметр зачем там нужен? У него ключом является не колонка, а ее значение

я потерял нить разговора) первый параметр где? В mktime?

"FROM_UNIXTIME(cdate_int, '%Y')" => date("Y", $newsM->cdate_int)

Эта часть

Странная конструкция как по мне)

Но по идее она просто года сравнивает, и если они не совпадают, то строки не затрагиваются

Но по идее она просто года сравнивает, и если они не совпадают, то строки не затрагиваются

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

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

Просто вторая часть делает то же самое) день месяц и год берет или я неправильно ее понял

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

Ну по идее этот запрос и удаляет все, что добавили перед этим. $newsM это что? Откуда берется?

Ну по идее этот запрос и удаляет все, что добавили перед этим. $newsM это что? Откуда берется?

$newsM = News::findOne(["id" => $id]); это запись из другой таблицы, ищет её по id

берет её время и удаляет запись)

А почему из другой таблицы? Интересно как

А дату можно без mktime делать, например, date('Y-m-d 00:00:00', time())

А почему из другой таблицы? Интересно как

Так может в $model попадает другая модель, а не та, которую ожидаешь

Сейчас пофикшу, а то ерунда какая то

Сейчас пофикшу, а то ерунда какая то

И в запросе лучше не вычислять дату, заранее ее в переменную запиши, MySQL кеширует запросы

А дату можно без mktime делать, например, date('Y-m-d 00:00:00', time())

не, нужна дата из базы, точная до секунд) а date мне такой точности не дает вроде)

не, нужна дата из базы, точная до секунд) а date мне такой точности не дает вроде)

Так я вон пример написал, как получить ровно полночь

Зачем хранить даты в разных форматах чтобы их потом конвертировать)

Храни в timestamp тогда between можно делать проще или указать дату без часов минут и секунд

Храни в timestamp тогда between можно делать проще или указать дату без часов минут и секунд

так там и так timestamp, где я в mktime делаю

Я про cdate_int

Сохраняю просто: $model->validate(); $model->save();

save сам вызывает валидацию. Естественно, если аргумент отмены валидации не передан в save

Где вы работаете, что так серьезно все поставлено?

Дочерняя кампания Альфа Банка.

Я про cdate_int

https://implode.io/zjznHG переписал последнюю функцию, не могли бы Вы подсказать, есть какие то очевидные замечания там? Мб упустил что то

save сам вызывает валидацию. Естественно, если аргумент отмены валидации не передан в save

Исправил тоже, кстати) спасибо

https://implode.io/zjznHG переписал последнюю функцию, не могли бы Вы подсказать, есть какие то очевидные замечания там? Мб упустил что то

А если какая то сущность не будет найдена, что будет?

А если какая то сущность не будет найдена, что будет?

Вы о какой части кода говорите?

там просто несколько условий стоит)

$newsM = News::findOne(["id" => $id]); Например

а, пустоту передать нельзя, на клиенте проверка стоит, если не выбрано ничего, то кнопку "сохранить" нельзя нажать)

А если 2 человека работает, один удалил, второй сохранил)

Все передалось, но записи уже нет)

Все передалось, но записи уже нет)

Чтобы бы Вы посоветовали в таком случае предпринять?

Чтобы бы Вы посоветовали в таком случае предпринять?

проверять, нашлась ли запись, и если нашлась, только тогда производить какие то действия. Иначе будет ошибка

вот эта часть тоже вызывает вопросы public function editItems() { $editNews = Yii::$app->request->post('ThisDay'); foreach ($editNews["news_name"] as $year => $name) { $result[] = $name; } return $result; } что будет, если $editNews не массив или в нем нет элемента с ключом news_name

проверять, нашлась ли запись, и если нашлась, только тогда производить какие то действия. Иначе будет ошибка

Хорошая идея, спасибо

всегда надо проверять данные перед использованием. Массив ли? если должно прийти число, сделай intval вместо $model = ThisDay::findOne(['id_news' => $id]); надо $id = intval($id); $model = ThisDay::findOne(['id_news' => $id]);

и в модели наверное не надо работать с реквестом. получай его в контроллере и передавай в модель. я про метод public function editItems()

вместо public function editItems() { $editNews = Yii::$app->request->post('ThisDay'); foreach ($editNews["news_name"] as $year => $name) { $result[] = $name; } return $result; } лучше на мой взгляд будет public function editItems($editNews) { $result = []; if(is_array($editNews) && isset($editNews["news_name"])) { foreach ($editNews["news_name"] as $name) { $result[] = $name; } } return $result; }

соответственно, проверять в методе validateEdit() пришел ли пустой массив или нет

вот эта часть тоже вызывает вопросы public function editItems() { $editNews = Yii::$app->request->post('ThisDay'); foreach ($editNews["news_name"] as $year => $name) { $result[] = $name; } return $result; } что будет, если $editNews не массив или в нем нет элемента с ключом news_name

дело в том, что тут данные с представления "edit" в него нельзя попасть, если ты невыбрал хотя бы одну новость, соответсвенно, $editNews всегда массив, хотя бы с одной новостью и указанным ключом)

дело в том, что тут данные с представления "edit" в него нельзя попасть, если ты невыбрал хотя бы одну новость, соответсвенно, $editNews всегда массив, хотя бы с одной новостью и указанным ключом)

а если юзер изменил форму на своей стороне, чтобы вызвать ошибку?

жуликов хватает, надо все перепроверять и юзерам не доверять)

жуликов хватает, надо все перепроверять и юзерам не доверять)

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

Объясните пожалуйста что я сделал не так что у меня выводится такая ошибка? - PHP User Error – yii\base\ErrorException EditController module does not exist

Объясните пожалуйста что я сделал не так что у меня выводится такая ошибка? - PHP User Error – yii\base\ErrorException EditController module does not exist

очевидно, что модуль EditController не существует

в тексте ошибки же написано. ищи где его подключал или где надо подключить, и подключи

Ну так я его не создавал никогда и сам его нигде не использую

in D:\OpenServer\OSPanel\domains\Yii2\vendor\ezyang\htmlpurifier\library\HTMLPurifier\HTMLModuleManager.php

Я его не подключал

Я его не подключал

глянь полный стек ошибок

Это как?

Это как?

ты где эту ошибку получаешь?

когда нажимаю Time

Memory

у меня просто один контроллер грузится непростительно долго

когда нажимаю Time

Очень понятно))

Я пытаюсь выяснить причину

162 милисекунды это совсем не долго

тыркнул на значения 3 сек

покажи код profile/update

Это не он грузится долго

Это не он грузится долго

ошибка вызвана на этом роуте, значит начинать надо с него

покажи весь скрин с ошибками, ниже прокрути

https://pastebin.com/xDAjLzuz

Это код ProfileController

Так

$response = RestUpdate::restUpdate(Yii::$app->user->identity->email, Yii::$app->user->identity->address, Yii::$app->user->identity->apartment,Yii::$app->user->identity->lastName,Yii::$app->user->identity->firstName,Yii::$app->user->identity->thirdName, Yii::$app->user->identity->phone_number, Yii::$app->user->identity->login, $user->password, $user->individual); лютая дичь какая то. не проще в нем самом получать Yii::$app и раскидывать параметры? Надо будет потом поменять порядок, например, или добавить что-то, придется во всех методах менять

Это геморой