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 2013/07/27 10:15:24 UTC
svn commit: r1507596 -
/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java
Author: jleroux
Date: Sat Jul 27 08:15:23 2013
New Revision: 1507596
URL: http://svn.apache.org/r1507596
Log:
No functional change, temporary weird compile error workaround
There is an issue with commit r1507200. It does not compile.
I wonder though why Buildbot did not detect it and also why my IDE (Eclipse) does not detect it and also why this fails, because the same scheme is successfully used elsewhere.
The demo compilers (1.6.26, mine is 1.6.22, ) say:
classes:
[javac16] Compiling 11 source files to /home/ofbiz/trunk/applications/party/build/classes
[javac16] /home/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java:2097: incompatible types
[javac16] found : java.util.Map<java.lang.String,org.ofbiz.entity.GenericValue>
[javac16] required: java.util.Map<java.lang.String,java.lang.Object>
[javac16] Map<String, Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
[javac16] ^
[javac16] /home/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java:2117: incompatible types
[javac16] found : java.util.Map<java.lang.String,org.ofbiz.entity.GenericValue>
[javac16] required: java.util.Map<java.lang.String,java.lang.Object>
[javac16] Map<String, Object> telecomNumber = UtilMisc.toMap("userLogin", userLogin);
[javac16] ^
[javac16] 2 errors
Of course replacing by
java.util.Map<java.lang.String,java.lang.Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
as the compilers suggest does not help. I temporarily replaced by
Map<String, Object> postalAddress = new HashMap<String, Object>();
postalAddress.put("userLogin", userLogin);
Unrelated: I set my IDE to automatically clean imports...
Modified:
ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java
Modified: ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java?rev=1507596&r1=1507595&r2=1507596&view=diff
==============================================================================
--- ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java (original)
+++ ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java Sat Jul 27 08:15:23 2013
@@ -19,23 +19,24 @@
package org.ofbiz.party.party;
-import java.sql.Timestamp;
-import java.util.Collection;
-import java.util.List;
-import java.util.Locale;
-import java.util.Map;
import java.io.BufferedReader;
import java.io.IOException;
import java.io.StringReader;
import java.nio.ByteBuffer;
import java.nio.charset.Charset;
+import java.sql.Timestamp;
+import java.util.Collection;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Locale;
+import java.util.Map;
import javolution.util.FastList;
import javolution.util.FastMap;
import org.apache.commons.csv.CSVFormat;
-import org.apache.commons.csv.CSVRecord;
import org.apache.commons.csv.CSVFormat.CSVFormatBuilder;
+import org.apache.commons.csv.CSVRecord;
import org.ofbiz.base.util.Debug;
import org.ofbiz.base.util.UtilDateTime;
import org.ofbiz.base.util.UtilGenerics;
@@ -2094,7 +2095,9 @@ public class PartyServices {
lastEmailAddress = rec.get("emailAddress");
}
- Map<String, Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
+ //Map<String, Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
+ Map<String, Object> postalAddress = new HashMap<String, Object>();
+ postalAddress.put("userLogin", userLogin);
Boolean postalAddressChanged = false;
if ("POSTAL_ADDRESS".equals(currentContactMechTypeId)) {
postalAddress.put("address1", rec.get("address1"));
@@ -2114,7 +2117,9 @@ public class PartyServices {
lastCountryGeoId = (String) postalAddress.get("countryGeoId");
}
- Map<String, Object> telecomNumber = UtilMisc.toMap("userLogin", userLogin);
+ //Map<String, Object> telecomNumber = UtilMisc.toMap("userLogin", userLogin);
+ Map<String, Object> telecomNumber = new HashMap<String, Object>();
+ postalAddress.put("userLogin", userLogin);
Boolean telecomNumberChanged = false;
if ("TELECOM_NUMBER".equals(currentContactMechTypeId)) {
telecomNumber.put("countryCode", rec.get("telCountryCode"));
Re: svn commit: r1507596 -
/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.jav
a
Posted by Jacques Le Roux <ja...@les7arts.com>.
Yes I guess so, I have asked the infra team for the version Builbot uses...
Jacques
----- Original Message -----
From: "Hans Bakker" <ma...@antwebsystems.com>
To: <de...@ofbiz.apache.org>
Cc: <jl...@apache.org>; <co...@ofbiz.apache.org>
Sent: Saturday, July 27, 2013 10:32 AM
Subject: Re: svn commit: r1507596 - /ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java
> could it be java version?
>
> here under:
> OpenJDK Runtime Environment (IcedTea6 1.12.6) (6b27-1.12.6-1ubuntu0.12.04.2)
> OpenJDK 64-Bit Server VM (build 20.0-b12, mixed mode)
>
> is working fine?
>
> Regards,
> Hans
>
> On 07/27/2013 03:15 PM, jleroux@apache.org wrote:
>> Author: jleroux
>> Date: Sat Jul 27 08:15:23 2013
>> New Revision: 1507596
>>
>> URL: http://svn.apache.org/r1507596
>> Log:
>> No functional change, temporary weird compile error workaround
>>
>> There is an issue with commit r1507200. It does not compile.
>> I wonder though why Buildbot did not detect it and also why my IDE (Eclipse) does not detect it and also why this fails, because the same scheme is successfully used elsewhere.
>>
>> The demo compilers (1.6.26, mine is 1.6.22, ) say:
>>
>> classes:
>> [javac16] Compiling 11 source files to /home/ofbiz/trunk/applications/party/build/classes
>> [javac16] /home/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java:2097: incompatible types
>> [javac16] found : java.util.Map<java.lang.String,org.ofbiz.entity.GenericValue>
>> [javac16] required: java.util.Map<java.lang.String,java.lang.Object>
>> [javac16] Map<String, Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
>> [javac16] ^
>> [javac16] /home/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java:2117: incompatible types
>> [javac16] found : java.util.Map<java.lang.String,org.ofbiz.entity.GenericValue>
>> [javac16] required: java.util.Map<java.lang.String,java.lang.Object>
>> [javac16] Map<String, Object> telecomNumber = UtilMisc.toMap("userLogin", userLogin);
>> [javac16] ^
>> [javac16] 2 errors
>>
>>
>> Of course replacing by
>> java.util.Map<java.lang.String,java.lang.Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
>> as the compilers suggest does not help. I temporarily replaced by
>>
>> Map<String, Object> postalAddress = new HashMap<String, Object>();
>> postalAddress.put("userLogin", userLogin);
>>
>> Unrelated: I set my IDE to automatically clean imports...
>>
>> Modified:
>> ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java
>>
>> Modified: ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java?rev=1507596&r1=1507595&r2=1507596&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java (original)
>> +++ ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java Sat Jul 27 08:15:23 2013
>> @@ -19,23 +19,24 @@
>>
>> package org.ofbiz.party.party;
>>
>> -import java.sql.Timestamp;
>> -import java.util.Collection;
>> -import java.util.List;
>> -import java.util.Locale;
>> -import java.util.Map;
>> import java.io.BufferedReader;
>> import java.io.IOException;
>> import java.io.StringReader;
>> import java.nio.ByteBuffer;
>> import java.nio.charset.Charset;
>> +import java.sql.Timestamp;
>> +import java.util.Collection;
>> +import java.util.HashMap;
>> +import java.util.List;
>> +import java.util.Locale;
>> +import java.util.Map;
>>
>> import javolution.util.FastList;
>> import javolution.util.FastMap;
>>
>> import org.apache.commons.csv.CSVFormat;
>> -import org.apache.commons.csv.CSVRecord;
>> import org.apache.commons.csv.CSVFormat.CSVFormatBuilder;
>> +import org.apache.commons.csv.CSVRecord;
>> import org.ofbiz.base.util.Debug;
>> import org.ofbiz.base.util.UtilDateTime;
>> import org.ofbiz.base.util.UtilGenerics;
>> @@ -2094,7 +2095,9 @@ public class PartyServices {
>> lastEmailAddress = rec.get("emailAddress");
>> }
>>
>> - Map<String, Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
>> + //Map<String, Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
>> + Map<String, Object> postalAddress = new HashMap<String, Object>();
>> + postalAddress.put("userLogin", userLogin);
>> Boolean postalAddressChanged = false;
>> if ("POSTAL_ADDRESS".equals(currentContactMechTypeId)) {
>> postalAddress.put("address1", rec.get("address1"));
>> @@ -2114,7 +2117,9 @@ public class PartyServices {
>> lastCountryGeoId = (String) postalAddress.get("countryGeoId");
>> }
>>
>> - Map<String, Object> telecomNumber = UtilMisc.toMap("userLogin", userLogin);
>> + //Map<String, Object> telecomNumber = UtilMisc.toMap("userLogin", userLogin);
>> + Map<String, Object> telecomNumber = new HashMap<String, Object>();
>> + postalAddress.put("userLogin", userLogin);
>> Boolean telecomNumberChanged = false;
>> if ("TELECOM_NUMBER".equals(currentContactMechTypeId)) {
>> telecomNumber.put("countryCode", rec.get("telCountryCode"));
>>
>>
>
Re: svn commit: r1507596 - /ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java
Posted by Hans Bakker <ma...@antwebsystems.com>.
could it be java version?
here under:
OpenJDK Runtime Environment (IcedTea6 1.12.6) (6b27-1.12.6-1ubuntu0.12.04.2)
OpenJDK 64-Bit Server VM (build 20.0-b12, mixed mode)
is working fine?
Regards,
Hans
On 07/27/2013 03:15 PM, jleroux@apache.org wrote:
> Author: jleroux
> Date: Sat Jul 27 08:15:23 2013
> New Revision: 1507596
>
> URL: http://svn.apache.org/r1507596
> Log:
> No functional change, temporary weird compile error workaround
>
> There is an issue with commit r1507200. It does not compile.
> I wonder though why Buildbot did not detect it and also why my IDE (Eclipse) does not detect it and also why this fails, because the same scheme is successfully used elsewhere.
>
> The demo compilers (1.6.26, mine is 1.6.22, ) say:
>
> classes:
> [javac16] Compiling 11 source files to /home/ofbiz/trunk/applications/party/build/classes
> [javac16] /home/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java:2097: incompatible types
> [javac16] found : java.util.Map<java.lang.String,org.ofbiz.entity.GenericValue>
> [javac16] required: java.util.Map<java.lang.String,java.lang.Object>
> [javac16] Map<String, Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
> [javac16] ^
> [javac16] /home/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java:2117: incompatible types
> [javac16] found : java.util.Map<java.lang.String,org.ofbiz.entity.GenericValue>
> [javac16] required: java.util.Map<java.lang.String,java.lang.Object>
> [javac16] Map<String, Object> telecomNumber = UtilMisc.toMap("userLogin", userLogin);
> [javac16] ^
> [javac16] 2 errors
>
>
> Of course replacing by
> java.util.Map<java.lang.String,java.lang.Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
> as the compilers suggest does not help. I temporarily replaced by
>
> Map<String, Object> postalAddress = new HashMap<String, Object>();
> postalAddress.put("userLogin", userLogin);
>
> Unrelated: I set my IDE to automatically clean imports...
>
> Modified:
> ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java
>
> Modified: ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java?rev=1507596&r1=1507595&r2=1507596&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java (original)
> +++ ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyServices.java Sat Jul 27 08:15:23 2013
> @@ -19,23 +19,24 @@
>
> package org.ofbiz.party.party;
>
> -import java.sql.Timestamp;
> -import java.util.Collection;
> -import java.util.List;
> -import java.util.Locale;
> -import java.util.Map;
> import java.io.BufferedReader;
> import java.io.IOException;
> import java.io.StringReader;
> import java.nio.ByteBuffer;
> import java.nio.charset.Charset;
> +import java.sql.Timestamp;
> +import java.util.Collection;
> +import java.util.HashMap;
> +import java.util.List;
> +import java.util.Locale;
> +import java.util.Map;
>
> import javolution.util.FastList;
> import javolution.util.FastMap;
>
> import org.apache.commons.csv.CSVFormat;
> -import org.apache.commons.csv.CSVRecord;
> import org.apache.commons.csv.CSVFormat.CSVFormatBuilder;
> +import org.apache.commons.csv.CSVRecord;
> import org.ofbiz.base.util.Debug;
> import org.ofbiz.base.util.UtilDateTime;
> import org.ofbiz.base.util.UtilGenerics;
> @@ -2094,7 +2095,9 @@ public class PartyServices {
> lastEmailAddress = rec.get("emailAddress");
> }
>
> - Map<String, Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
> + //Map<String, Object> postalAddress = UtilMisc.toMap("userLogin", userLogin);
> + Map<String, Object> postalAddress = new HashMap<String, Object>();
> + postalAddress.put("userLogin", userLogin);
> Boolean postalAddressChanged = false;
> if ("POSTAL_ADDRESS".equals(currentContactMechTypeId)) {
> postalAddress.put("address1", rec.get("address1"));
> @@ -2114,7 +2117,9 @@ public class PartyServices {
> lastCountryGeoId = (String) postalAddress.get("countryGeoId");
> }
>
> - Map<String, Object> telecomNumber = UtilMisc.toMap("userLogin", userLogin);
> + //Map<String, Object> telecomNumber = UtilMisc.toMap("userLogin", userLogin);
> + Map<String, Object> telecomNumber = new HashMap<String, Object>();
> + postalAddress.put("userLogin", userLogin);
> Boolean telecomNumberChanged = false;
> if ("TELECOM_NUMBER".equals(currentContactMechTypeId)) {
> telecomNumber.put("countryCode", rec.get("telCountryCode"));
>
>