Mailing-List: contact camel-dev-help@activemq.apache.org; run by ezmlm
Precedence: bulk
Reply-To: camel-dev@activemq.apache.org
Message-ID: <212075505.1206872012909.JavaMail.jira@brutus>
Date: Sun, 30 Mar 2008 03:13:32 -0700 (PDT)
From: "Claus Ibsen (JIRA)" <jira@apache.org>
To: camel-dev@activemq.apache.org
Subject: [jira] Updated: (CAMEL-272) Rework of File Component to Remove
 Tangles and simplify code
In-Reply-To: <23365236.1198103608154.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit


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

Claus Ibsen updated CAMEL-272:
------------------------------

    Fix Version/s: 1.4.0
                       (was: 1.3.0)

> Rework of File Component to Remove Tangles and simplify code
> ------------------------------------------------------------
>
>                 Key: CAMEL-272
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-272
>             Project: Apache Camel
>          Issue Type: Improvement
>          Components: camel-core
>            Reporter: Christian Schneider
>            Assignee: Hadrian Zbarcea
>             Fix For: 1.4.0
>
>         Attachments: file-test.zip, file.zip, new no tangles.png, old.png, patch.txt, patchfile.txt
>
>
> After my first smaller rework in issue camel-262 I now completely reworked the file component to remove any cyclic references and clean up the code.
> The first thing I did was separating the FileComponent from the other classes in package file by moving them to a new package .impl. This way I was able to remove the cycle between the strategy package and the file package.
> The second step was to move all config data from FileEndPoint to a new class FileEndPointConfig. This way I was able to remove the cycle between FileEndPoint, FileProducer and FileConsumer.
> Next I removed FileMessage and FileExchange and instead relied on the default objects. The file is simply stored in DefaultMessage.body. There was even a comment in the code asking if FileExchange was necessary before. So I hope this is ok?
> I had to adapt the tests to the new structure but only changed the tests where it was absolutely necessary. All tests run fine.
> What do you think?

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