- edited description
Branch Yadzuka - список проблем, препятствующих запуску в эксплуатацию 2020-02-10
Issue #18
new
Проблем пока много, я буду записывать их по мере нахождения. Также, что-то буду устранять в процессе поиска, так что пока это моя задача, но вечером, когда я устану - она станет твоей
При сохранении, запись (версия) в файл пишется дважды. Чорт! это еще и плавающая ошибка… Кстати, если запись не изменилась - не надо ее записывать вообщеНе контролируются \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)
-
reporter -
reporter - edited description
-
reporter - edited description
-
reporter - edited description
-
Account Deactivated - edited description
-
Account Deactivated - edited description
-
reporter 2020-03-09 список найденных проблем:
- 17: 0101A00A - одна из А была русской, новый код на этом ломался при редактировании, вообще ничего нельзя отредактировать. Думаю генератор нового QR-кода. А вообще - назрел контроль качества
- 18: При редактировании значения в теге Input не экранируются - в результате строки с пробелами превращаются в свою первую лексему. и заключить в кавычки - это НЕДОСТАТОЧНО, строка сама может содержать кавычки. Я, кстати эту ошибку уже исправлял 31-го ПРОШУ ОБРАТИТЬ ВНИМАНИЕ! для лучшего осознания - поручу разбирать все карточки, которые были испорчены новым интерфейсов редактирования
- 19: Ссылки на QR-код должны отрываться в новом окне
- 20: Деньги превращать в ссылки не надо, я сейчас еще маркеров QR* наделаю QRPRODUCT, QRMODEL, QRMONEYGOT
- 21: при удалении - надо спрашивать, удаление только через POST, иначе я тут сделаю страничку, с картинками, заходя на которую пользователь, залогоненный в систему, сам того не зная удалит все, что я ему скажу
- 22: Вышесказанное также касается и любого редактирования (будем проверять метод, раз, да еще и сессионную куку в форму подмешивать)
- 23: “Деньги распознано” вернуть, суммирование - под ними
- 24: Удаление появилось, теперь надо сделать список удаленных записей и процедуру восстановления (hint: удаленную запись можно просмотреть по ZRID, и ее историю - тоже)
- 25: При удалении не надо повторять все поля, только заголовок
- 26: Назрел комментарий к версии, по аналогии комментария к commit. Думаю это в отдельный файл писать надо
- 27: Участник никак не может добиться от сотрудников синхронного заполнения поля product, надо решать через обязательный контроль качества
- 28: Участник пинает на отсутствие поля “Город”, надо сделать, чтоб отстал, раз, чтобы начала давать новые хотелки - два
- 29: кнопка “обновить” и “отмена” - я понял, надо самому сделать и показать
- 30: Я-бы еще форму редактирования переделал, чтоб покомпактнее было, как раньше - но конгресс перфекционистов не состоиться никогда, надо исправлять критичное и запускать. ЧИТАТЬ ТАК: “Здесь не все критично для запуска”
- 31: Да, после исправления п28 надо переделывать систему публикации
-
reporter 32. В новой записи сразу подставлять QR-код
-
Account Deactivated - edited description
-
Account Deactivated - edited description
-
reporter - edited description
-
reporter Вообще, почти все исправили, и уже запустили. Так что то, что осталось - переделать на отдельные задачи и закрыть
- Log in to comment