Revert "Item13897: Fixed a bug affecting test_roundtrip."

This reverts commit 36ab951. Attachment name is not required - documented as representing all attachments.
foswiki · Jan 14, 2016 · 9a3dac3 · 9a3dac3
1 parent 36ab951
commit 9a3dac3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/UnitTestContrib/test/unit/AddressTests.pm b/UnitTestContrib/test/unit/AddressTests.pm
@@ -32,6 +32,7 @@ my %test_roundtrip_range = (
     ],
     tompaths => [
         undef,
+        ['attachment'],
         [ 'attachment', 'Attachment' ],
         [ 'attachment', 'Attach.ent' ],
         [ 'attachment', 'Atta.h.ent' ],

diff --git a/core/lib/Foswiki/Address.pm b/core/lib/Foswiki/Address.pm
@@ -1448,8 +1448,8 @@ sub isValid {
                           and scalar( @{ $this->{tompath} } ) )
                       if DEBUG;
                     ASSERT(
-                        not(    $this->{tompath}->[0]
-                            and $this->{tompath}->[0] eq 'attachment' )
+                        not(    $this->{topmath}->[0]
+                            and $this->{topmath}->[0] eq 'attachment' )
                     ) if DEBUG;
                     ASSERT( $pathtypes{ $this->{tompath}->[0] } ) if DEBUG;
                     $this->{type} =