[open-ils-commits] r13814 - trunk/Open-ILS/src/c-apps (scottmk)

svn at svn.open-ils.org svn at svn.open-ils.org
Tue Aug 11 17:01:46 EDT 2009


Author: scottmk
Date: 2009-08-11 17:01:41 -0400 (Tue, 11 Aug 2009)
New Revision: 13814

Modified:
   trunk/Open-ILS/src/c-apps/oils_cstore.c
Log:
Undo previous change.  NULL can be a legitimate value for value.


Modified: trunk/Open-ILS/src/c-apps/oils_cstore.c
===================================================================
--- trunk/Open-ILS/src/c-apps/oils_cstore.c	2009-08-11 18:49:43 UTC (rev 13813)
+++ trunk/Open-ILS/src/c-apps/oils_cstore.c	2009-08-11 21:01:41 UTC (rev 13814)
@@ -1501,29 +1501,6 @@
 			value = jsonObjectToSimpleString( field_object );
 		}
 
-		if( ! value ) {
-			osrfAppSessionStatus(
-				ctx->session,
-				OSRF_STATUS_BADREQUEST,
-				"osrfMethodException",
-				ctx->request,
-				"No value available for column"
-			);
-			osrfLogError(
-				OSRF_LOG_MARK,
-				"%s: NULL value for field %s of class %s",
-				MODULENAME,
-				field_name,
-				osrfHashGet(meta, "classname")
-			);
-			osrfHashIteratorFree( field_itr );
-			buffer_free( val_buf );
-			buffer_free( col_buf );
-			buffer_free( table_buf );
-			*err = -1;
-			return jsonNULL;
-		}
-
 		if (first) {
 			first = 0;
 		} else {



More information about the open-ils-commits mailing list