midsuit
/
core-jgi
6
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

IDEMPIERE-3662 Error when inserting PO with a LOB mandatory field

This commit is contained in:
Carlos Ruiz 2018-03-20 18:18:07 -03:00
parent 75d8fd5ea6
commit 193b7cae19
1 changed files with 23 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
org.adempiere.base/src/org/compiere/model/PO.java
View File

@ -2846,7 +2846,8 @@ public abstract class PO
if (DisplayType.isLOB(dt)) if (DisplayType.isLOB(dt))
{ {
lobAdd (value, i, dt); lobAdd (value, i, dt);
continue; if (!p_info.isColumnMandatory(i))
continue;
} }
// ** add column ** // ** add column **
@ -2885,7 +2886,16 @@ public abstract class PO
else if (c == String.class) else if (c == String.class)
sqlValues.append (encrypt(i,DB.TO_STRING ((String)value))); sqlValues.append (encrypt(i,DB.TO_STRING ((String)value)));
else if (DisplayType.isLOB(dt)) else if (DisplayType.isLOB(dt))
sqlValues.append("null"); // no db dependent stuff here {
if (p_info.isColumnMandatory(i))
{
sqlValues.append("''"); // no db dependent stuff here -- at this point value is known to be not null
}
else
{
sqlValues.append("null");
}
}
else else
sqlValues.append (saveNewSpecial (value, i)); sqlValues.append (saveNewSpecial (value, i));
} }
@ -2910,7 +2920,17 @@ public abstract class PO
if (DisplayType.isLOB(dt)) if (DisplayType.isLOB(dt))
{ {
params.add(null); if (p_info.isColumnMandatory(i))
{
if (dt == DisplayType.Binary)
params.add(new byte[] {0}); // -- at this point value is known to be not null
else
params.add(""); // -- at this point value is known to be not null
}
else
{
params.add(null);
}
} }
else if (value == null || value.equals (Null.NULL)) else if (value == null || value.equals (Null.NULL))
{ {