- ModestNumberEditorPrivate *priv;
- gint error_code, fixup_value;
- const gchar *text;
- long value;
- gchar *tail;
- gboolean r;
-
- g_assert (MODEST_IS_NUMBER_EDITOR(editor));
-
- priv = MODEST_NUMBER_EDITOR_GET_PRIVATE (editor);
- g_assert (priv);
-
- text = gtk_entry_get_text (GTK_ENTRY (editor));
- error_code = -1;
- fixup_value = priv->default_val;
-
- if (text && text[0])
- {
- /* Try to convert entry text to number */
- value = strtol (text, &tail, 10);
-
- /* Check if conversion succeeded */
- if (tail[0] == 0)
- {
- /* Check if value is in allowed range. This is tricky in those
- cases when user is editing a value.
- For example: Range = [100, 500] and user have just inputted "4".
- This should not lead into error message. Otherwise value is
- resetted back to "100" and next "4" press will reset it back
- and so on. */
- if (allow_intermediate)
- {
- /* We now have the following error cases:
- * If inputted value as above maximum and
- maximum is either positive or then maximum
- negative and value is positive.
- * If inputted value is below minimum and minimum
- is negative or minumum positive and value
- negative or zero.
- In all other cases situation can be fixed just by
- adding new numbers to the string.
- */
- if (value > priv->end && (priv->end >= 0 || (priv->end < 0 && value >= 0)))
- {
- error_code = MODEST_NUMBER_EDITOR_ERROR_MAXIMUM_VALUE_EXCEED;
- fixup_value = priv->end;
- }
- else if (value < priv->start && (priv->start < 0 || (priv->start >= 0 && value <= 0)))
- {
- error_code = MODEST_NUMBER_EDITOR_ERROR_MINIMUM_VALUE_EXCEED;
- fixup_value = priv->start;
- }
- }
- else
- {
- if (value > priv->end) {
- error_code = MODEST_NUMBER_EDITOR_ERROR_MAXIMUM_VALUE_EXCEED;
- fixup_value = priv->end;
- }
- else if (value < priv->start) {
- error_code = MODEST_NUMBER_EDITOR_ERROR_MINIMUM_VALUE_EXCEED;
- fixup_value = priv->start;
- }
- }
- }
- /* The only valid case when conversion can fail is when we
- have plain '-', intermediate forms are allowed AND
- minimum bound is negative */
- else if (! allow_intermediate || strcmp (text, "-") != 0 || priv->start >= 0)
- error_code = MODEST_NUMBER_EDITOR_ERROR_ERRONEOUS_VALUE;
- }
- else if (! allow_intermediate)
- error_code = MODEST_NUMBER_EDITOR_ERROR_ERRONEOUS_VALUE;
-
- if (error_code != -1)
- {
- /* If entry is empty and intermediate forms are nor allowed,
- emit error signal */
- /* Change to default value */
- modest_number_editor_set_value (editor, fixup_value);
- g_signal_emit (editor, ModestNumberEditor_signal[RANGE_ERROR], 0, error_code, &r);
- add_select_all_idle (editor);
- }
+ ModestNumberEditorPrivate *priv;
+ gint error_code, fixup_value;
+ const gchar *text;
+ long value;
+ gchar *tail;
+ gboolean r;
+ gboolean is_valid = TRUE;
+
+ g_assert (MODEST_IS_NUMBER_EDITOR(editor));
+
+ priv = MODEST_NUMBER_EDITOR_GET_PRIVATE (editor);
+ g_assert (priv);
+
+ text = gtk_entry_get_text (GTK_ENTRY (editor));
+ error_code = -1;
+ fixup_value = priv->default_val;
+
+ if (text && text[0]) {
+ /* Try to convert entry text to number */
+ value = strtol (text, &tail, 10);
+
+ /* Check if conversion succeeded */
+ if (tail[0] == 0) {
+ /* Check if value is in allowed range. This is tricky in those
+ cases when user is editing a value.
+ For example: Range = [100, 500] and user have just inputted "4".
+ This should not lead into error message. Otherwise value is
+ resetted back to "100" and next "4" press will reset it back
+ and so on. */
+
+ if (allow_intermediate) {
+ /* We now have the following error cases:
+ * If inputted value as above maximum and
+ maximum is either positive or then maximum
+ negative and value is positive.
+ * If inputted value is below minimum and minimum
+ is negative or minumum positive and value
+ negative or zero.
+ In all other cases situation can be fixed just by
+ adding new numbers to the string.
+ */
+ if (value > priv->end && (priv->end >= 0 || (priv->end < 0 && value >= 0))) {
+ error_code = MODEST_NUMBER_EDITOR_ERROR_MAXIMUM_VALUE_EXCEED;
+ fixup_value = priv->end;
+ is_valid = FALSE;
+ } else if (value < priv->start && (priv->start < 0 || (priv->start >= 0 && value <= 0))) {
+ error_code = MODEST_NUMBER_EDITOR_ERROR_MINIMUM_VALUE_EXCEED;
+ fixup_value = priv->start;
+ is_valid = FALSE;
+ }
+ } else {
+ if (value > priv->end) {
+ error_code = MODEST_NUMBER_EDITOR_ERROR_MAXIMUM_VALUE_EXCEED;
+ fixup_value = priv->end;
+ is_valid = FALSE;
+ } else if (value < priv->start) {
+ error_code = MODEST_NUMBER_EDITOR_ERROR_MINIMUM_VALUE_EXCEED;
+ fixup_value = priv->start;
+ is_valid = FALSE;
+ }
+ }
+ /* The only valid case when conversion can fail is when we
+ have plain '-', intermediate forms are allowed AND
+ minimum bound is negative */
+ } else {
+ is_valid = FALSE;
+ if (! allow_intermediate || strcmp (text, "-") != 0 || priv->start >= 0)
+ error_code = MODEST_NUMBER_EDITOR_ERROR_ERRONEOUS_VALUE;
+ }
+ } else {
+ is_valid = FALSE;
+ if (! allow_intermediate)
+ error_code = MODEST_NUMBER_EDITOR_ERROR_ERRONEOUS_VALUE;
+ }
+
+ if (error_code != -1) {
+ /* If entry is empty and intermediate forms are nor allowed,
+ emit error signal */
+ /* Change to default value */
+ modest_number_editor_set_value (editor, fixup_value);
+ g_signal_emit (editor, ModestNumberEditor_signal[RANGE_ERROR], 0, error_code, &r);
+ add_select_all_idle (editor);
+ }
+
+ if (priv->is_valid != is_valid) {
+ g_signal_emit (editor, ModestNumberEditor_signal[VALID_CHANGED], 0, is_valid);
+ priv->is_valid = is_valid;
+ }