[interchange-cvs] interchange - racke modified dist/lib/UI/pages/include/Options/old/item_option_simple

[interchange-cvs at icdevgroup.org]

User:      racke
Date:      2003-09-04 18:03:37 GMT
Modified:  dist/lib/UI/pages/include/Options/old item_option_simple
Log:
protect against invalid option names for old options as well

Revision  Changes    Path
1.3       +10 -1     interchange/dist/lib/UI/pages/include/Options/old/item_option_simple


rev 1.3, prev_rev 1.2
Index: item_option_simple
===================================================================
RCS file: /var/cvs/interchange/dist/lib/UI/pages/include/Options/old/item_option_simple,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- item_option_simple	4 Sep 2003 15:10:50 -0000	1.2
+++ item_option_simple	4 Sep 2003 18:03:37 -0000	1.3
@@ -27,7 +27,16 @@
 
             next unless $name && $value;
 
-            unless($key) { $key = $CGI->{sku}."-$name"; }
+            unless($key) { 	
+				if($name =~ /\W/) {
+					$Tag->error({
+						name => 'Set option groups',
+						set => errmsg(qq{Group name "%s" contains non-word characters, will not set}, $name),
+					});
+					next;
+				}
+				$key = $CGI->{sku}."-$name"; 
+			}
 
             my @value = split("\r\n",$value);