Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
yoannawx opened this issue Jun 26, 2024 · 8 comments

Comments

@yoannawx
Copy link

yoannawx commented Jun 26, 2024

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

@gsbelarus
Copy link
Member

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

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

@Polfath
Copy link

Polfath commented Jun 27, 2024

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

@gsbelarus
Copy link
Member

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

@gsbelarus
Copy link
Member

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

@NikolayUkleyko
Copy link

NikolayUkleyko commented Jun 27, 2024

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

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

@yoannawx
Copy link
Author

#4479 (comment)
проверено. в этой версии на проблемных отчетах отрабатывает правильно

@gsbelarus
Copy link
Member

@yoannawx @alexandra-gsoftware @MIchaelShoihet

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

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

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

@MIchaelShoihet
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants