rewritten 0.15.2 → 0.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b7a6bb53aa68bfdf91a14ddb75bc1c84626d4722
-  data.tar.gz: f0b6c9dc59ae890105a08d8e718da649903949f4
+  metadata.gz: 1c46c2783b6c6c0b7513137ee064d35204d248a8
+  data.tar.gz: e30fe5d81af0086b38aca82dbd03e0a38b9833ef
 SHA512:
-  metadata.gz: 943f60c79e3c96113f0fb4cd6556cdbb079c1cd34caa476ca46065c2529096c3661c8a7d1a0cc5bc31d6be293694bffa3da898b5bc4b689eb2e05b7e4a9ac933
-  data.tar.gz: 135a2c929eaaedd54cd8558797885632eb42a22d806732c0dc221f8d984f20f31a870bc20bdadacbd42e023be8645267f7d7f7be51614b0f8d7faaffc21905f3
+  metadata.gz: d04627b4246a02ffb960087a07e54ac2a9d5f33196ef32d78b65e3b2ccaabe3515384a88f7536ed8305c9f5ceeb800bcf4c1fac8e9a1f109203839466727d113
+  data.tar.gz: 92736611e92c52327f5faa2754668c9cc97578f806c5aa145b2a7963c3616c0c137fb5c0333679b5312c1e61ecf2f30289494554b014b9599afa7cb92f878eb2

data/.rubocop.yml

@@ -0,0 +1,12 @@
+inherit_from: .rubocop_todo.yml
+
+#AllCops:
+#  Include:
+#    - '**/Rakefile'
+#    - '**/config.ru'
+#  Exclude:
+#    - 'test/fixtures/**/*'
+#    - 'db/**/*'
+#    - 'config/**/*'
+#    - 'vendor/**/*'
+#    - '/app/helpers/application_helper.rb'

data/.rubocop_todo.yml

@@ -0,0 +1,116 @@
+# This configuration was generated by `rubocop --auto-gen-config`
+# on 2015-07-31 12:23:42 +0200 using RuboCop version 0.32.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 1
+# Configuration parameters: AllowSafeAssignment.
+Lint/AssignmentInCondition:
+  Enabled: false
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: AlignWith, SupportedStyles, AutoCorrect.
+Lint/DefEndAlignment:
+  Enabled: false
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: AlignWith, SupportedStyles, AutoCorrect.
+Lint/EndAlignment:
+  Enabled: false
+
+# Offense count: 2
+Lint/UselessAssignment:
+  Enabled: false
+
+# Offense count: 9
+Metrics/AbcSize:
+  Max: 71
+
+# Offense count: 1
+# Configuration parameters: CountComments.
+Metrics/ClassLength:
+  Max: 209
+
+# Offense count: 4
+Metrics/CyclomaticComplexity:
+  Max: 16
+
+# Offense count: 40
+# Configuration parameters: AllowURI, URISchemes.
+Metrics/LineLength:
+  Max: 268
+
+# Offense count: 10
+# Configuration parameters: CountComments.
+Metrics/MethodLength:
+  Max: 35
+
+# Offense count: 1
+# Configuration parameters: CountComments.
+Metrics/ModuleLength:
+  Max: 231
+
+# Offense count: 4
+Metrics/PerceivedComplexity:
+  Max: 18
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/AndOr:
+  Enabled: false
+
+# Offense count: 2
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles, ProceduralMethods, FunctionalMethods, IgnoredMethods.
+Style/BlockDelimiters:
+  Enabled: false
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/BracesAroundHashParameters:
+  Enabled: false
+
+# Offense count: 2
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/ClassAndModuleChildren:
+  Enabled: false
+
+# Offense count: 14
+Style/Documentation:
+  Enabled: false
+
+# Offense count: 3
+# Configuration parameters: Exclude.
+Style/FileName:
+  Enabled: false
+
+# Offense count: 1
+# Configuration parameters: MinBodyLength.
+Style/GuardClause:
+  Enabled: false
+
+# Offense count: 1
+Style/ModuleFunction:
+  Enabled: false
+
+# Offense count: 2
+# Configuration parameters: EnforcedStyle, MinBodyLength, SupportedStyles.
+Style/Next:
+  Enabled: false
+
+# Offense count: 3
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles, AllowInnerSlashes.
+Style/RegexpLiteral:
+  Enabled: false
+
+# Offense count: 1
+# Configuration parameters: Methods.
+Style/SingleLineBlockParams:
+  Enabled: false

data/Gemfile

@@ -1,4 +1,4 @@
-source "http://rubygems.org"
+source 'http://rubygems.org'
 
 # Specify your gem's dependencies in rewritten.gemspec
 gemspec

data/HISTORY.rdoc

@@ -1,3 +1,7 @@
+== 0.16.0
+  
+  * Makes rewritten work with fullpath. Example: Matching on '/some/url?q=1' will work now
+
 == 0.15.2 
 
   * Fixes nested parameters

data/Rakefile

@@ -3,7 +3,7 @@ require 'rake/testtask'
 require 'rdoc/task'
 
 desc 'Default: run all tests'
-task :default => :test
+task default: :test
 
 Bundler::GemHelper.install_tasks
 
@@ -23,7 +23,3 @@ Rake::RDocTask.new(:rdoc) do |rdoc|
   rdoc.rdoc_files.include('HISTORY.rdoc')
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
-
-
-
-

data/bin/rewritten-dump.rb

@@ -6,40 +6,37 @@ require 'optparse'
 require 'rewritten'
 
 options = {
-  :out => 'rewritten.csv',
-  :verbose => false
+  out: 'rewritten.csv',
+  verbose: false
 }
 
 op = OptionParser.new do |opts|
-  opts.banner = "Usage: rewriten-dump.rb [options]"
+  opts.banner = 'Usage: rewriten-dump.rb [options]'
 
-  opts.on("-v", "--verbose", "be more verbose") do
+  opts.on('-v', '--verbose', 'be more verbose') do
     options[:verbose] = true
   end
 
-  opts.on("-o", "--out FILE", 'output file or "-" for stdout') do |o|
+  opts.on('-o', '--out FILE', 'output file or "-" for stdout') do |o|
     options[:out] = o
   end
 
-  opts.on("-u", "--uri URI", 'uri to the redis db') do |uri|
+  opts.on('-u', '--uri URI', 'uri to the redis db') do |uri|
     options[:uri] = uri
   end
 
-  opts.on("-h", "--help", 'print help') do
+  opts.on('-h', '--help', 'print help') do
     puts opts
     exit 0
   end
-
 end
 op.parse!
 
-Rewritten.redis = options[:uri] if options[:uri] 
+Rewritten.redis = options[:uri] if options[:uri]
 
-file = options[:out] == "-" ? STDOUT : File.open(options[:out], "w")
+file = options[:out] == '-' ? STDOUT : File.open(options[:out], 'w')
 
-file.puts "#from;to"
+file.puts '#from;to'
 Rewritten.all_tos.each do |to|
-  file.puts Rewritten.get_all_translations(to).map{|from| "#{Rewritten.full_line(from)};#{to}"}.join("\n")
+  file.puts Rewritten.get_all_translations(to).map { |from| "#{Rewritten.full_line(from)};#{to}" }.join("\n")
 end
-
-

data/bin/rewritten-import.rb

@@ -7,28 +7,27 @@ require 'rewritten'
 require 'multi_json'
 
 options = {
-  :drop => false
+  drop: false
 }
 
 op = OptionParser.new do |opts|
-  opts.banner = "Usage: rewriten-dump.rb [options]"
+  opts.banner = 'Usage: rewriten-dump.rb [options]'
 
-  opts.on("-v", "--verbose", "be more verbose") do |v|
+  opts.on('-v', '--verbose', 'be more verbose') do |v|
     options[:verbose] = v
   end
 
-  opts.on("-f", "--file FILE", 'input file') do |o|
+  opts.on('-f', '--file FILE', 'input file') do |o|
     options[:file] = o
   end
 
-  opts.on("-u", "--uri URI", 'uri to the redis db') do |uri|
+  opts.on('-u', '--uri URI', 'uri to the redis db') do |uri|
     options[:uri] = uri
   end
 
-  opts.on("-d", "--drop", 'drop translations first') do
+  opts.on('-d', '--drop', 'drop translations first') do
     options[:drop] = true
   end
-
 end
 
 op.parse!
@@ -38,15 +37,13 @@ unless options[:file]
   exit
 end
 
-Rewritten.redis = options[:uri] if options[:uri] 
+Rewritten.redis = options[:uri] if options[:uri]
 
 Rewritten.clear_translations if options[:drop]
 
 File.open(options[:file]).each do |line|
   next if line =~ /^#/
-  from,to = line.split(";")
+  from, to = line.split(';')
   puts "adding #{from} -> #{to}" if options[:verbose]
-  Rewritten.add_translation(from,to.chomp)
+  Rewritten.add_translation(from, to.chomp)
 end
-
-

data/bin/rewritten-web.rb

@@ -6,20 +6,18 @@ begin
 rescue LoadError
   require 'rubygems'
   require 'vegas'
-end  
+end
 
 require 'rewritten/server'
 
 Vegas::Runner.new(Rewritten::Server, 'rewritten-web', {
-  :before_run => lambda {|v|
-    path = (ENV['RESQUECONFIG'] || v.args.first)
-    load path.to_s.strip if path
-  }
-}) do |runner, opts, app|
-  opts.on('-N NAMESPACE', "--namespace NAMESPACE", "set the Redis namespace") {|namespace|
+                    before_run: lambda {|v|
+                      path = (ENV['RESQUECONFIG'] || v.args.first)
+                      load path.to_s.strip if path
+                    }
+                  }) do |runner, opts, _app|
+  opts.on('-N NAMESPACE', '--namespace NAMESPACE', 'set the Redis namespace') {|namespace|
     runner.logger.info "Using Redis namespace '#{namespace}'"
     Rewritten.redis.namespace = namespace
   }
 end
