Re: svn commit: r578518 - in /incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace: MRIDTransformationRule.java NamespaceRule.java

[Xavier Hanin <xa...@gmail.com>]

On 9/23/07, maartenc@apache.org <ma...@apache.org> wrote:
>
> Author: maartenc
> Date: Sat Sep 22 15:29:30 2007
> New Revision: 578518
>
> URL: http://svn.apache.org/viewvc?rev=578518&view=rev
> Log:
> Fixed some problems caused by the code cleanup.


Thanks, it seems I still need to be more careful when I do code cleanup...
Sorry about that.

Xavier

Modified:
>
>     incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/MRIDTransformationRule.java
>
>     incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/NamespaceRule.java
>
> Modified:
> incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/MRIDTransformationRule.java
> URL:
> http://svn.apache.org/viewvc/incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/MRIDTransformationRule.java?rev=578518&r1=578517&r2=578518&view=diff
>
> ==============================================================================
> ---
> incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/MRIDTransformationRule.java
> (original)
> +++
> incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/MRIDTransformationRule.java
> Sat Sep 22 15:29:30 2007
> @@ -120,7 +120,7 @@
>      }
>
>      public void addDest(MRIDRule dest) {
> -        if (dest != null) {
> +        if (this.dest != null) {
>              throw new IllegalArgumentException("only one dest is allowed
> per mapping");
>          }
>          this.dest = dest;
>
> Modified:
> incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/NamespaceRule.java
> URL:
> http://svn.apache.org/viewvc/incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/NamespaceRule.java?rev=578518&r1=578517&r2=578518&view=diff
>
> ==============================================================================
> ---
> incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/NamespaceRule.java
> (original)
> +++
> incubator/ivy/core/trunk/src/java/org/apache/ivy/plugins/namespace/NamespaceRule.java
> Sat Sep 22 15:29:30 2007
> @@ -31,7 +31,7 @@
>      }
>
>      public void addFromsystem(MRIDTransformationRule fromSystem) {
> -        if (fromSystem != null) {
> +        if (this.fromSystem != null) {
>              throw new IllegalArgumentException("only one fromsystem is
> allowed per rule");
>          }
>          this.fromSystem = fromSystem;
> @@ -42,7 +42,7 @@
>      }
>
>      public void addTosystem(MRIDTransformationRule toSystem) {
> -        if (toSystem != null) {
> +        if (this.toSystem != null) {
>              throw new IllegalArgumentException("only one tosystem is
> allowed per rule");
>          }
>          this.toSystem = toSystem;
>
>
>


-- 
Xavier Hanin - Independent Java Consultant
http://xhab.blogspot.com/
http://incubator.apache.org/ivy/
http://www.xoocode.org/