bookbindery 9.10.0 → 9.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 99c1606223bcc841a9f5c9abd0f8bb3df7212e13
-  data.tar.gz: f92ba6323770d66d4f00db7194535ff46bac06e3
+  metadata.gz: a71017cf24127c36b2352c29a791a0615505e1e9
+  data.tar.gz: 52f02876f8af186158833b6d5e45abfa4b15a138
 SHA512:
-  metadata.gz: 04376d1753d497e4b55ef42960ef90948de5a106d7706d296e8c225081e54b7b4e73cea9ee1c85e8791a4ce849a3c3563e9d623f95fe0c36b55aeadbd7ee47d1
-  data.tar.gz: 58d72ca85fda5c537c48da63752c041c1785da464b77e7928cfb34e4c289e1e4e1153d58eca3710a10f75d73ea7e98a95a7570ec4c12951b846d29f0c9dcdd30
+  metadata.gz: 7e798bc960b21e114bef43adfc4beccce3f314183121233a8cbe94e6971308e5ebc5c601caff83168c55be95100ecf27c7956f0f73786076153935f3cee8045c
+  data.tar.gz: da0b1eca48164044ceab210bd53d6d5c2b3454a64c2f91dcab7d19bc917716de1ae91942b118ed4b9b0b99194f459c4b3e4f23fa28c7101a832bd60391665372

data/bookbinder.gemspec

@@ -2,7 +2,7 @@ require 'base64'
 
 Gem::Specification.new do |s|
   s.name        = 'bookbindery'
-  s.version     = '9.10.0'
+  s.version     = '9.11.0'
   s.summary     = 'Markdown to Rackup application documentation generator'
   s.description = 'A command line utility to be run in Book repositories to stitch together their constituent Markdown repos into a static-HTML-serving application'
   s.authors     = ['Mike Grafton', 'Lucas Marks', 'Gavin Morgan', 'Nikhil Gajwani', 'Dan Wendorf', 'Brenda Chan', 'Matthew Boedicker', 'Andrew Bruce', 'Frank Kotsianas', 'Elena Sharma', 'Christa Hartsock', 'Michael Trestman', 'Alpha Chen', 'Sarah McAlear', 'Gregg Van Hove']
@@ -22,7 +22,6 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency 'middleman-syntax', ['~> 2.0']
   s.add_runtime_dependency 'rouge', '!= 1.9.1'
   s.add_runtime_dependency 'redcarpet', ['~> 3.2.3']
-  s.add_runtime_dependency 'anemone'
   s.add_runtime_dependency 'css_parser'
   s.add_runtime_dependency 'puma'
   s.add_runtime_dependency 'rack-rewrite'
@@ -37,6 +36,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'pry-byebug'
   s.add_development_dependency 'rake'
   s.add_development_dependency 'rspec'
-  s.add_development_dependency 'sendgrid-ruby'
+  s.add_development_dependency 'sendgrid-ruby', '< 3.0'
   s.add_development_dependency 'jasmine'
 end

data/lib/bookbinder/commands/bind.rb

@@ -11,7 +11,7 @@ module Bookbinder
                      config_decorator: nil,
                      file_system_accessor: nil,
                      middleman_runner: nil,
-                     broken_links_checker: nil,
+                     link_checker: nil,
                      preprocessor: nil,
                      cloner_factory: nil,
                      section_repository: nil,
@@ -23,7 +23,7 @@ module Bookbinder
         @config_decorator = config_decorator
         @file_system_accessor = file_system_accessor
         @middleman_runner = middleman_runner
-        @broken_links_checker = broken_links_checker
+        @link_checker = link_checker
         @preprocessor = preprocessor
         @cloner_factory = cloner_factory
         @section_repository = section_repository
@@ -75,12 +75,11 @@ module Bookbinder
         if generation_result.success?
           file_system_accessor.copy(output_locations.build_dir, output_locations.public_dir)
 
-          broken_links_checker.check!(bind_config.broken_link_exclusions)
-          broken_links_checker.announce(bind_options.streams)
+          link_checker.check!(bind_config.broken_link_exclusions)
 
           bind_options.streams[:success].puts "Bookbinder bound your book into #{output_locations.final_app_dir}"
 
