Bringing datatype BigDecimal into Service-Builder

• Type: Sub-task
• Status: Open
• Priority: Trivial
• Resolution: Unresolved
• Affects Version/s: 6.1.0 CE RC1
• Fix Version/s: None
• Component/s: Tools, Tools > Service Builder
• Labels: None
• Environment: At revision 72531
• Similar Issues: Show 5 results ; ;

  LPS-35364	Service Builder DTD does not match Service Builder requirements
  LPS-21171	Sample Service Builder plugin portlet fails to compile
  LPS-25506	Service Builder: Namespaces with a "_" causes a RuntimeException
  LPS-31580	Adding Foo in Service Builder portlet causes exceptions
  LPS-1789	Service Builder Generates NoSuchException if the <namespace> and <entity> are the same

same improvement was addressed as LEP-841.

Details got addressed at https://www.liferay.com/community/forums/-/message_boards/message/7459432.

Any application or portlet dealing with currencies should really use the java BigDecimal class. Thus It would be nice for the Service-Builder to support data-type BigDecimal.

A proposal:

1) provide data type called big_decimal in service-builder DTD

e.g.,
<column name="field6" type="big_decimal" convert-null="false" />
<column name="field7" type="big_decimal" convert-null="true" />

It would be better to add two elements for type BigDecimal in service-builder.

constraint-precision: the precision
constraint-scale: the scale

For example,

salary DECIMAL(5,2)

constraint-precision: 5
constraint-scale: 2

It can be presented as
<column name="salary" type="big_decimal"
constraint-precision="5" constraint-scale="2" convert-null="false" />

2) add new data type in Liferay core called BigDecimalType, where specifies default value;

3) map big_decimal into NUMERIC or DECIMAL as part of SQL scripts

4) map big_decimal into data type BigDecimalType

[Jonas Yuan]

solution proposal was addressed at http://www.liferay.com/web/jonas.yuan/blog/-/blogs/bringing-data-type-bigdecimal-into-service-builder.

A fix patch for trunk version will be attached here shortly.

[Prakash Khanchandani]

Any updates on the fix-patch?