You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tomee.apache.org by David Salter <da...@davidsalter.uk> on 2020/07/07 16:43:02 UTC

TOMEE-1417 Fix

Hi,

I've submitted a PR (https://github.com/apache/tomee/pull/671) for 
TOMEE-1417.


The code tests that 2 Properties objects are the same, but doesn't flag 
errors correctly when they contain the same keys but different values.  
I've fixed the logic so it now works in this case.

Please can someone take a look and merge if its OK?


Cheers,

Dave.

-- 
David Salter

Re: TOMEE-1417 Fix

Posted by Daniel Dias Dos Santos <da...@gmail.com>.

Hello David,

great, thanks .   :  )
--

*Daniel Dias dos Santos*
Java Developer
SouJava & JCP Member
GitHub: https://github.com/Daniel-Dos
Linkedin: www.linkedin.com/in/danieldiasjava
Twitter: http://twitter.com/danieldiasjava


Em ter., 7 de jul. de 2020 às 13:43, David Salter <da...@davidsalter.uk>
escreveu:

> Hi,
>
> I've submitted a PR (https://github.com/apache/tomee/pull/671) for
> TOMEE-1417.
>
>
> The code tests that 2 Properties objects are the same, but doesn't flag
> errors correctly when they contain the same keys but different values.
> I've fixed the logic so it now works in this case.
>
> Please can someone take a look and merge if its OK?
>
>
> Cheers,
>
> Dave.
>
> --
> David Salter
>
>
>