[Gaim-commits] SF.net SVN: gaim: [16710] trunk/src/request.c

[<dat...@us...>]

Revision: 16710
Author:   datallah
Date:     2006-08-11 12:56:55 -0700 (Fri, 11 Aug 2006)
ViewCVS:  http://svn.sourceforge.net/gaim/?rev=16710&view=rev

Log Message:
-----------
Fix CID 101.  Also fix the behavior of adding a group to a fields list so that it doesn't matter if you add the group to a fields list before you add fields to the group or not.

Modified Paths:
--------------
    trunk/src/request.c
Modified: trunk/src/request.c
===================================================================
--- trunk/src/request.c	2006-08-11 19:02:47 UTC (rev 16709)
+++ trunk/src/request.c	2006-08-11 19:56:55 UTC (rev 16710)
@@ -84,7 +84,13 @@
 		field = l->data;
 
 		g_hash_table_insert(fields->fields,
-							g_strdup(gaim_request_field_get_id(field)), field);
+			g_strdup(gaim_request_field_get_id(field)), field);
+
+		if (gaim_request_field_is_required(field)) {
+			fields->required_fields =
+				g_list_append(fields->required_fields, field);
+		}
+
 	}
 }
 
@@ -280,15 +286,16 @@
 	{
 		g_hash_table_insert(group->fields_list->fields,
 							g_strdup(gaim_request_field_get_id(field)), field);
+
+		if (gaim_request_field_is_required(field))
+		{
+			group->fields_list->required_fields =
+				g_list_append(group->fields_list->required_fields, field);
+		}
 	}
 
 	field->group = group;
 
-	if (gaim_request_field_is_required(field))
-	{
-		group->fields_list->required_fields =
-			g_list_append(group->fields_list->required_fields, field);
-	}
 }
 
 const char *


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.