extend/validate option/value lengths
authorTony Cook <tony@develop-help.com>
Thu, 23 Jul 2009 13:40:47 +0000 (13:40 +0000)
committertony <tony@45cb6cf1-00bc-42d2-bb5a-07f51df49f94>
Thu, 23 Jul 2009 13:40:47 +0000 (13:40 +0000)
schema/bse.sql
site/cgi-bin/modules/BSE/Edit/Product.pm
site/util/mysql.str

index 1a60511..c4a0da1 100644 (file)
@@ -831,7 +831,7 @@ drop table if exists bse_product_options;
 create table bse_product_options (
   id integer not null auto_increment primary key,
   product_id integer not null references product(productId),
-  name varchar(40) not null,
+  name varchar(255) not null,
   type varchar(10) not null,
   global_ref integer null,
   display_order integer not null,
@@ -844,7 +844,7 @@ drop table if exists bse_product_option_values;
 create table bse_product_option_values (
   id integer not null auto_increment primary key,
   product_option_id integer not null references bse_product_options(id),
-  value varchar(40) not null,
+  value varchar(255) not null,
   display_order integer not null,
   index option_order(product_option_id, display_order)
 ) type=innodb;
index e318e7f..f267d07 100644 (file)
@@ -536,31 +536,37 @@ my %option_fields =
     description => "Option name",
     required => 1,
     rules => "dh_one_line",
+    maxlength => 255,
    },
    value1 =>
    {
     description => "Value 1",
     rules => "dh_one_line",
+    maxlength => 255,
    },
    value2 =>
    {
     description => "Value 2",
     rules => "dh_one_line",
+    maxlength => 255,
    },
    value3 =>
    {
     description => "Value 3",
     rules => "dh_one_line",
+    maxlength => 255,
    },
    value4 =>
    {
     description => "Value 4",
     rules => "dh_one_line",
+    maxlength => 255,
    },
    value5 =>
    {
     description => "Value 5",
     rules => "dh_one_line",
+    maxlength => 255,
    },
   );
 
@@ -765,6 +771,7 @@ my %option_name =
    {
     description => "Option name",
     rules => "required;dh_one_line"
+    maxlength => 255,
    },
    default_value =>
    {
@@ -777,6 +784,7 @@ my %option_value =
   (
    description => "Value",
    rules => "required;dh_one_line"
+   maxlength => 255,
   );
 
 =item a_save_option
@@ -988,7 +996,8 @@ my %add_option_value_fields =
    value =>
    {
     description => "Value",
-    rules => "required",
+    rules => "required;dh_one_line",
+    maxlength => 255,
    },
   );
 
@@ -1164,6 +1173,7 @@ my %save_option_value_fields =
    value => 
    {
     rules => "required;dh_one_line",
+    maxlength => 255,
    },
   );
 
index 34b308a..7c406ca 100644 (file)
@@ -135,14 +135,14 @@ Index item_order;0;[order_item_id;display_order]
 Table bse_product_option_values
 Column id;int(11);NO;NULL;auto_increment
 Column product_option_id;int(11);NO;NULL;
-Column value;varchar(40);NO;NULL;
+Column value;varchar(255);NO;NULL;
 Column display_order;int(11);NO;NULL;
 Index PRIMARY;1;[id]
 Index option_order;0;[product_option_id;display_order]
 Table bse_product_options
 Column id;int(11);NO;NULL;auto_increment
 Column product_id;int(11);NO;NULL;
-Column name;varchar(40);NO;NULL;
+Column name;varchar(255);NO;NULL;
 Column type;varchar(10);NO;NULL;
 Column global_ref;int(11);YES;NULL;
 Column display_order;int(11);NO;NULL;