You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2020/10/03 09:26:34 UTC
[ofbiz-framework] branch trunk updated: Improved: "description" and
"comment" datatypes unnecessarily limited to 255 char (OFBIZ-12021)
This is an automated email from the ASF dual-hosted git repository.
jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git
The following commit(s) were added to refs/heads/trunk by this push:
new 686f01c Improved: "description" and "comment" datatypes unnecessarily limited to 255 char (OFBIZ-12021)
686f01c is described below
commit 686f01c8cfc1427ca2a5b623c7e7a5b970cb8e0c
Author: Jacques Le Roux <ja...@les7arts.com>
AuthorDate: Sat Oct 3 11:26:27 2020 +0200
Improved: "description" and "comment" datatypes unnecessarily limited to 255 char (OFBIZ-12021)
"Description" and "comment" fields are restricted to 255 char.
However after some research I found that all the supported databases support
longer then 255 char
I would expect users to want to write more then 255 char in a comment
(I know I personally can).
A discussion has been started on dev ML:
https://markmail.org/message/7utrgejc2c44i37u
Also adds a note in DATAMODEL_CHANGES.adoc
Thanks: ddev
---
applications/datamodel/DATAMODEL_CHANGES.adoc | 3 +++
framework/entity/fieldtype/fieldtypeadvantage.xml | 4 ++--
framework/entity/fieldtype/fieldtypeaxion.xml | 4 ++--
framework/entity/fieldtype/fieldtypedaffodil.xml | 4 ++--
framework/entity/fieldtype/fieldtypederby.xml | 4 ++--
framework/entity/fieldtype/fieldtypefirebird.xml | 4 ++--
framework/entity/fieldtype/fieldtypehsql.xml | 4 ++--
framework/entity/fieldtype/fieldtypemssql.xml | 6 +++---
framework/entity/fieldtype/fieldtypemysql.xml | 4 ++--
framework/entity/fieldtype/fieldtypeoracle.xml | 4 ++--
framework/entity/fieldtype/fieldtypepostgres.xml | 4 ++--
framework/entity/fieldtype/fieldtypesapdb.xml | 4 ++--
framework/entity/fieldtype/fieldtypesybase.xml | 4 ++--
13 files changed, 28 insertions(+), 25 deletions(-)
diff --git a/applications/datamodel/DATAMODEL_CHANGES.adoc b/applications/datamodel/DATAMODEL_CHANGES.adoc
index bdb52b4..3fc80a5 100644
--- a/applications/datamodel/DATAMODEL_CHANGES.adoc
+++ b/applications/datamodel/DATAMODEL_CHANGES.adoc
@@ -37,6 +37,9 @@ The following file contains information about the data model changes in the Apac
1. ProductPromoCodeEmail
+=== Field type changes
+field types "description" and "comment" are now made the same type as "very-long", (meaning people can now have comments that go longer then 255 charactors)
+
=== Field Changes
[cols="<,^,^,^,>",options="header",]
|=======================================================
diff --git a/framework/entity/fieldtype/fieldtypeadvantage.xml b/framework/entity/fieldtype/fieldtypeadvantage.xml
index 9f81c3e..c7fd691 100644
--- a/framework/entity/fieldtype/fieldtypeadvantage.xml
+++ b/framework/entity/fieldtype/fieldtypeadvantage.xml
@@ -50,8 +50,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="Char(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="Memo" java-type="String"/>
- <field-type-def type="comment" sql-type="Char(255)" java-type="String"/>
- <field-type-def type="description" sql-type="Char(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="Memo" java-type="String"/>
+ <field-type-def type="description" sql-type="Memo" java-type="String"/>
<field-type-def type="name" sql-type="Char(100)" java-type="String"/>
<field-type-def type="value" sql-type="Char(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypeaxion.xml b/framework/entity/fieldtype/fieldtypeaxion.xml
index 2ce5f3b..bc9cd40 100644
--- a/framework/entity/fieldtype/fieldtypeaxion.xml
+++ b/framework/entity/fieldtype/fieldtypeaxion.xml
@@ -46,8 +46,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="CLOB" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="CLOB" java-type="String"/>
+ <field-type-def type="description" sql-type="CLOB" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypedaffodil.xml b/framework/entity/fieldtype/fieldtypedaffodil.xml
index 8abda56..5a43a1e 100644
--- a/framework/entity/fieldtype/fieldtypedaffodil.xml
+++ b/framework/entity/fieldtype/fieldtypedaffodil.xml
@@ -46,8 +46,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR2(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="CLOB" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR2(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR2(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="CLOB" java-type="String"/>
+ <field-type-def type="description" sql-type="CLOB" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR2(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR2(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypederby.xml b/framework/entity/fieldtype/fieldtypederby.xml
index 1f40642..daf7f15 100644
--- a/framework/entity/fieldtype/fieldtypederby.xml
+++ b/framework/entity/fieldtype/fieldtypederby.xml
@@ -47,8 +47,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="CLOB" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="CLOB" java-type="String"/>
+ <field-type-def type="description" sql-type="CLOB" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypefirebird.xml b/framework/entity/fieldtype/fieldtypefirebird.xml
index d449c1c..191ef84 100644
--- a/framework/entity/fieldtype/fieldtypefirebird.xml
+++ b/framework/entity/fieldtype/fieldtypefirebird.xml
@@ -46,8 +46,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="BLOB SUB_TYPE 1" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="BLOB SUB_TYPE 1" java-type="String"/>
+ <field-type-def type="description" sql-type="BLOB SUB_TYPE 1" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypehsql.xml b/framework/entity/fieldtype/fieldtypehsql.xml
index fe96fff..3c72490 100644
--- a/framework/entity/fieldtype/fieldtypehsql.xml
+++ b/framework/entity/fieldtype/fieldtypehsql.xml
@@ -46,8 +46,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="CLOB" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="CLOB" java-type="String"/>
+ <field-type-def type="description" sql-type="CLOB" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypemssql.xml b/framework/entity/fieldtype/fieldtypemssql.xml
index 25e2d05..115dc46 100644
--- a/framework/entity/fieldtype/fieldtypemssql.xml
+++ b/framework/entity/fieldtype/fieldtypemssql.xml
@@ -44,10 +44,10 @@ under the License.
<field-type-def type="very-short" sql-type="VARCHAR(10)" java-type="String"/>
<field-type-def type="short-varchar" sql-type="VARCHAR(60)" java-type="String"/>
<field-type-def type="long-varchar" sql-type="VARCHAR(255)" java-type="String"/>
- <field-type-def type="very-long" sql-type="TEXT" java-type="String"/>
+ <field-type-def type="very-long" sql-type="VARCHAR(MAX)" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="VARCHAR(MAX)" java-type="String"/>
+ <field-type-def type="description" sql-type="VARCHAR(MAX)" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypemysql.xml b/framework/entity/fieldtype/fieldtypemysql.xml
index 843a697..a384465 100644
--- a/framework/entity/fieldtype/fieldtypemysql.xml
+++ b/framework/entity/fieldtype/fieldtypemysql.xml
@@ -47,8 +47,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="LONGTEXT" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="LONGTEXT" java-type="String"/>
+ <field-type-def type="description" sql-type="LONGTEXT" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypeoracle.xml b/framework/entity/fieldtype/fieldtypeoracle.xml
index 03e66ab..394951a 100644
--- a/framework/entity/fieldtype/fieldtypeoracle.xml
+++ b/framework/entity/fieldtype/fieldtypeoracle.xml
@@ -60,8 +60,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR2(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="CLOB" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR2(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR2(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="CLOB" java-type="String"/>
+ <field-type-def type="description" sql-type="CLOB" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR2(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR2(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypepostgres.xml b/framework/entity/fieldtype/fieldtypepostgres.xml
index a2ac980..6d0bd64 100644
--- a/framework/entity/fieldtype/fieldtypepostgres.xml
+++ b/framework/entity/fieldtype/fieldtypepostgres.xml
@@ -47,8 +47,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="TEXT" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="TEXT" java-type="String"/>
+ <field-type-def type="description" sql-type="TEXT" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR(255)" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypesapdb.xml b/framework/entity/fieldtype/fieldtypesapdb.xml
index d10aea3..2b44d60 100644
--- a/framework/entity/fieldtype/fieldtypesapdb.xml
+++ b/framework/entity/fieldtype/fieldtypesapdb.xml
@@ -46,8 +46,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR(255) UNICODE" sql-type-alias="VARCHAR() UNICODE" java-type="String"/>
<field-type-def type="very-long" sql-type="LONG UNICODE" sql-type-alias="LONG UNICODE" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR(255) UNICODE" sql-type-alias="VARCHAR() UNICODE" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR(255) UNICODE" sql-type-alias="VARCHAR() UNICODE" java-type="String"/>
+ <field-type-def type="comment" sql-type="LONG UNICODE" sql-type-alias="VARCHAR() UNICODE" java-type="String"/>
+ <field-type-def type="description" sql-type="LONG UNICODE" sql-type-alias="VARCHAR() UNICODE" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR(100) UNICODE" sql-type-alias="VARCHAR() UNICODE" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR(255) UNICODE" sql-type-alias="VARCHAR() UNICODE" java-type="String"/>
diff --git a/framework/entity/fieldtype/fieldtypesybase.xml b/framework/entity/fieldtype/fieldtypesybase.xml
index e61cd84..5cd0231 100644
--- a/framework/entity/fieldtype/fieldtypesybase.xml
+++ b/framework/entity/fieldtype/fieldtypesybase.xml
@@ -46,8 +46,8 @@ under the License.
<field-type-def type="long-varchar" sql-type="VARCHAR(255)" java-type="String"/>
<field-type-def type="very-long" sql-type="TEXT" java-type="String"/>
- <field-type-def type="comment" sql-type="VARCHAR(255)" java-type="String"/>
- <field-type-def type="description" sql-type="VARCHAR(255)" java-type="String"/>
+ <field-type-def type="comment" sql-type="TEXT" java-type="String"/>
+ <field-type-def type="description" sql-type="TEXT" java-type="String"/>
<field-type-def type="name" sql-type="VARCHAR(100)" java-type="String"/>
<field-type-def type="value" sql-type="VARCHAR(255)" java-type="String"/>