Branch Yadzuka - список проблем, препятствующих запуску в эксплуатацию 2020-02-10

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

При сохранении, запись (версия) в файл пишется дважды. Чорт! это еще и плавающая ошибка… Кстати, если запись не изменилась - не надо ее записывать вообще
~~Не контролируются \n\r при записи в файл~~
также проверить контроль “;” и “:”, готов поспорить на деньги, что не контролируется. Точно - проверил, не контролируется, да еще и новая пустая запись в файл влетела, которую теперь не могу даже отредактировать
~~Пустые записи вообще нельзя отредактировать! и сообщений об ошибке нет!~~
~~При создании новой записи пропала функция получения нового QR-кода - вернуть!~~
~~QR-код посинел от холода? отогреть!~~
вообще, надо при сохранении прикрутить контроль качества, например для QR - если он не соответствует диапазону, для PRODTYPE/MODEL - если заполнено второе, заполните первое и т.п. Все это не критично, при этом надо нормально сохранить, но надо предусмотреть критичные ошибки, при которых будет отказ в сохранении. Перевод ошибки из некритичной в критичную - должен быть тривиальным, и наверное описываться в csv.tab, но это уже совсем другая задча будет
~~При редактировании потеряли описание правил заполнения полей~~
Поставил права “только на чтение” - получил Permission denied при чтении старая версия нормально читает - это правильно, а вот предупредить при редактировании, если прав нет - стоит - проблема в том, что код пытается открыть файл с доступом первого уровня (чтение + запись), а ОС этого не даёт - с того и ошибка.
~~zrid в запросе != zrid в файле, это неправильно, нужно в уме делать zrid+-1,~~
кроме того надо сделать zrid → ZRID (s/zrid/ZRID/g 10000)
~~Надо наконец-таки сделать удаление! думаю после этого еще 100500 ошибок вылезет~~
в jsp надо сделать кнопку “Обновить”, которая работает по принципу “отправить на сервер, провести контроль качества и вернуть обратно на редактирование” еще можно показать разницу с тем что сейчас в файле
~~Страница с историей изменений~~
~~ввести стандартный STD_HEANOR и начать делать СРД,~~
а еще - конвергировать с нашим форматом wiki, но это совсем другая задача (будет). А еще- этот пункт нужен, чтобы сохранить единую нумерацию замечаний здесь и ниже.
0101A00A - одна из А была русской, новый код на этом ломался при редактировании, вообще ничего нельзя отредактировать. Думаю генератор нового QR-кода. А вообще - назрел контроль качества
При редактировании значения в теге Input не экранируются - в результате строки с пробелами превращаются в свою первую лексему. и заключить в кавычки - это НЕДОСТАТОЧНО, строка сама может содержать кавычки. Я, кстати эту ошибку уже исправлял 31-го ПРОШУ ОБРАТИТЬ ВНИМАНИЕ! для лучшего осознания - поручу разбирать все карточки, которые были испорчены новым интерфейсов редактирования
~~Ссылки на QR-код должны отрываться в новом окне~~
~~Деньги превращать в ссылки не надо, я сейчас еще маркеров QR* наделаю QRPRODUCT, QRMODEL, QRMONEYGOT~~
При удалении - надо спрашивать, удаление только через POST, иначе я тут сделаю страничку, с картинками, заходя на которую пользователь, залогоненный в систему, сам того не зная удалит все, что я ему скажу. И это Свершилось! нас проиндексировали, и все поудаляли!
Вышесказанное также касается и любого редактирования (будем проверять метод, раз, да еще и сессионную куку в форму подмешивать)
“Деньги распознано” вернуть, суммирование - под ними
Удаление появилось, теперь надо сделать список удаленных записей и процедуру восстановления (hint: удаленную запись можно просмотреть по ZRID, и ее историю - тоже)
~~При удалении не надо повторять все поля, только заголовок~~
Назрел комментарий к версии, по аналогии комментария к commit. Думаю это в отдельный файл писать надо
Участник никак не может добиться от сотрудников синхронного заполнения поля product, надо решать через обязательный контроль качества
Участник пинает на отсутствие поля “Город”, надо сделать, чтоб отстал, раз, чтобы начала давать новые хотелки - два
~~Кнопка “обновить” и “отмена” - я понял, надо самому сделать и показать~~
Я-бы еще форму редактирования переделал, чтоб покомпактнее было, как раньше - но конгресс перфекционистов не состоиться никогда, надо исправлять критичное и запускать. ЧИТАТЬ ТАК: “Здесь не все критично для запуска”
~~Да, после исправления п28 надо переделывать систему публикации~~
~~При создании новой записи сразу подставлять новый qr~~

‌

Comments (12)