Отсутствие проверки введенных значений в fTools

[manisha]

В инструментах fTools нет проверки на корректность введенных значений, можно запустить построение буфера на расстояние "do sosednego lesa" и в питон error будет написано:

Код: Выделить всё

Traceback (most recent call last):
  File "C:\OSGeo4W\apps\qgis-dev\python\plugins\fTools\tools\doGeoprocessing.py", line 83, in accept
    parameter = float( self.param.text() )
ValueError: invalid literal for float(): do sosednego lesa

Другие инструменты не проверял.
fTools 0.5.10, QGIS r12231.

[Максим Дубинин]

подтверждаю, r12280

[Voltron]

В инструментах fTools нет проверки на корректность введенных значений, можно запустить построение буфера на расстояние "do sosednego lesa" и в питон error будет написано

Подтверждаю не глядя, т.к. более-менее знаю код fTools.
Дело в том, что питон - язык без строгой типизации, разделения на целые, с плавающей точкой, строки и другие типы у него отсутствует. Из-за этого проверка введенных значений задача нетривиальная, строку нужно раскладывать на составляющие и каждую проверять на "цифра/не цифра".

[Максим Дубинин]

в принципе, какую-то простую проверку добавить типа "если есть символы - ругаться" можно?

[Voltron]

в принципе, какую-то простую проверку добавить типа "если есть символы - ругаться" можно?

Вот так сразу не скажу. Как-то точно можно сделать, надо смотреть

[yellow-sky]

Сделать то можно. Для ввода используется обычный QLineEdit. Соответственно для него можно установить как inputMask так и любой валидатор, в данном случае, я так думаю, должен подойти QDoubleValidator (кстати ему нужно будет обязательно добавить ограничение на нижнее значение = 0). Но проблема в том, что форма, используемая для всех функций геопроцессинга - одна! и строится динамически при выборе какой либо из функций. Эта проблема решается путем добавления соответствующих валидаторов в том или ином случае, как происходит и с подписями полей ввода. Но я думаю, что это лучше сделать создателю этого плагина. Вероятность того, что тот кто будет писать патч с валидаторами запутается или что нибудь забудет - очень велика

[Denis Rykov]

Тикет #2252

[Voltron]

Исправлено в r13312