Changeset 12031

Timestamp:

Aug 8, 2011, 11:04:04 PM (13 years ago)

Author:

sdlime

Message:

Tentative fix for SDE/query issues... (#3977)

File:

• branches/branch-6-0/mapserver/mapsde.c (modified) (7 diffs)

branches/branch-6-0/mapserver/mapsde.c

@@ -36 +36 @@
 #include "mapthread.h"
 
-
-
 #ifdef USE_SDE
 #include <sdetype.h> /* ESRI SDE Client Includes */
@@ -97 +95 @@
 int msSDELayerIsOpen(layerObj *layer) {
 #ifdef USE_SDE
-
-    if(layer->layerinfo) 
-        return(MS_TRUE); 
-    
-    return MS_FALSE;
-
+  if(layer->layerinfo) 
+    return(MS_TRUE); 
+  return MS_FALSE;
 #else
-    msSetError(MS_MISCERR, "SDE support is not available.",
-             "msSDELayerIsOpen()");
-    return(MS_FALSE);
+  msSetError(MS_MISCERR, "SDE support is not available.",  "msSDELayerIsOpen()");
+  return(MS_FALSE);
 #endif
 } 
@@ -1396 +1390 @@
 int  msSDELayerClose(layerObj *layer) {
 #ifdef USE_SDE
-
-
     msSDELayerInfo *sde=NULL;
-    
     sde = layer->layerinfo;
     
@@ -1406 +1397 @@
     
     if(layer->debug) 
-        msDebug("msSDELayerClose(): Closing layer %s.\n", layer->name);
+       msDebug("msSDELayerClose(): Closing layer %s.\n", layer->name);
     
     if (sde->layerinfo) SE_layerinfo_free(sde->layerinfo);
@@ -1424 +1415 @@
  
 #else
-    msSetError( MS_MISCERR, 
-              "SDE support is not available.", 
-              "msSDELayerClose()");
+    msSetError( MS_MISCERR, "SDE support is not available.", "msSDELayerClose()");
     return(MS_FALSE);
 #endif
@@ -2082 +2071 @@
 int msSDELayerGetItems(layerObj *layer) {
 #ifdef USE_SDE
-    int status;
-    status = msSDELayerInitItemInfo(layer);
-
-    if (status != MS_SUCCESS) {
-        msSetError( MS_MISCERR,  
-                    "Unable to create SDE column info", 
-                    "msSDELayerGetItems()");
-        return(MS_FAILURE);     
-    }    
-    return (MS_SUCCESS);
-
+  int status;
+  status = msSDELayerInitItemInfo(layer);
+  if (status != MS_SUCCESS) {
+    msSetError( MS_MISCERR, "Unable to create SDE column info", "msSDELayerGetItems()");
+    return(MS_FAILURE);     
+  }    
+  return (MS_SUCCESS);
 #else
-    msSetError( MS_MISCERR, 
-                "SDE support is not available.", 
-                "msSDELayerGetItems()");
-    return(MS_FAILURE);
+  msSetError( MS_MISCERR, "SDE support is not available.", "msSDELayerGetItems()");
+  return(MS_FAILURE);
 #endif
 }
@@ -2104 +2087 @@
 /* msSDELayerFreeItemInfo                                               */
 /* -------------------------------------------------------------------- */
-void msSDELayerFreeItemInfo(layerObj *layer)
-{
+void msSDELayerFreeItemInfo(layerObj *layer) {
 #ifdef USE_SDE
-    msSDELayerInfo *sde = NULL;
-    int i;
-    if (!msSDELayerIsOpen(layer)) {
-        msSetError( MS_SDEERR,
-                    "SDE layer has not been opened.",
-                    "msSDELayerFreeItemInfo()");
-    }
-    sde = layer->layerinfo;
+  msSDELayerInfo *sde = NULL;
+
+  sde = layer->layerinfo;
+  if(sde) {
     if (sde->basedefs) {
-        SE_table_free_descriptions(sde->basedefs);  
-        sde->basedefs = NULL;
+      SE_table_free_descriptions(sde->basedefs);  
+      sde->basedefs = NULL;
     }
     if (sde->joindefs) {
-        SE_table_free_descriptions(sde->joindefs);
-        sde->joindefs = NULL;
-    }
-    if (layer->iteminfo) {
-        msFree(layer->iteminfo);
-        layer->iteminfo = NULL;
-    }
-
-     if (layer->items) {
-         for (i=0; i< layer->numitems; i++) {
-             msFree(layer->items[i]);
-         }
-         msFree(layer->items);
-         layer->items = NULL;
-         layer->numitems = 0;
-     }
-
+      SE_table_free_descriptions(sde->joindefs);
+      sde->joindefs = NULL;
+    }
+  }
+
+  if (layer->iteminfo) {
+    msFree(layer->iteminfo);
+    layer->iteminfo = NULL;
+  }
 #else
-    msSetError( MS_MISCERR, 
-                "SDE support is not available.", 
-                "msSDELayerFreeItemInfo()");
+  msSetError( MS_MISCERR, "SDE support is not available.", "msSDELayerFreeItemInfo()");
+  return;
 #endif
 }