-          broken_links_checker.has_broken_links? ? 1 : 0
+          link_checker.has_errors? ? 1 : 0
         else
           bind_options.streams[:err].puts "Your bind failed. Rerun with --verbose to troubleshoot."
           1
@@ -100,7 +99,7 @@ module Bookbinder
         :output_locations,
         :preprocessor,
         :section_repository,
-        :broken_links_checker,
+        :link_checker,
         :middleman_runner,
       )
 

data/lib/bookbinder/commands/collection.rb

@@ -17,7 +17,7 @@ require_relative '../ingest/cloner_factory'
 require_relative '../ingest/section_repository'
 require_relative '../local_filesystem_accessor'
 require_relative '../middleman_runner'
-require_relative '../postprocessing/broken_links_checker'
+require_relative '../postprocessing/link_checker'
 require_relative '../preprocessing/dita_html_preprocessor'
 require_relative '../preprocessing/dita_pdf_preprocessor'
 require_relative '../preprocessing/link_to_site_gen_dir'
@@ -69,7 +69,7 @@ module Bookbinder
           config_fetcher: configuration_fetcher(Config::Configuration),
           config_decorator: Config::ConfigurationDecorator.new(loader: config_loader, config_filename: 'bookbinder.yml'),
           file_system_accessor: local_filesystem_accessor,
