[open-ils-commits] r11558 - trunk/Open-ILS/src/c-apps

[svn at svn.open-ils.org]

Author: miker
Date: 2008-12-14 23:59:50 -0500 (Sun, 14 Dec 2008)
New Revision: 11558

Modified:
   trunk/Open-ILS/src/c-apps/oils_cstore.c
   trunk/Open-ILS/src/c-apps/oils_idl-core.c
Log:
typo in idl parser; more pcrud debugging

Modified: trunk/Open-ILS/src/c-apps/oils_cstore.c
===================================================================
--- trunk/Open-ILS/src/c-apps/oils_cstore.c	2008-12-15 04:47:23 UTC (rev 11557)
+++ trunk/Open-ILS/src/c-apps/oils_cstore.c	2008-12-15 04:59:50 UTC (rev 11558)
@@ -918,9 +918,26 @@
 	            osrfLogDebug( OSRF_LOG_MARK, "Object not found in the database with primary key %s of %s", pkey, pkey_value );
                 jsonObjectFree(_tmp_params);
                 jsonObjectFree(_list);
+
+                growing_buffer* msg = buffer_init(128);
+                buffer_fadd(
+                    msg,
+                    "%s: no object found with primary key %s of %s",
+                    MODULENAME,
+                    pkey,
+                    pkey_value
+                );
+        
+                char* m = buffer_release(msg);
+                osrfAppSessionStatus( ctx->session, OSRF_STATUS_BADREQUEST, "osrfMethodException", ctx->request, m );
+        
+                free(m);
+                free(pkey_value);
+
                 return 0;
             }
 
+            free(pkey_value);
             jsonObjectFree(_tmp_params);
             jsonObjectFree(_list);
 

Modified: trunk/Open-ILS/src/c-apps/oils_idl-core.c
===================================================================
--- trunk/Open-ILS/src/c-apps/oils_idl-core.c	2008-12-15 04:47:23 UTC (rev 11557)
+++ trunk/Open-ILS/src/c-apps/oils_idl-core.c	2008-12-15 04:59:50 UTC (rev 11558)
@@ -321,7 +321,7 @@
 
 							osrfStringArray* map = osrfNewStringArray(0);
 							string_tmp = NULL;
-							if( (string_tmp = (char*)xmlGetProp(_l, BAD_CAST "permission") )) {
+							if( (string_tmp = (char*)xmlGetProp(_a, BAD_CAST "permission") )) {
 								char* map_list = strdup( string_tmp );
 								osrfLogDebug(OSRF_LOG_MARK, "Permacrud permission list is %s", string_tmp );
 	
@@ -338,11 +338,11 @@
 							}
 							osrfHashSet( _tmp, map, "permission");
 
-					    	osrfHashSet( _tmp, (char*)xmlGetProp(_l, BAD_CAST "global_required"), "global_required");
+					    	osrfHashSet( _tmp, (char*)xmlGetProp(_a, BAD_CAST "global_required"), "global_required");
 
 							map = osrfNewStringArray(0);
 							string_tmp = NULL;
-							if( (string_tmp = (char*)xmlGetProp(_l, BAD_CAST "context_field") )) {
+							if( (string_tmp = (char*)xmlGetProp(_a, BAD_CAST "context_field") )) {
 								char* map_list = strdup( string_tmp );
 								osrfLogDebug(OSRF_LOG_MARK, "Permacrud context_field list is %s", string_tmp );
 	
@@ -362,7 +362,7 @@
 							osrfHash* foreign_context = osrfNewHash();
 							osrfHashSet( _tmp, foreign_context, "foreign_context");
 
-							xmlNodePtr _f = _l->children;
+							xmlNodePtr _f = _a->children;
 
 							while(_f) {
 								if ( strcmp( (char*)_f->name, "context" ) ) {