[jira] [Updated] (VALIDATOR-356) IDN.toASCII drops trailing dot in Java 6 & 7

2015-01-14 Thread Sebb (JIRA)

 [ 
https://issues.apache.org/jira/browse/VALIDATOR-356?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb updated VALIDATOR-356:
---
Affects Version/s: 1.4.1 Release

> IDN.toASCII drops trailing dot in Java 6 & 7
> 
>
> Key: VALIDATOR-356
> URL: https://issues.apache.org/jira/browse/VALIDATOR-356
> Project: Commons Validator
>  Issue Type: Bug
>Affects Versions: 1.4.1 Release
>Reporter: Sebb
> Fix For: 1.5.0
>
>
> The Java 6 method {{String java.net.IDN.toASCII(String)}} drops the final 
> input character if it is a dot (or equivalent, see RFC3490 3.1 1)
> This causes problems for authority and domain validation.
> This problem has been seen in 
> Java(TM) SE Runtime Environment (build 1.6.0_65-b14-462-11M4609)
> and
> Java(TM) SE Runtime Environment (build 1.7.0_65-b17)
> but appears to be fixed in
> Java(TM) SE Runtime Environment (build 1.8.0_25-b17)
> Note also that the Java 8 version throws 
> java.lang.IllegalArgumentException: Empty label is not a legal name
> if the argument contains two consecutive dots.
> Fix to follow.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Resolved] (VALIDATOR-356) IDN.toASCII drops trailing dot in Java 6 & 7

2015-01-14 Thread Sebb (JIRA)

 [ 
https://issues.apache.org/jira/browse/VALIDATOR-356?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb resolved VALIDATOR-356.

   Resolution: Fixed
Fix Version/s: 1.5.0

URL: http://svn.apache.org/r1651873
Log:
VALIDATOR-356 IDN.toASCII drops trailing dot in Java 6 & 7

Modified:
commons/proper/validator/trunk/src/changes/changes.xml

commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/routines/DomainValidator.java

commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/DomainValidatorTest.java

commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java


> IDN.toASCII drops trailing dot in Java 6 & 7
> 
>
> Key: VALIDATOR-356
> URL: https://issues.apache.org/jira/browse/VALIDATOR-356
> Project: Commons Validator
>  Issue Type: Bug
>Affects Versions: 1.4.1 Release
>Reporter: Sebb
> Fix For: 1.5.0
>
>
> The Java 6 method {{String java.net.IDN.toASCII(String)}} drops the final 
> input character if it is a dot (or equivalent, see RFC3490 3.1 1)
> This causes problems for authority and domain validation.
> This problem has been seen in 
> Java(TM) SE Runtime Environment (build 1.6.0_65-b14-462-11M4609)
> and
> Java(TM) SE Runtime Environment (build 1.7.0_65-b17)
> but appears to be fixed in
> Java(TM) SE Runtime Environment (build 1.8.0_25-b17)
> Note also that the Java 8 version throws 
> java.lang.IllegalArgumentException: Empty label is not a legal name
> if the argument contains two consecutive dots.
> Fix to follow.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (VALIDATOR-356) IDN.toASCII drops trailing dot in Java 6 & 7

2015-01-14 Thread Sebb (JIRA)
Sebb created VALIDATOR-356:
--

 Summary: IDN.toASCII drops trailing dot in Java 6 & 7
 Key: VALIDATOR-356
 URL: https://issues.apache.org/jira/browse/VALIDATOR-356
 Project: Commons Validator
  Issue Type: Bug
Reporter: Sebb


The Java 6 method {{String java.net.IDN.toASCII(String)}} drops the final input 
character if it is a dot (or equivalent, see RFC3490 3.1 1)

This causes problems for authority and domain validation.

This problem has been seen in 
Java(TM) SE Runtime Environment (build 1.6.0_65-b14-462-11M4609)
and
Java(TM) SE Runtime Environment (build 1.7.0_65-b17)
but appears to be fixed in
Java(TM) SE Runtime Environment (build 1.8.0_25-b17)

Note also that the Java 8 version throws 
java.lang.IllegalArgumentException: Empty label is not a legal name
if the argument contains two consecutive dots.

Fix to follow.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DBCP-423) PoolingDataSource should implement Closeable