-          broken_links_checker: Postprocessing::BrokenLinksChecker.build(final_app_directory, sitemap_port),
+          link_checker: Postprocessing::LinkChecker.new(local_filesystem_accessor, final_app_directory, streams),
           preprocessor: Preprocessing::Preprocessor.new(
             Preprocessing::DitaHTMLPreprocessor.new(
               local_filesystem_accessor,

data/lib/bookbinder/css_link_checker.rb

@@ -1,5 +1,4 @@
 require 'css_parser'
-require_relative 'spider'
 
 module Bookbinder
   class CssLinkChecker
@@ -24,7 +23,7 @@ module Bookbinder
       css.each_selector do |s, declaration, sp|
         contents_of_url_value = /url\((.*?)\)/
         file_url = declaration.match contents_of_url_value
-        localized_uris << Spider.prepend_location(stylesheet, file_url[1]) if file_url
+        localized_uris << "#{URI(stylesheet).path} => #{file_url[1]}" if file_url
       end
 
       localized_uris

data/lib/bookbinder/ingest/local_filesystem_cloner.rb

@@ -18,7 +18,8 @@ module Bookbinder
         link!(
           source_repo_name,
           Pathname(user_repo_dir).join(source_repo_name.split('/').last),
-          Pathname(destination_parent_dir).join(DestinationDirectory.new(source_repo_name, destination_dir_name))
+          Pathname(destination_parent_dir).join(DestinationDirectory.new(source_repo_name, destination_dir_name)),
+          source_ref
         )
       end
 
@@ -26,7 +27,7 @@ module Bookbinder
 
       attr_reader :streams, :filesystem, :user_repo_dir
 
-      def link!(source_repo_name, source_dir, dest_dir)
+      def link!(source_repo_name, source_dir, dest_dir, source_ref)
         source_exists = filesystem.file_exist?(source_dir)
 
         if source_exists && filesystem.file_exist?(dest_dir)
@@ -34,6 +35,7 @@ module Bookbinder
           WorkingCopy.new(
             copied_to: dest_dir,
             full_name: source_repo_name,
+            ref: source_ref
           )
         elsif source_exists
           announce(source_dir)
@@ -41,6 +43,7 @@ module Bookbinder
           WorkingCopy.new(
             copied_to: dest_dir,
             full_name: source_repo_name,
+            ref: source_ref
           )
         else
           streams[:out].puts "  skipping (not found) #{source_dir}"

data/lib/bookbinder/local_filesystem_accessor.rb

@@ -1,6 +1,5 @@
 require 'find'
 require 'pathname'
-require 'nokogiri'
 require_relative 'errors/programmer_mistake'
 
 module Bookbinder
@@ -10,6 +9,14 @@ module Bookbinder
       File.exist?(path)
     end
 
+    def is_file?(path)
+      File.file?(path)
+    end
+
+    def is_dir?(path)
+      Dir.exists?(path)
+    end
+
     def write(to: nil, text: nil)
       make_directory(File.dirname to)
 

data/lib/bookbinder/postprocessing/link_checker.rb

@@ -0,0 +1,144 @@
+require 'nokogiri'
+require_relative '../css_link_checker'
+
+module Bookbinder
+  module Postprocessing
+    class LinkChecker
+      def initialize(fs, root_path, output_streams)
+        @fs = fs
+        @root_path = root_path
+        @output_streams = output_streams
+        @broken_link_count = 0
+        @redirect_regexes = {}
+        @redirect_strings = {}
+
+        @convert_to_relative = %r{\A.*#{root_path.to_s}/public}
+        @default_link_exclusions = %r{\A(?:https?://|javascript:|mailto:)}
+        @excluded_pages = %r{\A/(?:404\.html|subnavs|javascripts|stylesheets|style_guide)}
+      end
+
+      def check!(link_exclusions = /(?!.*)/)
+        @output_streams[:out].puts "\nChecking for broken links..."
+        load_redirects!
+        load_page_links
+
+        report_broken_links!(link_exclusions)
+        report_orphaned_pages!
+
+        if has_errors?
+          err "\nFound #{@broken_link_count} broken links!"
+        else
+          out "\nNo broken links!"
+        end
+      end
+
+      def has_errors?
+        @broken_link_count > 0
+      end
+
+      private
+
+      def report_broken_links!(link_exclusions)
+        @page_links.each do |page, links|
+          links.each do |link|
+            next if skip?(link, link_exclusions)
+
+            absolute_link, fragment = normalize_link(link, page)
+
+            if !page_exists?(absolute_link) && !file_exists?(absolute_link)
+              @broken_link_count += 1
+              err "#{page} => #{absolute_link}#{fragment ? "##{fragment}" : ''}"
+            end
+          end
+        end
+
+        broken_css_links = Dir.chdir(@root_path) { CssLinkChecker.new.broken_links_in_all_stylesheets }
+
+        @broken_link_count += broken_css_links.size
+        broken_css_links.each do |link|
+          err link
+        end
+      end
+
+      def report_orphaned_pages!
+        linked_pages = @page_links.map do |page, links|
+          links.map do |link|
+            normalize_link(link, page)[0]
+          end
+        end.flatten.uniq
+
+        orphaned_pages = @page_links.keys.reject { |page| page == '/index.html' || linked_pages.include?(page) }
+        if orphaned_pages.size > 0
+          err "\nOrphaned pages"
+          orphaned_pages.each do |page|
+            err "No links to => #{page}"
+          end
+        end
+      end
+
+      def skip?(link_path, link_exclusions)
+        @default_link_exclusions.match(link_path) || link_path.match(link_exclusions)
+      end
+
+      def page_exists?(link)
+        @page_links.has_key?(link) ||
+          @redirect_strings.has_key?(link) ||
+          @redirect_regexes.keys.detect {|reg| reg.match(link)}
+      end
+
+      def normalize_link(link, page)
+        return [page, link.sub(/\A#/, '')] if link[0] == '#'
+
+        absolute_link = link[0] == '/' ? link : File.expand_path(link, File.dirname(page))
+        absolute_link.split('#')
+      end
+
+      def file_exists?(link)
+        full_path = File.join(@root_path, 'public', link)
+        @fs.is_file?(full_path) || (@fs.is_dir?(full_path) && @fs.is_file?(File.join(full_path, 'index.html')))
+      end
+
+      def load_page_links
+        files = @fs.find_files_with_ext('html', File.join(@root_path, 'public'))
+
+        @page_links = files.each.with_object({}) do |file_path, links|
+          public_path = file_path.sub(@convert_to_relative, '')
+
+          if !@excluded_pages.match(public_path)
+            html = Nokogiri::HTML(@fs.read(file_path))
+
+            links[public_path] = html.css('a[href]').map { |link| link['href'] }
+          end
+        end
+      end
+
+      def load_redirects!
+        redirects_path = File.join(@root_path, 'redirects.rb')
+        if @fs.is_file?(redirects_path)
+          contents = @fs.read(redirects_path)
+          instance_eval contents
+        end
+      end
+
+      def out(str)
+        @output_streams[:out].puts(str)
+      end
+
+      def err(str)
+        @output_streams[:err].puts(str)
+      end
+
+      def r301(source, dest, options={})
+        return if options.has_key?(:if)
+
+        case source
+        when Regexp
+          @redirect_regexes[source] = dest
+        when String
+          @redirect_strings[source] = dest
+        end
+      end
+      alias r302 r301
+    end
+  end
+end

data/master_middleman/quicklinks_renderer.rb

@@ -45,6 +45,7 @@ class QuicklinksRenderer < Redcarpet::Render::Base
     doc = Nokogiri::HTML(text)
     target_anchor = doc.css('a').first
     return unless target_anchor && target_anchor['id']
+    return if (target_anchor['class'] || '').match(/\bno-quick-link\b/)
 
     anchor = Nokogiri::XML::Node.new('a', document)
     anchor['href'] = "##{target_anchor['id']}"

data/master_middleman/source/layouts/layout.erb

@@ -43,7 +43,7 @@
         <%= partial 'layouts/local-header' %>
         <%= partial 'layouts/title' %>
         <% if quick_links %>
-          <div id="js-quick-links">
+          <div id="js-quick-links" <%= 'class="list-style-none"' if data.page.list_style_none %>>
             <%= quick_links %>
           </div>
         <% end %>

data/master_middleman/source/stylesheets/base.scss

@@ -198,6 +198,10 @@ category: basics
   }
 }
 
+.list-style-none .quick-links > ul {
+  list-style-type: none;
+}
+
 // ~CONTENT
 // ===================================================
 // Designate external links with an icon
@@ -344,6 +348,7 @@ category: basics
     bottom: 4em;
     position: fixed;
     right: 4em;
+    z-index: 10;
     &:hover {
       background: none;
     }

data/template_app/Gemfile

@@ -6,5 +6,5 @@ gem 'rack'
 gem 'rack-rewrite'
 gem 'puma'
 gem 'therubyracer'
-gem 'sendgrid-ruby'
+gem 'sendgrid-ruby', '< 3.0'
 gem 'elasticsearch'

data/template_app/Gemfile.lock

@@ -36,7 +36,7 @@ DEPENDENCIES
   puma
   rack
   rack-rewrite
-  sendgrid-ruby
+  sendgrid-ruby (< 3.0)
   therubyracer
 
 BUNDLED WITH

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bookbindery
 version: !ruby/object:Gem::Version
-  version: 9.10.0
+  version: 9.11.0
 platform: ruby
 authors:
 - Mike Grafton
@@ -22,7 +22,7 @@ authors:
 autorequire: 
 bindir: install_bin
 cert_chain: []
-date: 2016-06-14 00:00:00.000000000 Z
+date: 2016-06-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fog-aws
@@ -122,20 +122,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 3.2.3
-- !ruby/object:Gem::Dependency
-  name: anemone
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: css_parser
   requirement: !ruby/object:Gem::Requirement
@@ -322,16 +308,16 @@ dependencies:
   name: sendgrid-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: jasmine
   requirement: !ruby/object:Gem::Requirement
@@ -405,16 +391,13 @@ files:
 - lib/bookbinder/ingest/working_copy.rb
 - lib/bookbinder/local_filesystem_accessor.rb
 - lib/bookbinder/middleman_runner.rb
-- lib/bookbinder/postprocessing/broken_links_checker.rb
+- lib/bookbinder/postprocessing/link_checker.rb
 - lib/bookbinder/preprocessing/dita_html_preprocessor.rb
 - lib/bookbinder/preprocessing/dita_pdf_preprocessor.rb
 - lib/bookbinder/preprocessing/link_to_site_gen_dir.rb
 - lib/bookbinder/preprocessing/preprocessor.rb
 - lib/bookbinder/server_director.rb
 - lib/bookbinder/sheller.rb
-- lib/bookbinder/sieve.rb
-- lib/bookbinder/spider.rb
-- lib/bookbinder/stabilimentum.rb
 - lib/bookbinder/streams/colorized_stream.rb
 - lib/bookbinder/streams/filter_stream.rb
 - lib/bookbinder/subnav/navigation_entries_from_html_toc.rb

data/lib/bookbinder/postprocessing/broken_links_checker.rb

@@ -1,43 +0,0 @@
-require_relative '../server_director'
-require_relative '../spider'
-require_relative '../../../template_app/rack_app'
-
-module Bookbinder
-  module Postprocessing
-    class BrokenLinksChecker
-      def self.build(final_app_directory, port)
-        new(
-          Spider.new(app_dir: final_app_directory),
-          ServerDirector.new(
-            app: RackApp.new(Pathname('redirects.rb'), auth_required: false).app,
-            directory: final_app_directory,
-            port: port
-          )
-        )
-      end
-
-      def initialize(spider, server_director)
-        @spider = spider
-        @server_director = server_director
-      end
-
-      def check!(broken_link_exclusions)
-        server_director.use_server { |port|
-          @result = spider.find_broken_links(port, broken_link_exclusions: broken_link_exclusions)
-        }
-      end
-
-      def announce(streams)
-        @result.announce_broken_links(streams)
-      end
-
-      def has_broken_links?
-        @result.has_broken_links?
-      end
-
-      private
-
-      attr_reader :server_director, :spider
-    end
-  end
-end

data/lib/bookbinder/sieve.rb

@@ -1,67 +0,0 @@
-require_relative 'spider'
-
-module Bookbinder
-  class Sieve
-    def initialize(domain: ->(){ raise 'You must supply a domain parameter.' }.call)
-      @unverified_fragments_by_url = {}
-      @domain = domain
-    end
-
-    def links_from(page, is_first_pass)
-      if page.not_found? && page.referer
-        working = []
-        broken  = [Spider.prepend_location(page.referer, page.url)]
-      elsif page.not_found?
-        working = []
-        broken  = []
-      else
-        working = [page.url.to_s]
-        broken  = broken_fragments_targeting(page, is_first_pass)
-        store_unverified_fragments_from(page) if is_first_pass
-      end
-
-      return broken, working
-    end
-
-    private
-
-    def store_unverified_fragments_from(page)
-      @unverified_fragments_by_url.merge! fragments_targeting_other_pages_from page
-    end
-
-    def broken_fragments_targeting(page, first_pass)
-      first_pass ? local_fragments_missing_from(page) : remote_fragments_missing_from(page)
-    end
-
-    def local_fragments_missing_from(page)
-      local_fragments = page.fragment_identifiers targeting_locally: true
-      local_fragments.reject { |uri| page.has_target_for?(uri) }.map { |uri| Spider.prepend_location(page.url, uri) }
-    end
-
-    def remote_fragments_missing_from(page)
-      @unverified_fragments_by_url.fetch(page.url, []).reject { |localized_identifier| page.has_target_for? URI(strip_location(localized_identifier)) }
-    end
-
-    def fragments_targeting_other_pages_from(page)
-      uris_with_fragments = page.fragment_identifiers(targeting_locally: false)
-      uris_with_fragments.reduce({}) { |dict, uri| merge_uris_under_targets(dict, page, uri) }
-    end
-
-    def merge_uris_under_targets(dict, page, uri)
-      target_url = URI::join @domain, uri.path
-      localized_identifier = Spider.prepend_location(page.url, "##{uri.fragment}")
-
-      if dict.has_key? target_url
-        dict[target_url] << localized_identifier
-      else
-        dict[target_url] = [localized_identifier]
-      end
-
-      dict
-    end
-
-    def strip_location(id)
-      id.split('=> ').last
-    end
-  end
-end

data/lib/bookbinder/spider.rb

@@ -1,77 +0,0 @@
-require 'anemone'
-require 'pty'
-require_relative 'css_link_checker'
-require_relative 'sieve'
-require_relative 'stabilimentum'
-
-module Bookbinder
-  class Spider
-    class Result
-      def initialize(broken_links)
-        @broken_links = broken_links
-      end
-
-      def has_broken_links?
-        @broken_links.any?
-      end
-
-      def announce_broken_links(streams)
-        if @broken_links.none?
-          streams[:out].puts "\nNo broken links!"
-        else
-          streams[:err].puts(<<-MESSAGE)
-
-Found #{@broken_links.count} broken links!
-
-#{@broken_links.sort.join("\n")}
-
-Found #{@broken_links.count} broken links!
-          MESSAGE
-        end
-      end
-    end
-
-    def self.prepend_location(location, url)
-      "#{URI(location).path} => #{url}"
-    end
-
-    def initialize(app_dir: nil)
-      @app_dir = app_dir || raise('Spiders must be initialized with an app directory.')
-    end
-
-    def find_broken_links(port, broken_link_exclusions: /(?!.*)/)
-      temp_host = "localhost:#{port}"
-      sieve = Sieve.new domain: "http://#{temp_host}"
-      broken_links = crawl_from "http://#{temp_host}#{ENV['CUSTOM_ROOT']}/index.html", sieve
-      public_broken_links = broken_links.reject {|l| l.match(broken_link_exclusions)}
-
-      Result.new(public_broken_links)
-    end
-
-    private
-
-    attr_reader :app_dir
-
-    def crawl_from(url, sieve)
-      broken_links = []
-      2.times do |i|
-        is_first_pass = (i==0)
-
-        Anemone.crawl(url, discard_page_bodies: true) do |anemone|
-          dont_visit_fragments(anemone)
-          anemone.on_every_page do |page|
-            broken, working = sieve.links_from(Stabilimentum.new(page), is_first_pass)
-            broken_links.concat broken
-          end
-        end
-      end
-
-      broken_links.concat Dir.chdir(@app_dir) { CssLinkChecker.new.broken_links_in_all_stylesheets }
-      broken_links.compact.uniq
-    end
-
-    def dont_visit_fragments(anemone)
-      anemone.focus_crawl { |page| page.links.reject { |link| link.to_s.match(/%23/) } }
-    end
-  end
-end

data/lib/bookbinder/stabilimentum.rb

@@ -1,63 +0,0 @@
-module Bookbinder
-  class Spider
-    class Stabilimentum # Decorates a piece of the web.
-      FudgedUri = Struct.new(:path, :fragment, :to_s)
-
-      def initialize(page)
-        @page = page
-      end
-
-      def referer
-        @page.referer
-      end
-
-      def not_found?
-        @page.not_found?
-      end
-
-      def url
-        @page.url
-      end
-
-      def has_target_for?(uri)
-        id_selector = uri.fragment
-        name_selector = "[name=#{uri.fragment}]"
-
-        if @page.doc
-          @page.doc.css("##{id_selector}").any? || @page.doc.css(name_selector).any?
-        else
-          false
-        end
-      rescue Nokogiri::CSS::SyntaxError
-        false
-      end
-
-      def fragment_identifiers(targeting_locally: false)
-        if targeting_locally
-          fragment_anchor_uris.select { |uri| uri.path.empty? }
-        else
-          fragment_anchor_uris.reject { |uri| uri.path.empty? }
-        end
-      end
-
-      private
-
-      def fragment_anchor_uris
-        anchors = @page.doc ? @page.doc.css('a') : []
-        anchors.map { |a| convert_to_uri(a) }.select { |u| u.fragment }
-      end
-
-      def convert_to_uri(anchor)
-        URI anchor['href'].to_s
-      rescue URI::InvalidURIError
-        create_fudged_uri(anchor['href'])
-      end
-
-      def create_fudged_uri(target)
-        path = target.split('#')[0]
-        fragment = target.include?('#') ? '#' + target.split('#')[1] : nil
-        FudgedUri.new(path, fragment, target)
-      end
-    end
-  end
-end