breakout_parser 0.0.18 → 0.0.19

data/ChangeLog

@@ -1,3 +1,8 @@
+[ 0.0.19 ]
+
+    * skip redundant '/' in making relative urls absolute
+        (https://www.assembla.com/spaces/breakout/tickets/6513)
+
 [ 0.0.18 ]
     
     * added README

data/ext/breakout_parser/parser.tab.c

@@ -2273,6 +2273,7 @@ process_url_link(const char*target,const char* proto){
         // relative link
         if( absolute_urls && site_url ){
             concat2(site_url);
+            if(*(bufptr-1) == '/') bufptr--; // skip redundant '/'
         }
     }
     for(c=target; *c && *c != ']' && *c != '|'; c++) concat_raw_char(*c);

data/ext/breakout_parser/parser.y

@@ -342,6 +342,7 @@ process_url_link(const char*target,const char* proto){
         // relative link
         if( absolute_urls && site_url ){
             concat2(site_url);
+            if(*(bufptr-1) == '/') bufptr--; // skip redundant '/'
         }
     }
     for(c=target; *c && *c != ']' && *c != '|'; c++) concat_raw_char(*c);

data/spec/parser_spec.rb

@@ -939,6 +939,12 @@ describe 'BreakoutParser' do
           parse("[[url:/rel|text]]", :absolute_urls => v, :site_url => 'http://www.ru').should == 
             '<a rel="nofollow" href="http://www.ru/rel">text</a>'
         end
+        it "should strip redundant '/' when absolute_urls = #{v.inspect}" do
+          parse("[[url:/rel]]", :absolute_urls => v, :site_url => 'http://www.ru/').should == 
+            '<a rel="nofollow" href="http://www.ru/rel">/rel</a>'
+          parse("[[url:/rel|text]]", :absolute_urls => v, :site_url => 'http://www.ru/').should == 
+            '<a rel="nofollow" href="http://www.ru/rel">text</a>'
+        end
       end
       # 'false' values
       [false, nil].each do |v|

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: breakout_parser
 version: !ruby/object:Gem::Version 
-  hash: 59
+  hash: 57
   prerelease: false
   segments: 
   - 0
   - 0
-  - 18
-  version: 0.0.18
+  - 19
+  version: 0.0.19
 platform: ruby
 authors: 
 - Andrey "Zed" Zaikin
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-08-25 00:00:00 +04:00
+date: 2010-08-29 00:00:00 +04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency