Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Message-ID: <22302501.1187155410634.JavaMail.jira@brutus>
Date: Tue, 14 Aug 2007 22:23:30 -0700 (PDT)
From: "Henri Yandell (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Subject: [jira] Updated: (COLLECTIONS-256) ListUtils.transformedList(list,
 transformer) doesn't transform
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Henri Yandell updated COLLECTIONS-256:
--------------------------------------

    Fix Version/s: 3.3

Given that javadoc states:

"Returns a transformed list backed by the given list."

Then this seems like a definite bug that needs fixing.

> ListUtils.transformedList(list, transformer) doesn't transform
> --------------------------------------------------------------
>
>                 Key: COLLECTIONS-256
>                 URL: https://issues.apache.org/jira/browse/COLLECTIONS-256
>             Project: Commons Collections
>          Issue Type: Bug
>          Components: List
>    Affects Versions: 3.2
>         Environment: Java 1.5
>            Reporter: Kurtis Williams
>             Fix For: 3.3
>
>
> The static method ListUtils.transformedList(List list, Transformer trans) does not apply the transformer before returning the list.  I've traced into the code and it falls all the way through to the TransformedCollection constructor which has documentation that states that items existing in the list are NOT transformed.  This yields bizarre unexpected behavior for this method.
> Conversely, the ListUtils.predicatedList(List list, Predicate pred) DOES apply the predicate.  These methods are very similar but behave inconsistently.
> If the transformer is not applied to the list, the method's usefulness is somewhat limited and causes the user to create an extra unwanted list in some circumstances:
> // To transform an existing list
> List myList = ... some list to be transformed ...
> List transformed = ListUtils.transformedList(new ArrayList(), myTransformer);
> transformed.addAll(myList);
> I would expect a much shorter implementation such as:
> List transformed = ListUtils.transformedList(myList, myTransformer)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.