Use isNumeric

elpaso · elpaso · commit 2bf72b770807 · 2018-09-02T09:42:37.000+02:00
diff --git a/src/gui/editorwidgets/qgstexteditwrapper.cpp b/src/gui/editorwidgets/qgstexteditwrapper.cpp
@@ -225,7 +225,7 @@ void QgsTextEditWrapper::setWidgetValue( const QVariant &val )
   // when the user is editing the field value.
   // We are checking for editable layer because in the form field context we do not
   // want to strip the separator unless the layer is editable
-  if ( layer() && layer()->isEditable() && ! QLocale().groupSeparator().isNull() && ( field().type() == QVariant::Int || field().type() == QVariant::Double || field().type() == QVariant::LongLong ) )
+  if ( layer() && layer()->isEditable() && ! QLocale().groupSeparator().isNull() && field().isNumeric() )
   {
     v = v.remove( QLocale().groupSeparator() );
   }

Original file line number	Diff line number	Diff line change
`@@ -225,7 +225,7 @@ void QgsTextEditWrapper::setWidgetValue( const QVariant &val )`
`225`	`225`	`// when the user is editing the field value.`
`226`	`226`	`// We are checking for editable layer because in the form field context we do not`
`227`	`227`	`// want to strip the separator unless the layer is editable`
`228`		`- if ( layer() && layer()->isEditable() && ! QLocale().groupSeparator().isNull() && ( field().type() == QVariant::Int \|\| field().type() == QVariant::Double \|\| field().type() == QVariant::LongLong ) )`
	`228`	`+ if ( layer() && layer()->isEditable() && ! QLocale().groupSeparator().isNull() && field().isNumeric() )`
`229`	`229`	`{`
`230`	`230`	`v = v.remove( QLocale().groupSeparator() );`
`231`	`231`	`}`