Fix for handling self-closing tags.

Author: gregjacobs

src/Autolinker.js

@@ -243,7 +243,7 @@
 						'(?:\\s*=\\s*' + attrValueRegex.source + ')?',  // optional '=[value]'
 					')*',
 
-					'\\s*',  // any trailing spaces before the closing '>'
+					'\\s*/?',  // any trailing spaces and optional '/' before the closing '>'
 				'>'
 			].join( "" ), 'g' );
 		} )(),

tests/AutolinkerSpec.js

@@ -518,11 +518,26 @@ describe( "Autolinker", function() {
 			} );
 
 
-			it( "should NOT automatically link URLs within existing HTML tags", function() {
+			it( "should NOT automatically link URLs within the attributes of existing HTML tags", function() {
 				var result = autolinker.link( '<p>Joe went to <a href="http://www.yahoo.com">yahoo</a></p>' );
 				expect( result ).toBe( '<p>Joe went to <a href="http://www.yahoo.com">yahoo</a></p>' );
 			} );
 
+		
+			it( "should NOT automatically link URLs within the attributes of existing HTML tags when there are prefixed or suffixed spaces in the attribute values", function() {
+				var result = autolinker.link( '<p>Joe went to <a href=" http://www.yahoo.com">yahoo</a></p>' );
+				expect( result ).toBe( '<p>Joe went to <a href=" http://www.yahoo.com">yahoo</a></p>' );
+				
+				var result2 = autolinker.link( '<p>Joe went to <a href="http://www.yahoo.com ">yahoo</a></p>' );
+				expect( result2 ).toBe( '<p>Joe went to <a href="http://www.yahoo.com ">yahoo</a></p>' );
+			} );
+			
+			
+			it( "should NOT automatically link URLs within self-closing tags", function() {
+				var result = autolinker.link( 'Just a flower image <img src="https://farm9.staticflickr.com/8378/8578790632_83c6471f3f_b.jpg" />' );
+				expect( result ).toBe( 'Just a flower image <img src="https://farm9.staticflickr.com/8378/8578790632_83c6471f3f_b.jpg" />' );
+			} );
+			
 
 			it( "should NOT automatically link a URL found within the inner text of a pre-existing anchor tag", function() {
 				var result = autolinker.link( '<p>Joe went to <a href="http://www.yahoo.com">yahoo.com</a></p> yesterday.' );
@@ -548,6 +563,12 @@ describe( "Autolinker", function() {
 			} );
 
 
+			it( "should NOT automatically link an image tag with a URL inside of it, when it has another attribute which has extraneous spaces surround its value (Issue #45)", function() {
+				var result = autolinker.link( "Testing <img src='http://terryshoemaker.files.wordpress.com/2013/03/placeholder1.jpg' style=' height: 22px; background-color: rgb(0, 188, 204); border-radius: 7px; padding: 2px; margin: 0px 2px;'>" );
+				expect( result ).toBe( "Testing <img src='http://terryshoemaker.files.wordpress.com/2013/03/placeholder1.jpg' style=' height: 22px; background-color: rgb(0, 188, 204); border-radius: 7px; padding: 2px; margin: 0px 2px;'>" );
+			} );
+			
+			
 			it( "should allow the full range of HTML attribute name characters as specified in the W3C HTML syntax document (http://www.w3.org/TR/html-markup/syntax.html)", function() {
 				// Note: We aren't actually expecting the HTML to be modified by this test
 				var inAndOutHtml = '<ns:p>Foo <a data-qux-="" href="http://www.example.com">Bar<\/a> Baz<\/ns:p>';