2015-01-14 Thread Thomas Neidhart (JIRA)

[ 
https://issues.apache.org/jira/browse/DBCP-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14277809#comment-14277809
 ] 

Thomas Neidhart commented on DBCP-423:
--

It is unfortunate that we can not touch the exception behavior but this should 
be done in another major release imho.

On a second thought, the approach in BDS should be further improved as it now 
catches a generic Exception and wraps it into a SQLException. If the pool will 
throw an exception in its close method it will most likely be already a 
SQLException (coming from the PoolableConnection implementation for example).

Thus we should not wrap a SQLException inside a SQLException but propagate the 
SQLException. The same is already done in the PoolableConnection.close() method.

> PoolingDataSource should implement Closeable
> 
>
> Key: DBCP-423
> URL: https://issues.apache.org/jira/browse/DBCP-423
> Project: Commons Dbcp
>  Issue Type: Improvement
>Affects Versions: 2.0
>Reporter: Christian Schneider
> Fix For: 2.1
>
> Attachments: DBCP-423.patch
>
>
> Currently PoolingDataSource only implements DataSource. 
> I have the following case in ops4j pax-jdbc. I offer a DataSourceFactory in 
> one bundle that can create a pooling DataSource.
> Then in another bundle I create DataSources based on config in 
> ConfigurationAdmin. So when the config appears I create the DataSource, when 
> the config goes away I have to destroy it.
> It is important to correctly dispose the DataSource as the pool has to be 
> closed. As I can not depend on dbcp in the bundle that destroys the 
> DataSource I currently have no simple way to destroy the DataSource.
> This is where I create the DataSource:
> https://github.com/ops4j/org.ops4j.pax.jdbc/blob/master/pax-jdbc-pool/src/main/java/org/ops4j/pax/jdbc/pool/impl/PooledDataSourceFactory.java
> I was able to solve it by a kind of hack. I extended the PoolingDataSource 
> with a class that supports Closeable. So from the other bundle I could check 
> if the object implements Closeable and call close.
> It would be a lot easier if PoolingDataSource would implement Closeable like 
> I did. The same is true for GenericObjectPool and the other pools. If they 
> would implement Closeable then handling them would be a lot easier.
> They already support a close method so it would be a small change.
> 
> public class CloseablePoolingDataSource extends 
> PoolingDataSource implements Closeable {
> public CloseablePoolingDataSource(ObjectPool pool) {
> super(pool);
> }
> @Override
> public void close() throws IOException {
> getPool().close();
> }
> }



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (MATH-1195) Move the "FastMathTestPerformance" source

2015-01-14 Thread Gilles (JIRA)
Gilles created MATH-1195:


 Summary: Move the "FastMathTestPerformance" source
 Key: MATH-1195
 URL: https://issues.apache.org/jira/browse/MATH-1195
 Project: Commons Math
  Issue Type: Sub-task
Reporter: Gilles
Assignee: Gilles
Priority: Minor
 Fix For: 3.4.2


"FastMathTestPerformance" is stored in the "test" part of the repository but is 
not a test, it's a benchmark.
Its code should be moved to the "src/userguide" part of the repository, where 
it can be updated to generate a report in a format suitable for publishing on 
the web site.




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (MATH-1194) Create "Commons Math Reports"

2015-01-14 Thread Gilles (JIRA)
Gilles created MATH-1194:


 Summary: Create "Commons Math Reports"
 Key: MATH-1194
 URL: https://issues.apache.org/jira/browse/MATH-1194
 Project: Commons Math
  Issue Type: Task
Reporter: Gilles
Priority: Minor


The "examples" codes in "src/userguide", in addition to illustrating how to use 
the CM library, could contain applications that create reports on various 
aspects of CM (e.g. benchmarks).

The format of the generated output is yet TBD.




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (MATH-1193) Root-mean-square computation

2015-01-14 Thread Gilles (JIRA)
Gilles created MATH-1193:


 Summary: Root-mean-square computation
 Key: MATH-1193
 URL: https://issues.apache.org/jira/browse/MATH-1193
 Project: Commons Math
  Issue Type: New Feature
Reporter: Gilles
Priority: Minor
 Fix For: 3.4.2


Add root-mean-square (a.k.a. quadratic mean) computation in
* DescriptiveStatistics
* SummaryStatistics



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (COLLECTIONS-530) Rejecting items on predicate failure without throwing an Exception

2015-01-14 Thread Erik (JIRA)

 [ 
https://issues.apache.org/jira/browse/COLLECTIONS-530?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Erik updated COLLECTIONS-530:
-
Attachment: (was: COLLECTIONS-530-v0.0.1.alpha.zip)

> Rejecting items on predicate failure without throwing an Exception
> --
>
> Key: COLLECTIONS-530
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-530
> Project: Commons Collections
>  Issue Type: Improvement
>  Components: List
>Affects Versions: 4.0
>Reporter: Erik
> Fix For: 4.x
>
> Attachments: COLLECTIONS-530-v0.0.1.alpha.zip
>
>
> The PredicatedList class doesn't allow entries that fail the predicate, but 
> throws an Exception on entry.
> The problem I have with this, is that it places the onus of filtering out 
> invalid entries on the caller.
> I typically add items in a loop. The item added is the result of a method 
> call (which returns null if it can't create one).
> This problem is so common for me that I have created my own FilteredList 
> class that simply ignores invalid entries.
> I would like the PredicatedList class to be capable of rejecting items 
> without throwing an exception.
> I don't mind writing the code for this, but there are a great many ways in 
> which this can be done.
> So I was wondering what the interface should look like.
> Separate FilteredList class.
> Works, but seems a little verbose for the purpose
> New factory method: filteredList(List list, Predicate 
> predicate) 
> Nice and simple, but doesn't allow extension; other ways of dealing with 
> predicate failure.
> New factory method with enum: predicatedList(List list, Predicate T> predicate, PredicateFailEnum action)
> More verbose to use and adds an extra class, but allows more alternative ways 
> to deal with predicate failure.
> One more nice thing is that it might be less confusing, 
> because choosing between predicatedList and the above filteredList might not 
> be so obvious.
> New factory method with interface: filteredList(List list, Predicate super T> predicate, PredicateFailInterface action)
> Complex, but the most flexible way of dealing with predicate failure.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Issue Comment Deleted] (COLLECTIONS-530) Rejecting items on predicate failure without throwing an Exception

2015-01-14 Thread Erik (JIRA)

 [ 
https://issues.apache.org/jira/browse/COLLECTIONS-530?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Erik updated COLLECTIONS-530:
-
Comment: was deleted

(was: first implementation suggestion)

> Rejecting items on predicate failure without throwing an Exception
> --
>
> Key: COLLECTIONS-530
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-530
> Project: Commons Collections
>  Issue Type: Improvement
>  Components: List
>Affects Versions: 4.0
>Reporter: Erik
> Fix For: 4.x
>
> Attachments: COLLECTIONS-530-v0.0.1.alpha.zip
>
>
> The PredicatedList class doesn't allow entries that fail the predicate, but 
> throws an Exception on entry.
> The problem I have with this, is that it places the onus of filtering out 
> invalid entries on the caller.
> I typically add items in a loop. The item added is the result of a method 
> call (which returns null if it can't create one).
> This problem is so common for me that I have created my own FilteredList 
> class that simply ignores invalid entries.
> I would like the PredicatedList class to be capable of rejecting items 
> without throwing an exception.
> I don't mind writing the code for this, but there are a great many ways in 
> which this can be done.
> So I was wondering what the interface should look like.
> Separate FilteredList class.
> Works, but seems a little verbose for the purpose
> New factory method: filteredList(List list, Predicate 
> predicate) 
> Nice and simple, but doesn't allow extension; other ways of dealing with 
> predicate failure.
> New factory method with enum: predicatedList(List list, Predicate T> predicate, PredicateFailEnum action)
> More verbose to use and adds an extra class, but allows more alternative ways 
> to deal with predicate failure.
> One more nice thing is that it might be less confusing, 
> because choosing between predicatedList and the above filteredList might not 
> be so obvious.
> New factory method with interface: filteredList(List list, Predicate super T> predicate, PredicateFailInterface action)
> Complex, but the most flexible way of dealing with predicate failure.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (COLLECTIONS-530) Rejecting items on predicate failure without throwing an Exception

2015-01-14 Thread Erik (JIRA)

 [ 
https://issues.apache.org/jira/browse/COLLECTIONS-530?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Erik updated COLLECTIONS-530:
-
Attachment: COLLECTIONS-530-v0.0.1.alpha.zip

> Rejecting items on predicate failure without throwing an Exception
> --
>
> Key: COLLECTIONS-530
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-530
> Project: Commons Collections
>  Issue Type: Improvement
>  Components: List
>Affects Versions: 4.0
>Reporter: Erik
> Fix For: 4.x
>
> Attachments: COLLECTIONS-530-v0.0.1.alpha.zip
>
>
> The PredicatedList class doesn't allow entries that fail the predicate, but 
> throws an Exception on entry.
> The problem I have with this, is that it places the onus of filtering out 
> invalid entries on the caller.
> I typically add items in a loop. The item added is the result of a method 
> call (which returns null if it can't create one).
> This problem is so common for me that I have created my own FilteredList 
> class that simply ignores invalid entries.
> I would like the PredicatedList class to be capable of rejecting items 
> without throwing an exception.
> I don't mind writing the code for this, but there are a great many ways in 
> which this can be done.
> So I was wondering what the interface should look like.
> Separate FilteredList class.
> Works, but seems a little verbose for the purpose
> New factory method: filteredList(List list, Predicate 
> predicate) 
> Nice and simple, but doesn't allow extension; other ways of dealing with 
> predicate failure.
> New factory method with enum: predicatedList(List list, Predicate T> predicate, PredicateFailEnum action)
> More verbose to use and adds an extra class, but allows more alternative ways 
> to deal with predicate failure.
> One more nice thing is that it might be less confusing, 
> because choosing between predicatedList and the above filteredList might not 
> be so obvious.
> New factory method with interface: filteredList(List list, Predicate super T> predicate, PredicateFailInterface action)
> Complex, but the most flexible way of dealing with predicate failure.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (COLLECTIONS-530) Rejecting items on predicate failure without throwing an Exception

2015-01-14 Thread Erik (JIRA)

 [ 
https://issues.apache.org/jira/browse/COLLECTIONS-530?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Erik updated COLLECTIONS-530:
-
Attachment: COLLECTIONS-530-v0.0.1.alpha.zip

first implementation suggestion

> Rejecting items on predicate failure without throwing an Exception
> --
>
> Key: COLLECTIONS-530
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-530
> Project: Commons Collections
>  Issue Type: Improvement
>  Components: List
>Affects Versions: 4.0
>Reporter: Erik
> Fix For: 4.x
>
> Attachments: COLLECTIONS-530-v0.0.1.alpha.zip
>
>
> The PredicatedList class doesn't allow entries that fail the predicate, but 
> throws an Exception on entry.
> The problem I have with this, is that it places the onus of filtering out 
> invalid entries on the caller.
> I typically add items in a loop. The item added is the result of a method 
> call (which returns null if it can't create one).
> This problem is so common for me that I have created my own FilteredList 
> class that simply ignores invalid entries.
> I would like the PredicatedList class to be capable of rejecting items 
> without throwing an exception.
> I don't mind writing the code for this, but there are a great many ways in 
> which this can be done.
> So I was wondering what the interface should look like.
> Separate FilteredList class.
> Works, but seems a little verbose for the purpose
> New factory method: filteredList(List list, Predicate 
> predicate) 
> Nice and simple, but doesn't allow extension; other ways of dealing with 
> predicate failure.
> New factory method with enum: predicatedList(List list, Predicate T> predicate, PredicateFailEnum action)
> More verbose to use and adds an extra class, but allows more alternative ways 
> to deal with predicate failure.
> One more nice thing is that it might be less confusing, 
> because choosing between predicatedList and the above filteredList might not 
> be so obvious.
> New factory method with interface: filteredList(List list, Predicate super T> predicate, PredicateFailInterface action)
> Complex, but the most flexible way of dealing with predicate failure.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (COLLECTIONS-530) Rejecting items on predicate failure without throwing an Exception

2015-01-14 Thread Erik (JIRA)

[ 
https://issues.apache.org/jira/browse/COLLECTIONS-530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14277665#comment-14277665
 ] 

Erik commented on COLLECTIONS-530:
--

I created a suggested implementation.
Not sure what the procedure is, but I will attach it to this issue, tell me 
what you think.

> Rejecting items on predicate failure without throwing an Exception
> --
>
> Key: COLLECTIONS-530
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-530
> Project: Commons Collections
>  Issue Type: Improvement
>  Components: List
>Affects Versions: 4.0
>Reporter: Erik
> Fix For: 4.x
>
>
> The PredicatedList class doesn't allow entries that fail the predicate, but 
> throws an Exception on entry.
> The problem I have with this, is that it places the onus of filtering out 
> invalid entries on the caller.
> I typically add items in a loop. The item added is the result of a method 
> call (which returns null if it can't create one).
> This problem is so common for me that I have created my own FilteredList 
> class that simply ignores invalid entries.
> I would like the PredicatedList class to be capable of rejecting items 
> without throwing an exception.
> I don't mind writing the code for this, but there are a great many ways in 
> which this can be done.
> So I was wondering what the interface should look like.
> Separate FilteredList class.
> Works, but seems a little verbose for the purpose
> New factory method: filteredList(List list, Predicate 
> predicate) 
> Nice and simple, but doesn't allow extension; other ways of dealing with 
> predicate failure.
> New factory method with enum: predicatedList(List list, Predicate T> predicate, PredicateFailEnum action)
> More verbose to use and adds an extra class, but allows more alternative ways 
> to deal with predicate failure.
> One more nice thing is that it might be less confusing, 
> because choosing between predicatedList and the above filteredList might not 
> be so obvious.
> New factory method with interface: filteredList(List list, Predicate super T> predicate, PredicateFailInterface action)
> Complex, but the most flexible way of dealing with predicate failure.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (MATH-1192) "Descriptive statistics" has a non-final protected field

2015-01-14 Thread Gilles (JIRA)
Gilles created MATH-1192:


 Summary: "Descriptive statistics" has a non-final protected field
 Key: MATH-1192
 URL: https://issues.apache.org/jira/browse/MATH-1192
 Project: Commons Math
  Issue Type: Bug
Affects Versions: 3.4
Reporter: Gilles
Priority: Trivial
 Fix For: 4.0


Field "windowSize" should be private (there is setter method).





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (CSV-147) Better error handling in CSV, see included code

2015-01-14 Thread Steven Peterson (JIRA)

 [ 
https://issues.apache.org/jira/browse/CSV-147?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Steven Peterson updated CSV-147:

Attachment: csv-exceptionsupport.patch

Please see attached patch containing a new test in CSVParserTest and a new 
method for CSVParser that supports exception reporting and recovery.

> Better error handling in CSV, see included code
> ---
>
> Key: CSV-147
> URL: https://issues.apache.org/jira/browse/CSV-147
> Project: Commons CSV
>  Issue Type: Improvement
>  Components: Parser
>Affects Versions: 1.1
>Reporter: Steven Peterson
>  Labels: easyfix
> Fix For: Discussion
>
> Attachments: csv-exceptionsupport.patch
>
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> When parsing long files, and there is problem with parsing the data, it would 
> be very useful to know the exact data that is bad when an exception is 
> thrown.  As CSVParser maintains a portion of the current record that was read 
> when an exception is thrown (in the 'record' property), can we make this 
> information public through something like this:
> {code:java}
> public String[] getLastRecordData() {
>   return this.record.toArray(new String[this.record.size()]);
> }
> {code}
> With a method like this, it would be easy to pull in part of the data that 
> was in the faulty record after an exception was thrown, aiding in cleaning up 
> the data.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (LANG-1082) Add option to disable the "objectsTriviallyEqual" test in DiffBuilder.

2015-01-14 Thread Jonathan Baker (JIRA)

 [ 
https://issues.apache.org/jira/browse/LANG-1082?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jonathan Baker updated LANG-1082:
-
Summary: Add option to disable the "objectsTriviallyEqual" test in 
DiffBuilder.  (was: Allow the objectsTriviallyEqual test to be disabled.)

> Add option to disable the "objectsTriviallyEqual" test in DiffBuilder.
> --
>
> Key: LANG-1082
> URL: https://issues.apache.org/jira/browse/LANG-1082
> Project: Commons Lang
>  Issue Type: Improvement
>  Components: lang.builder.*
>Affects Versions: 3.3.2
>Reporter: Jonathan Baker
>Priority: Trivial
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> For example, a bean (e.g. a JPA record) has equals() overridden with a 
> implementation that tests its "id" property.   If two beans have the same ID, 
> they correspond to the same record in the database.  However, without 
> tricking the objectsTriviallyEqual check in the DiffBuilder constructor, 
> there is no way to build a DiffResult using "before" and "after" instances 
> with changes to other properties (name, modification time, etc).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (LANG-1082) Allow the objectsTriviallyEqual test to be disabled.

2015-01-14 Thread Jonathan Baker (JIRA)
Jonathan Baker created LANG-1082:


 Summary: Allow the objectsTriviallyEqual test to be disabled.
 Key: LANG-1082
 URL: https://issues.apache.org/jira/browse/LANG-1082
 Project: Commons Lang
  Issue Type: Improvement
  Components: lang.builder.*
Affects Versions: 3.3.2
Reporter: Jonathan Baker
Priority: Trivial


For example, a bean (e.g. a JPA record) has equals() overridden with a 
implementation that tests its "id" property.   If two beans have the same ID, 
they correspond to the same record in the database.  However, without tricking 
the objectsTriviallyEqual check in the DiffBuilder constructor, there is no way 
to build a DiffResult using "before" and "after" instances with changes to 
other properties (name, modification time, etc).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (LANG-1081) DiffBuilder.append(String, Object left, Object right) does not do a left.equals(right) check.

2015-01-14 Thread Jonathan Baker (JIRA)

 [ 
https://issues.apache.org/jira/browse/LANG-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jonathan Baker updated LANG-1081:
-
Attachment: DiffBuilder-equals_test_in_append_Objects.patch
DiffBuilderAppendsNonEqualObjectsFixed.java
DiffBuilderAppendsNonEqualObjects.java

Attached are examples of how to detect and correct the issue.

> DiffBuilder.append(String, Object left, Object right) does not do a 
> left.equals(right) check.
> -
>
> Key: LANG-1081
> URL: https://issues.apache.org/jira/browse/LANG-1081
> Project: Commons Lang
>  Issue Type: Bug
>  Components: lang.builder.*
>Affects Versions: 3.3.2
>Reporter: Jonathan Baker
>Priority: Trivial
> Attachments: DiffBuilder-equals_test_in_append_Objects.patch, 
> DiffBuilderAppendsNonEqualObjects.java, 
> DiffBuilderAppendsNonEqualObjectsFixed.java
>
>   Original Estimate: 1m
>  Remaining Estimate: 1m
>
> Only testing == without testing equals() seems wrong.
> Should an equals() test be added after determining that the parameters are 
> not arrays?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (LANG-1081) DiffBuilder.append(String, Object left, Object right) does not do a left.equals(right) check.

2015-01-14 Thread Jonathan Baker (JIRA)
Jonathan Baker created LANG-1081:


 Summary: DiffBuilder.append(String, Object left, Object right) 
does not do a left.equals(right) check.
 Key: LANG-1081
 URL: https://issues.apache.org/jira/browse/LANG-1081
 Project: Commons Lang
  Issue Type: Bug
  Components: lang.builder.*
Affects Versions: 3.3.2
Reporter: Jonathan Baker
Priority: Trivial


Only testing == without testing equals() seems wrong.
Should an equals() test be added after determining that the parameters are not 
arrays?




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)