-
-

data/config.ru

@@ -1,4 +1,3 @@
 require 'rewritten'
 
 run Rewritten::Server
-

data/lib/rack/canonical.rb

@@ -1,11 +1,8 @@
 require 'rack'
 
 module Rack
-
   module Rewritten
-
     class Canonical
-
       def initialize(app)
         @app = app
       end
@@ -15,30 +12,25 @@ module Rack
 
         status, headers, response = @app.call(req.env)
 
-        if status == 200 && headers["Content-Type"] =~ /text\/html|application\/xhtml\+xml/
-          body = ""
+        if status == 200 && headers['Content-Type'] =~ /text\/html|application\/xhtml\+xml/
+          body = ''
           response.each { |part| body << part }
-          index = body.rindex("</head>")
+          index = body.rindex('</head>')
           if index
             # go with a request duplicate since infinitive works on translations
             target_req = req.dup
-            target_req.path_info = ::Rewritten.infinitive( ::Rewritten.get_current_translation(req.path) )
+            target_req.path_info = ::Rewritten.infinitive(::Rewritten.get_current_translation(req.path))
             target_req.env['QUERY_STRING'] = ''
             target = target_req.url
 
-            body.insert(index, %Q|<link rel="canonical" href="#{target}"/>| )
-            headers["Content-Length"] = body.length.to_s
+            body.insert(index, %(<link rel="canonical" href="#{target}"/>))
+            headers['Content-Length'] = body.length.to_s
             response = [body]
           end
 
         end
         [status, headers, response]
       end
-
     end
-
   end
-
 end
-
-

data/lib/rack/dummy.rb

@@ -1,25 +1,18 @@
 require 'rack'
 
 module Rack
-
   class Dummy
-
     def call(env)
-      puts "-> Rack::Dummy"
+      puts '-> Rack::Dummy'
       lines = []
       req = Rack::Request.new(env)
-      lines <<  req.path
-      lines <<  req.params.inspect
-      lines <<  req.host
-      lines <<  req.env.inspect
-      lines <<  "SUBDOMAIN: #{env['SUBDOMAIN']}"
+      lines << req.path
+      lines << req.params.inspect
+      lines << req.host
+      lines << req.env.inspect
+      lines << "SUBDOMAIN: #{env['SUBDOMAIN']}"
       lines << '<a href="/some/resource">'
-      [200, {"Content-Type"  => "text/plain"}, lines.join("\n")]
+      [200, { 'Content-Type'  => 'text/plain' }, lines.join("\n")]
     end
-
   end
-
 end
-
-
-

data/lib/rack/html.rb

@@ -1,11 +1,8 @@
 require 'rack'
 
 module Rack
-
   module Rewritten
-
     class Html
-
       def initialize(app)
         @app = app
       end
@@ -20,15 +17,13 @@ module Rack
             links = line.scan(/href="([^"]+)"/).flatten.uniq
             res = line
             links.each do |link|
-
               if ::Rewritten.exist_translation_for?(link)
                 t = ::Rewritten.get_current_translation(link)
-                res.gsub!(%Q|href="#{link}"|, %Q|href="#{t}"|) if t
-                res.gsub!(%Q|href='#{link}'|, %Q|href='#{t}'|) if t
-                res.gsub!(%Q|href="#{link}?|, %Q|href="#{t}?|) if t
-                res.gsub!(%Q|href='#{link}?|, %Q|href='#{t}?|) if t
+                res.gsub!(%(href="#{link}"), %(href="#{t}")) if t
+                res.gsub!(%(href='#{link}'), %(href='#{t}')) if t
+                res.gsub!(%(href="#{link}?), %(href="#{t}?)) if t
+                res.gsub!(%(href='#{link}?), %(href='#{t}?)) if t
               end
-
             end
             new_response << res
           end
@@ -38,10 +33,6 @@ module Rack
 
         [status, headers, new_response]
       end
-
     end
-
   end
 end
-
-

data/lib/rack/record.rb

@@ -1,11 +1,8 @@
 require 'rack'
 
 module Rack
-
   module Rewritten
-
     class Record
-
       def initialize(app)
         @app = app
       end
@@ -13,18 +10,15 @@ module Rack
       def call(env)
         req = Rack::Request.new(env.clone)
         status, headers, response = @app.call(env)
-        puts "-> Rack::Rewritten::Record"
+        puts '-> Rack::Rewritten::Record'
         puts headers.inspect
 
-        if [200,301,302].include?(status)
-          ::Rewritten.add_hit(req.path,status, headers["Content-Type"]) if headers["Content-Type"] =~ /text\/html/
+        if [200, 301, 302].include?(status)
+          ::Rewritten.add_hit(req.path, status, headers['Content-Type']) if headers['Content-Type'] =~ /text\/html/
         end
 
         [status, headers, response]
       end
-
     end
-
   end
-
 end