Thanks for mentioning this. I have checked the code in the applications and 
there is no business logic that uses the enumCode; of course there could be 
custom code outside that may rely on this but this is definitely out of our 
control and may be an issue with several changes we do in the trunk, not just 
the ones related to data.

Jacopo

On May 15, 2013, at 5:35 PM, Adam Heath <doo...@brainfood.com> wrote:

> Hmm.  enumCode with a typo.  Couldn't this possibly break something
> somewhere for somebody?
> 
> On 05/15/2013 01:27 AM, jaco...@apache.org wrote:
>> Author: jacopoc
>> Date: Wed May 15 06:27:41 2013
>> New Revision: 1482689
>> 
>> URL: http://svn.apache.org/r1482689
>> Log:
>> Fixed typos
>> 
>> Modified:
>>    ofbiz/trunk/applications/order/data/OrderTypeData.xml
>> 
>> Modified: ofbiz/trunk/applications/order/data/OrderTypeData.xml
>> URL: 
>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/data/OrderTypeData.xml?rev=1482689&r1=1482688&r2=1482689&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/order/data/OrderTypeData.xml (original)
>> +++ ofbiz/trunk/applications/order/data/OrderTypeData.xml Wed May 15 
>> 06:27:41 2013
>> @@ -76,13 +76,13 @@ under the License.
>>     <EnumerationType description="Order Sales Channel" 
>> enumTypeId="ORDER_SALES_CHANNEL" hasTable="N" parentTypeId=""/>
>>     <Enumeration description="Web Channel" enumCode="WEB_CHANNEL" 
>> enumId="WEB_SALES_CHANNEL" sequenceId="01" enumTypeId="ORDER_SALES_CHANNEL"/>
>>     <Enumeration description="POS Channel" enumCode="POS_CHANNEL" 
>> enumId="POS_SALES_CHANNEL" sequenceId="02" enumTypeId="ORDER_SALES_CHANNEL"/>
>> -    <Enumeration description="Fax Channel" enumCode="FAX_CAHNNEL" 
>> enumId="FAX_SALES_CHANNEL" sequenceId="03" enumTypeId="ORDER_SALES_CHANNEL"/>
>> -    <Enumeration description="Phone Channel" enumCode="PHONE_CAHNNEL" 
>> enumId="PHONE_SALES_CHANNEL" sequenceId="04" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>> -    <Enumeration description="E-Mail Channel" enumCode="EMAIL_CAHNNEL" 
>> enumId="EMAIL_SALES_CHANNEL" sequenceId="05" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>> -    <Enumeration description="Snail Mail Channel" enumCode="SNAIL_CAHNNEL" 
>> enumId="SNAIL_SALES_CHANNEL" sequenceId="06" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>> +    <Enumeration description="Fax Channel" enumCode="FAX_CHANNEL" 
>> enumId="FAX_SALES_CHANNEL" sequenceId="03" enumTypeId="ORDER_SALES_CHANNEL"/>
>> +    <Enumeration description="Phone Channel" enumCode="PHONE_CHANNEL" 
>> enumId="PHONE_SALES_CHANNEL" sequenceId="04" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>> +    <Enumeration description="E-Mail Channel" enumCode="EMAIL_CHANNEL" 
>> enumId="EMAIL_SALES_CHANNEL" sequenceId="05" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>> +    <Enumeration description="Snail Mail Channel" enumCode="SNAIL_CHANNEL" 
>> enumId="SNAIL_SALES_CHANNEL" sequenceId="06" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>>     <Enumeration description="Affiliate Channel" enumCode="AFFIL_CHANNEL" 
>> enumId="AFFIL_SALES_CHANNEL" sequenceId="07" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>>     <Enumeration description="eBay Channel" enumCode="EBAY_CHANNEL" 
>> enumId="EBAY_SALES_CHANNEL" sequenceId="08" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>> -    <Enumeration description="Unknown Channel" enumCode="UNKOWN_CAHNNEL" 
>> enumId="UNKNWN_SALES_CHANNEL" sequenceId="99" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>> +    <Enumeration description="Unknown Channel" enumCode="UNKOWN_CHANNEL" 
>> enumId="UNKNWN_SALES_CHANNEL" sequenceId="99" 
>> enumTypeId="ORDER_SALES_CHANNEL"/>
>>     <EnumerationType description="Order Item Change Type" 
>> enumTypeId="ODR_ITM_CHANGE_TYPE" hasTable="N" parentTypeId=""/>
>>     <Enumeration description="Order Item Update" enumCode="ITM_UPDATE" 
>> enumId="ODR_ITM_UPDATE" sequenceId="01" enumTypeId="ODR_ITM_CHANGE_TYPE"/>
>>     <Enumeration description="Order Item Cancel" enumCode="ITM_CANCEL" 
>> enumId="ODR_ITM_CANCEL" sequenceId="02" enumTypeId="ODR_ITM_CHANGE_TYPE"/>
>> 
>> 
> 

Reply via email to