search

GoldenSoftwareLtd / gedemin

22 stars 13 forks source link

Проблема с передачей параметра типа "Логический" (ехе от 24.06.2024) #4479

Open yoannawx opened 5 months ago

yoannawx commented 5 months ago

При использовании (замечено в отчетах) параметра типа Логический, он передается в функцию, как 0 или 1, а не как False или True. При этом конструкция "if not Param then..." отрабатывает некорректно

gsbelarus commented 5 months ago

мы же вроде уже обсуждали и даже исправляли это?

в базу данных мы пишем 0, 1 но во всех остальных местах это должно быть True False

Polfath commented 4 months ago

Вернул как было. Там весь сыр-бор разгорелся из-за одного отчета на Березе, в котором некорректно обрабатывались логические параметры. Его подкорректировали. @yoannawx проверь плиз.

gsbelarus commented 4 months ago

кем обрабатывались некорректно?

gsbelarus commented 4 months ago

и где вернули? не вижу пул реквеста.

NikolayUkleyko commented 4 months ago

”Вернул как было. Там весь сыр-бор разгорелся из-за одного отчета на Березе, в котором некорректно обрабатывались логические параметры. Его подкорректировали. @yoannawx проверь плиз.”

@Polfath , не спеши. Вопрос с отчетом (по книге продаж) на Береза Мкк смотрела Александра https://discord.com/channels/1104433152576991282/1104433152576991285/1242176435976933489. Позже, внесли правки в gedemin.exe.

yoannawx commented 4 months ago

https://github.com/GoldenSoftwareLtd/gedemin/issues/4479#issuecomment-2194449781 проверено. в этой версии на проблемных отчетах отрабатывает правильно

gsbelarus commented 4 months ago

@yoannawx @alexandra-gsoftware @MIchaelShoihet

давайте с этим вопросом определимся. если у нас была схема которая, работала 20 лет, то зачем ее менять? ведь старые отчеты все перестанут работать.

в пользу 1/0 то, что там информация хранится в базе данных. т.е. не надо мучаться, в запросе мы пишем <> 0 и в отчете пишем <> 0.

мне кажется на данной стадии жизни гедымина не место таким мелким изменениям, которые деляют несовместимым старый код.

MIchaelShoihet commented 4 months ago

В Delphi5 в скрипт отчета параметр передавался True и False, в FastReport 0 и 1. В Delphi7 до изменений стало передаваться True и False. В березе был отчет один единственный в котором шла проверка <> 0. Тогда и стали исправлять. Исправили так что в редактор скрипт объектов стал передаваться 0 и 1, в результате чего сломалось много отчетов. Сейчас вернули обратно. Почему вдруг в FastReport стало передаваться True False, а не так как было раньше, пока не разобрались. Но оно и не так страшно, за все время работы нового exe в Березе вылез один такой отчет, который я уже поправил