HELLCORE
Добавлен H,
опубликован
Хотите узнать почему багов на сайте много, а каждый фикс добаляет еще 3 новых бага в совершенно не предсказуемых местах? Все дело в чудо коде, в которым смешаны самые разне патерны программирования от кучи разных людей. Но все же баги тоже бывают забавными и не очень..
ЗАМЕТКА
При добавлении новых фич, релиз которых в версии 8.2 уже скоро состоится, я что-то делал с комментариями, как вдруг выяснилось что нельзя переключить страницу в комментариях. Вместо другой страницы сайт подгружал огромную табличку "Опаньки! Проект не найден". Начал смотреть код и последние изменения этого файла. Алекс там что-то менял, я добавлял новые фичи но в целом вроде все должно быть ок.
При добавлении новых фич, релиз которых в версии 8.2 уже скоро состоится, я что-то делал с комментариями, как вдруг выяснилось что нельзя переключить страницу в комментариях. Вместо другой страницы сайт подгружал огромную табличку "Опаньки! Проект не найден". Начал смотреть код и последние изменения этого файла. Алекс там что-то менял, я добавлял новые фичи но в целом вроде все должно быть ок.
Alexprey:
весело
весело
H:
ну у тебя там проверка в которой ресурс уже существовать должен
$project = get_project($resource['projectid']);
ну у тебя там проверка в которой ресурс уже существовать должен
$project = get_project($resource['projectid']);
ЗАМЕТКА:
у нас id проекта хранится в ресурсах. Т.е ресурс знает к какому проекту он принадлежит. Зная ресурс, можно получить и его проект.
у нас id проекта хранится в ресурсах. Т.е ресурс знает к какому проекту он принадлежит. Зная ресурс, можно получить и его проект.
H:
а ресурса нет еще
а ресурса нет еще
ЗАМЕТКА:
при получение ресурса для полной его инициализации (проверки прав и т.п.) уже должен быть известен проект. В функцию get_resource как раз передается id проекта первым аргументом. А только потом id ресурса.
при получение ресурса для полной его инициализации (проверки прав и т.п.) уже должен быть известен проект. В функцию get_resource как раз передается id проекта первым аргументом. А только потом id ресурса.
ЗАМЕТКА:
$id - индификатор проекта к которому принадлежит ресурс
$id - индификатор проекта к которому принадлежит ресурс
безысходность!
Alexprey:
тлен!
тлен!
ЗАМЕТКА:
в резульате баг был устранен небольшим walkaround'ом..
в резульате баг был устранен небольшим walkaround'ом..
Чтобы оставить комментарий, пожалуйста, войдите на сайт.
сделайте вы лог разработки
чтобы было видно кто,что и где поменял
а то 1 написал одно,потом другой другое и в результате 100500 багов
а всё потому-что второй не знал что именно первый поменял
правда от багов это не спасет
какие файлы трогал
какие строки
и чтобы другие админы это прочитали
полностью баги не уберёт но хоть немного поможет
H, будешь выкладывать веселуху?) Хорошо что этот баг всплыл до релиза
Что же ещё могут готовить два тех администратора? Прямо в тексте ресурса написано.