[mapserver-commits] r10544 - trunk/mapserver

[svn at osgeo.org]

Author: tamas
Date: 2010-09-30 11:04:28 +0000 (Thu, 30 Sep 2010)
New Revision: 10544

Modified:
   trunk/mapserver/HISTORY.TXT
   trunk/mapserver/mapogr.cpp
Log:
Fixed msOGRGetSymbolId according to the changes in gdal 1.8 (#3556)

Modified: trunk/mapserver/HISTORY.TXT
===================================================================
--- trunk/mapserver/HISTORY.TXT	2010-09-30 03:17:20 UTC (rev 10543)
+++ trunk/mapserver/HISTORY.TXT	2010-09-30 11:04:28 UTC (rev 10544)
@@ -14,6 +14,8 @@
 Current Version (SVN trunk):
 ----------------------------
 
+- Fixed msOGRGetSymbolId according to the changes in gdal 1.8 (#3556)
+
 - Support holding WMS client requests in RAM instead of writing to disk (#3555)
 
 - RFC-61: Enhance MapServer Feature Style Support (#3544)

Modified: trunk/mapserver/mapogr.cpp
===================================================================
--- trunk/mapserver/mapogr.cpp	2010-09-30 03:17:20 UTC (rev 10543)
+++ trunk/mapserver/mapogr.cpp	2010-09-30 11:04:28 UTC (rev 10544)
@@ -2717,14 +2717,21 @@
     int   numparams;
     int   nSymbol = -1;
 
-    if (pszSymbolId && pszSymbolId[0] != '\0' &&
-        (params = msStringSplit(pszSymbolId, '.', &numparams))!=NULL)
+    if (pszSymbolId && pszSymbolId[0] != '\0')
     {
-        for(int j=0; j<numparams && nSymbol == -1; j++)
+#if GDAL_VERSION_NUM >= 1800 /* Use comma as the separator */
+        params = msStringSplit(pszSymbolId, ',', &numparams);
+#else
+        params = msStringSplit(pszSymbolId, '.', &numparams);
+#endif
+        if (params != NULL)
         {
-            nSymbol = msGetSymbolIndex(symbolset, params[j], MS_FALSE);
+            for(int j=0; j<numparams && nSymbol == -1; j++)
+            {
+                nSymbol = msGetSymbolIndex(symbolset, params[j], MS_FALSE);
+            }
+            msFreeCharArray(params, numparams);
         }
-        msFreeCharArray(params, numparams);
     }
     if (nSymbol == -1 && pszDefaultSymbol)
     {