Hi Depak, I think we can just remove these lines instead of replacing them as these field types are already mapped with Birt data type & got redundant now.
private final static Map<String, String> entityFieldTypeBirtTypeMap = MapUtils.unmodifiableMap(UtilMisc.toMap( *"id", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING,* "url", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "name", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, * "id", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING,* "value", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "email", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "comment", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, * "id-long",DesignChoiceConstants.COLUMN_DATA_TYPE_STRING,* * "id-vlong", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING,* "very-long", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "indicator", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "very-short", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "tel-number", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, * "id-long", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING,* * "id-vlong", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING,* "description", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "long-varchar", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "short-varchar", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "credit-card-date", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "credit-card-number", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, "date-time", DesignChoiceConstants.COLUMN_DATA_TYPE_DATETIME, "date", DesignChoiceConstants.COLUMN_DATA_TYPE_DATE, "time", DesignChoiceConstants.COLUMN_DATA_TYPE_TIME, "currency-amount", DesignChoiceConstants.COLUMN_DATA_TYPE_DECIMAL, "currency-precise", DesignChoiceConstants.COLUMN_DATA_TYPE_DECIMAL, "fixed-point", DesignChoiceConstants.COLUMN_DATA_TYPE_DECIMAL, "floating-point", DesignChoiceConstants.COLUMN_DATA_TYPE_DECIMAL, "numeric", DesignChoiceConstants.COLUMN_DATA_TYPE_INTEGER, "object", DesignChoiceConstants.COLUMN_DATA_TYPE_JAVA_OBJECT, "blob", DesignChoiceConstants.COLUMN_DATA_TYPE_BLOB)); Thanks and Regards, *Aditya Sharma* | Enterprise Software Engineer HotWax Systems <http://www.hotwaxsystems.com/> <https://www.linkedin.com/in/aditya-sharma-78291810a/> On Sun, Aug 6, 2017 at 11:22 AM, <dee...@apache.org> wrote: > Author: deepak > Date: Sun Aug 6 05:52:43 2017 > New Revision: 1804224 > > URL: http://svn.apache.org/viewvc?rev=1804224&view=rev > Log: > Improved: Removed ne references from BirtUtil class file (OFBIZ-9351) > Thanks Aditya for your contribution. > I removed them but forgot to commit :) > > Modified: > ofbiz/ofbiz-plugins/trunk/birt/src/main/java/org/apache/ > ofbiz/birt/flexible/BirtUtil.java > > Modified: ofbiz/ofbiz-plugins/trunk/birt/src/main/java/org/apache/ > ofbiz/birt/flexible/BirtUtil.java > URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-plugins/trunk/ > birt/src/main/java/org/apache/ofbiz/birt/flexible/BirtUtil. > java?rev=1804224&r1=1804223&r2=1804224&view=diff > ============================================================ > ================== > --- ofbiz/ofbiz-plugins/trunk/birt/src/main/java/org/apache/ > ofbiz/birt/flexible/BirtUtil.java (original) > +++ ofbiz/ofbiz-plugins/trunk/birt/src/main/java/org/apache/ > ofbiz/birt/flexible/BirtUtil.java Sun Aug 6 05:52:43 2017 > @@ -47,7 +47,7 @@ public final class BirtUtil { > "id", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > "url", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > "name", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > - "id-ne", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > + "id", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > "value", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > "email", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > "comment", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > @@ -57,8 +57,8 @@ public final class BirtUtil { > "indicator", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > "very-short", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > "tel-number", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > - "id-long-ne", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > - "id-vlong-ne", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > + "id-long", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > + "id-vlong", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > "description", DesignChoiceConstants.COLUMN_DATA_TYPE_STRING, > "long-varchar", DesignChoiceConstants.COLUMN_ > DATA_TYPE_STRING, > "short-varchar", DesignChoiceConstants.COLUMN_ > DATA_TYPE_STRING, > @@ -79,7 +79,7 @@ public final class BirtUtil { > "id", DesignChoiceConstants.PARAM_TYPE_STRING, > "url", DesignChoiceConstants.PARAM_TYPE_STRING, > "name", DesignChoiceConstants.PARAM_TYPE_STRING, > - "id-ne", DesignChoiceConstants.PARAM_TYPE_STRING, > + "id", DesignChoiceConstants.PARAM_TYPE_STRING, > "value", DesignChoiceConstants.PARAM_TYPE_STRING, > "email", DesignChoiceConstants.PARAM_TYPE_STRING, > "comment", DesignChoiceConstants.PARAM_TYPE_STRING, > @@ -89,8 +89,8 @@ public final class BirtUtil { > "indicator", DesignChoiceConstants.PARAM_TYPE_STRING, > "very-short", DesignChoiceConstants.PARAM_TYPE_STRING, > "tel-number", DesignChoiceConstants.PARAM_TYPE_STRING, > - "id-long-ne", DesignChoiceConstants.PARAM_TYPE_STRING, > - "id-vlong-ne", DesignChoiceConstants.PARAM_TYPE_STRING, > + "id-long", DesignChoiceConstants.PARAM_TYPE_STRING, > + "id-vlong", DesignChoiceConstants.PARAM_TYPE_STRING, > "description", DesignChoiceConstants.PARAM_TYPE_STRING, > "long-varchar", DesignChoiceConstants.PARAM_TYPE_STRING, > "short-varchar", DesignChoiceConstants.PARAM_TYPE_STRING, > > >