Opened 21 years ago

Closed 16 years ago

#238 closed defect (invalid)

Order in handling ROSA-CGI-Parameters

Reported by: silke@… Owned by: sdlime
Priority: high Milestone:
Component: MapServer CGI Version: 3.6
Severity: normal Keywords:
Cc:

Description (last modified by sdlime)

In handling the ROSA-CGI-Parameters it is important, that first the input type
than the input coordinates are read. Normally this isn't a problem we use the
Mapserver within Zope and the Zopeserver changes the order of the parameters.
The following patch solves the problem:

-----------------------------------------------------

--- mapserv.c.orig      Wed Oct 30 10:04:22 2002
+++ mapserv.c   Wed Oct 30 10:05:25 2002
@@ -187,7 +187,7 @@
 */
 void loadForm()
 {
-  int i,j,k,n;
+  int i,j,k,n,rosaloop;
   char **tokens, *tmpstr;
   regex_t re;
   int rosa_type=0;
@@ -762,22 +762,25 @@
 /*                                                                      */
 /* -------------------------------------------------------------------- */

--------------------------------------------------------
Many greetings,

 
-    if(strcasecmp(msObj->ParamNames[i],"INPUT_TYPE") == 0)
-    { /* Rosa input type */
-        if(strcasecmp(msObj->ParamValues[i],"auto_rect") == 0) 
-        {
-            rosa_type=1; /* rectangle */
-            continue;
-        }
-

Change History (3)

comment:1 by silke@…, 21 years ago

I justed recognized that I set the 'many greetings' not at the end but within th
patch . The full patch is of course:

+++ mapserv.c   Wed Oct 30 10:05:25 2002
@@ -187,7 +187,7 @@
 */
 void loadForm()
 {
-  int i,j,k,n;
+  int i,j,k,n,rosaloop;
   char **tokens, *tmpstr;
   regex_t re;
   int rosa_type=0;
@@ -762,22 +762,25 @@
 /*                                                                      */
 /* -------------------------------------------------------------------- */
 
-    if(strcasecmp(msObj->ParamNames[i],"INPUT_TYPE") == 0)
-    { /* Rosa input type */
-        if(strcasecmp(msObj->ParamValues[i],"auto_rect") == 0) 
-        {
-            rosa_type=1; /* rectangle */
-            continue;
-        }
-            


Now, really many greetings!
     Silke

comment:2 by sdlime, 21 years ago

I may be missing something but the patch doesn't seem to do anything but add a
declaration of rosaloop. I don't see how that helps. The patch seems incomplete,
please advise...

Steve

comment:3 by sdlime, 16 years ago

Description: modified (diff)
Resolution: invalid
Status: newclosed

Closing as invalid...

Steve

Note: See TracTickets for help on using tickets.