pidgin3: comparison finch/gntrequest.c

-:18acb99a0fa6
+:118067ca0367
 	return window;
 }
 static void
-finch_close_request(PurpleRequestType type, gpointer ui_handle)
+finch_close_request(G_GNUC_UNUSED PurpleRequestType type, gpointer ui_handle) {
-{
 	GntWidget *widget = GNT_WIDGET(ui_handle);
-	if (type == PURPLE_REQUEST_FIELDS) {
-		PurpleRequestFields *fields = g_object_get_data(G_OBJECT(widget), "fields");
-		purple_request_fields_destroy(fields);
-	}
 	widget = gnt_widget_get_toplevel(widget);
 	gnt_widget_destroy(widget);
 }
 static void
 	return window;
 }
 static void
-request_fields_cb(GntWidget *button, PurpleRequestFields *fields)
+request_fields_cb(GntWidget *button, PurpleRequestPage *page) {
-{
 	PurpleRequestFieldsCb callback = g_object_get_data(G_OBJECT(button), "activate-callback");
 	gpointer data = g_object_get_data(G_OBJECT(button), "activate-userdata");
 	GList *list;
 	GntWidget *window;
 	/* Update the data of the fields. Pidgin does this differently. Instead of
 	 * updating the fields at the end like here, it updates the appropriate field
 	 * instantly whenever a change is made. That allows it to make sure the
 	 * 'required' fields are entered before the user can hit OK. It's not the case
 	 * here, although it can be done. */
-	for (list = purple_request_fields_get_groups(fields); list; list = list->next)
+	for(list = purple_request_page_get_groups(page); list; list = list->next) {
-	{
 		PurpleRequestGroup *group = list->data;
 		GList *fields = purple_request_group_get_fields(group);
 		for (; fields ; fields = fields->next)
 		{
 	}
 	purple_notify_close_with_handle(button);
 	if (!g_object_get_data(G_OBJECT(button), "cancellation-function") &&
-			(!purple_request_fields_all_required_filled(fields) ||
+			(!purple_request_page_all_required_filled(page) ||
-			!purple_request_fields_all_valid(fields))) {
+			!purple_request_page_all_valid(page))) {
 		purple_notify_error(button, _("Error"),
 			_("You must properly fill all the required fields."),
 			_("The required fields are underlined."), NULL);
 		return;
 	}
-	if (callback)
+	if(callback) {
-		callback(data, fields);
+		callback(data, page);
+	}
 	window = gnt_widget_get_toplevel(button);
 	purple_request_close(PURPLE_REQUEST_FIELDS, window);
 }
 	gnt_widget_set_size(combo, 20, 3); /* ew */
 	return combo;
 }
 static void
-multifield_extra_cb(GntWidget *button, PurpleRequestFields *allfields)
+multifield_extra_cb(GntWidget *button, PurpleRequestPage *page) {
-{
 	PurpleRequestFieldsCb cb;
 	gpointer cb_data;
 	gpointer handle;
 	handle = g_object_get_data(G_OBJECT(button), "ui-handle");
 	cb = g_object_get_data(G_OBJECT(button), "extra-cb");
 	cb_data = g_object_get_data(G_OBJECT(button), "extra-cb-data");
-	if (cb != NULL)
+	if(cb != NULL) {
-		cb(cb_data, allfields);
+		cb(cb_data, page);
+	}
 	action_performed(button, handle);
 	purple_request_close(PURPLE_REQUEST_FIELDS, handle);
 }
 static void *
 finch_request_fields(const char *title, const char *primary,
-		const char *secondary, PurpleRequestFields *allfields,
+		const char *secondary, PurpleRequestPage *page,
 		const char *ok, GCallback ok_cb,
 		const char *cancel, GCallback cancel_cb,
 		PurpleRequestCommonParameters *cpar,
 		void *userdata)
 {
 	 * stacked vertically one after the other. A GntLine will be separating
 	 * the groups. */
 	box = gnt_vbox_new(FALSE);
 	gnt_box_set_pad(GNT_BOX(box), 0);
 	gnt_box_set_fill(GNT_BOX(box), TRUE);
-	for (grlist = purple_request_fields_get_groups(allfields); grlist; grlist = grlist->next)
+	for(grlist = purple_request_page_get_groups(page); grlist;
+	    grlist = grlist->next)
 	{
 		PurpleRequestGroup *group = grlist->data;
 		GList *fields = purple_request_group_get_fields(group);
 		GntWidget *hbox;
 		const char *title = purple_request_group_get_title(group);
 		if (grlist->next)
 			gnt_box_add_widget(GNT_BOX(box), gnt_hline_new());
 	}
 	gnt_box_add_widget(GNT_BOX(window), box);
-	box = setup_button_box(window, userdata, request_fields_cb, allfields,
+	box = setup_button_box(window, userdata, request_fields_cb, page,
-			ok, ok_cb, cancel, cancel_cb, NULL);
+	                       ok, ok_cb, cancel, cancel_cb, NULL);
 	extra_actions = purple_request_cpar_get_extra_actions(cpar);
 	for (GSList *it = extra_actions; it; it = it->next) {
 		PurpleKeyValuePair *extra_action = it->data;
 		gnt_box_add_widget_in_front(GNT_BOX(box), button);
 		g_object_set_data(G_OBJECT(button), "ui-handle", window);
 		g_object_set_data(G_OBJECT(button), "extra-cb", extra_action->value);
 		g_object_set_data(G_OBJECT(button), "extra-cb-data", userdata);
 		g_signal_connect(G_OBJECT(button), "activate",
-			G_CALLBACK(multifield_extra_cb), allfields);
+		                 G_CALLBACK(multifield_extra_cb), page);
 	}
 	help_cb = purple_request_cpar_get_help_cb(cpar, &help_data);
 	if (help_cb) {
 		GntWidget *button = gnt_button_new(_("Help"));
 	if (username && accountlist) {
 		g_signal_connect(username, "completion", G_CALLBACK(update_selected_account), accountlist);
 	}
-	g_object_set_data(G_OBJECT(window), "fields", allfields);
+	g_object_set_data_full(G_OBJECT(window), "fields", page, g_object_unref);
 	return window;
 }
 static void
 {
 }
 void
 finch_request_save_in_prefs(G_GNUC_UNUSED gpointer data,
-PurpleRequestFields *allfields)
+PurpleRequestPage *page)
 {
 	GList *list;
-	for (list = purple_request_fields_get_groups(allfields); list; list = list->next) {
+	for(list = purple_request_page_get_groups(page); list; list = list->next) {
 		PurpleRequestGroup *group = list->data;
 		GList *fields = purple_request_group_get_fields(group);
 		for (; fields ; fields = fields->next) {
 			PurpleRequestField *field = fields->data;

Mercurial > pidgin3 / file comparison

comparison: finch/gntrequest.c

finch/gntrequest.c