-
Notifications
You must be signed in to change notification settings - Fork 5
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
Округление #9
base: master
Are you sure you want to change the base?
Округление #9
Conversation
Добавление округления значения
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Добавил несколько комментариев по реализации.
@@ -534,6 +534,7 @@ function dbInstall($data) | |||
modbusdevices: REQUEST_START int(10) NOT NULL DEFAULT '0' | |||
modbusdevices: REQUEST_TOTAL int(10) NOT NULL DEFAULT '0' | |||
modbusdevices: MULTIPLIER varchar(50) NOT NULL DEFAULT '' | |||
modbusdevices: ROUND tinyint(10) NOT NULL DEFAULT '' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
для числовых значений DEFAULT лучше указывать числом, а не пустой строкой (см строку 535 к примеру).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Может быть поставить по умолчанию 2, пусть до сотой округляются значения. Вряд ли кому то нужно в УД более точные значения?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
по-умолчанию 2 мне кажется будет норм. но надо ещё проверить, чтоб при добавлении новой записи тоже по-умолчанию в выставлялась двойка в интерфейсе. т.е. надо этот момент добавления протестировать.
Если не указывать округление, по умолчанию до сотых (2)
исправление комментария к полю
После применения последних изменений, протестировал - добавляется по умолчанию округление до сотых. |
до целых округлять тогда в принципе не получится?) я не знаю нужно ли будет это (округление до целых) кому-то в принципе, но просто интересно, правильно ли я понял текущую логику. |
так я потому и переспросил про указание нуля, ведь если указать ноль, то он на двойку при сохранении заменится, разве нет? по коду так выходит... |
if (!$rec['ROUND']) $rec['ROUND'] = '2'; |
исправление
Добавляем возможность округления значений