{
gboolean enable_ok = TRUE;
ModestProtocolRegistry *protocol_registry;
+ ModestSecurityOptionsView *sec_view;
ModestDefaultAccountSettingsDialogPrivate *priv;
priv = MODEST_DEFAULT_ACCOUNT_SETTINGS_DIALOG_GET_PRIVATE (self);
-
+
/* The account details title is mandatory: */
if (entry_is_empty(priv->entry_account_title))
enable_ok = FALSE;
/* The user details username is mandatory: */
if (enable_ok && entry_is_empty(priv->entry_user_username))
enable_ok = FALSE;
-
+
/* The user details email address is mandatory: */
if (enable_ok && entry_is_empty (priv->entry_user_email))
enable_ok = FALSE;
if (enable_ok && entry_is_empty(priv->entry_incomingserver))
enable_ok = FALSE;
+ sec_view = MODEST_SECURITY_OPTIONS_VIEW (priv->incoming_security);
+ if (enable_ok &&
+ modest_security_options_view_has_missing_mandatory_data (sec_view))
+ enable_ok = FALSE;
+
/* The custom outgoing server is mandatory: */
if (enable_ok && entry_is_empty(priv->entry_outgoingserver))
enable_ok = FALSE;
+ sec_view = MODEST_SECURITY_OPTIONS_VIEW (priv->outgoing_security);
+ if (enable_ok &&
+ modest_security_options_view_has_missing_mandatory_data (sec_view))
+ enable_ok = FALSE;
+
protocol_registry = modest_runtime_get_protocol_registry ();
-
+
/* Enable the buttons,
* identifying them via their associated response codes:
*/
#define PORT_MIN 1
#define PORT_MAX 65535
+typedef struct _ModestMaemoSecurityOptionsViewPrivate ModestMaemoSecurityOptionsViewPrivate;
+struct _ModestMaemoSecurityOptionsViewPrivate {
+ gboolean missing_data;
+};
+
+#define MODEST_MAEMO_SECURITY_OPTIONS_VIEW_GET_PRIVATE(o) \
+ (G_TYPE_INSTANCE_GET_PRIVATE((o), \
+ MODEST_TYPE_MAEMO_SECURITY_OPTIONS_VIEW, \
+ ModestMaemoSecurityOptionsViewPrivate))
+
static void modest_maemo_security_options_view_init (ModestMaemoSecurityOptionsView *obj);
static void modest_maemo_security_options_view_finalize (GObject *obj);
static void modest_maemo_security_options_view_class_init (ModestMaemoSecurityOptionsViewClass *klass);
gpointer user_data)
{
ModestSecurityOptionsView* self;
+ ModestMaemoSecurityOptionsViewPrivate *priv;
ModestSecurityOptionsViewPrivate *ppriv;
ModestProtocolType auth_proto;
ModestSecureauthPicker *picker;
- gboolean is_secure, missing;
+ gboolean is_secure;
ModestProtocolRegistry *protocol_registry;
self = MODEST_SECURITY_OPTIONS_VIEW (user_data);
+ priv = MODEST_MAEMO_SECURITY_OPTIONS_VIEW_GET_PRIVATE (self);
ppriv = MODEST_SECURITY_OPTIONS_VIEW_GET_PRIVATE (self);
protocol_registry = modest_runtime_get_protocol_registry ();
/* Outgoing username is mandatory if outgoing auth is secure */
picker = MODEST_SECUREAUTH_PICKER (ppriv->auth_view);
auth_proto = modest_secureauth_picker_get_active_secureauth (picker);
- is_secure = modest_protocol_registry_protocol_type_is_secure (protocol_registry,
+ is_secure = modest_protocol_registry_protocol_type_is_secure (protocol_registry,
auth_proto);
- if (is_secure &&
+ if (is_secure &&
!g_ascii_strcasecmp (hildon_entry_get_text (HILDON_ENTRY (ppriv->user_entry)), "")) {
- missing = TRUE;
+ priv->missing_data = TRUE;
} else {
- missing = FALSE;
+ priv->missing_data = FALSE;
}
- if (!missing && ppriv->full && !modest_number_editor_is_valid (MODEST_NUMBER_EDITOR (ppriv->port_view)))
- missing = TRUE;
-
+ if (!priv->missing_data &&
+ ppriv->full &&
+ !modest_number_editor_is_valid (MODEST_NUMBER_EDITOR (ppriv->port_view)))
+ priv->missing_data = TRUE;
+
/* Emit a signal to notify if mandatory data is missing */
- g_signal_emit_by_name (G_OBJECT (self), "missing_mandatory_data",
- missing, NULL);
+ g_signal_emit_by_name (G_OBJECT (self), "missing_mandatory_data",
+ priv->missing_data, NULL);
}
void
return (GtkWidget *) self;
}
-static void
-modest_maemo_security_options_view_load_settings (ModestSecurityOptionsView* self,
+gboolean
+modest_security_options_view_has_missing_mandatory_data (ModestSecurityOptionsView* self)
+{
+ ModestMaemoSecurityOptionsViewPrivate *priv;
+
+ priv = MODEST_MAEMO_SECURITY_OPTIONS_VIEW_GET_PRIVATE (self);
+
+ return priv->missing_data;
+}
+
+static void
+modest_maemo_security_options_view_load_settings (ModestSecurityOptionsView* self,
ModestAccountSettings *settings)
{
ModestSecurityOptionsViewPrivate *ppriv;
modest_maemo_security_options_view_parent_class = g_type_class_peek_parent (klass);
+ g_type_class_add_private (gobject_class, sizeof (ModestMaemoSecurityOptionsViewPrivate));
gobject_class->finalize = modest_maemo_security_options_view_finalize;
MODEST_SECURITY_OPTIONS_VIEW_CLASS (klass)->load_settings =
typedef struct _ModestMaemoSecurityOptionsViewPrivate ModestMaemoSecurityOptionsViewPrivate;
struct _ModestMaemoSecurityOptionsViewPrivate {
+ gboolean missing_data;
};
#define MODEST_MAEMO_SECURITY_OPTIONS_VIEW_GET_PRIVATE(o) \
gpointer user_data)
{
ModestSecurityOptionsView* self;
+ ModestMaemoSecurityOptionsViewPrivate *priv;
ModestSecurityOptionsViewPrivate *ppriv;
ModestProtocolType auth_proto;
ModestSecureauthComboBox *combo;
- gboolean is_secure, missing;
+ gboolean is_secure;
ModestProtocolRegistry *protocol_registry;
self = MODEST_SECURITY_OPTIONS_VIEW (user_data);
+ priv = MODEST_MAEMO_SECURITY_OPTIONS_VIEW_GET_PRIVATE (self);
ppriv = MODEST_SECURITY_OPTIONS_VIEW_GET_PRIVATE (self);
protocol_registry = modest_runtime_get_protocol_registry ();
/* Outgoing username is mandatory if outgoing auth is secure */
combo = MODEST_SECUREAUTH_COMBO_BOX (ppriv->auth_view);
auth_proto = modest_secureauth_combo_box_get_active_secureauth (combo);
- is_secure = modest_protocol_registry_protocol_type_is_secure (protocol_registry,
+ is_secure = modest_protocol_registry_protocol_type_is_secure (protocol_registry,
auth_proto);
- if (is_secure &&
+ if (is_secure &&
!g_ascii_strcasecmp (gtk_entry_get_text (GTK_ENTRY (ppriv->user_entry)), "")) {
- missing = TRUE;
+ priv->missing_data = TRUE;
} else {
- missing = FALSE;
+ priv->missing_data = FALSE;
}
-
+
/* Emit a signal to notify if mandatory data is missing */
- g_signal_emit_by_name (G_OBJECT (self), "missing_mandatory_data",
- missing, NULL);
+ g_signal_emit_by_name (G_OBJECT (self), "missing_mandatory_data",
+ priv->missing_data, NULL);
}
static void
return FALSE;
}
+gboolean
+modest_security_options_view_has_missing_mandatory_data (ModestSecurityOptionsView* self)
+{
+ ModestMaemoSecurityOptionsViewPrivate *priv;
+
+ priv = MODEST_MAEMO_SECURITY_OPTIONS_VIEW_GET_PRIVATE (self);
+
+ return priv->missing_data;
+}
+
static void
modest_maemo_security_options_view_init (ModestMaemoSecurityOptionsView *obj)
{