trenni 3.1.0 → 3.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1ca1689a6184db9df2773f6d3b4b6211ed558f1a
-  data.tar.gz: 97d4dc5f0e2fa4f7f165519bc8f3c86c8e36c822
+  metadata.gz: '081ca65498a2ec0527b25ce54a105bc8e19341a0'
+  data.tar.gz: 224372fa6f3c9715858d8b1c700a050c33cf226c
 SHA512:
-  metadata.gz: c57c4f0c3463d35c0d73142be0451ea2243901fc67801d36f9dded3acc1118a85104be7138da3a442028e267333099772d2f867e6517ab0c6abb47acf731dc2a
-  data.tar.gz: 1b84a6d7d545ac7fc8de48c203f8d8cd1ab8652ce422f6c8da1e754d2d67023db807e96d2c236926ad34d88b4cf1f938c5a60ca498daf3a3f1c27b305f7d6ccc
+  metadata.gz: 3235d940fbf6eff830ab88623dc46fae3f92c6a31b20f2f37f28cac9f9f304cfdc7eb3a1d8eb0692bc5578aac09ffeb72763b3d594e51b08c4d8cb51b2aca300
+  data.tar.gz: 9d47d3e2feb3e03b42a9abf43430113c0aef7e1e4d25bc23aaafd2c2119d620810cc1bb10a50cc92f545a605abcfce918544f91d48d05a6a3641b675e4d8a8f2

data/lib/trenni/uri.rb

@@ -20,21 +20,32 @@
 
 module Trenni
 	class URI
-		def initialize(base, fragment, query)
-			@base = base
+		def initialize(path, query_string, fragment, parameters)
+			@path = path
+			@query_string = query_string
 			@fragment = fragment
-			@query = query
+			@parameters = parameters
 		end
 		
-		attr :base
+		# The path component of the URI, e.g. /foo/bar/index.html
+		attr :path
+		
+		# The un-parsed query string of the URI, e.g. 'x=10&y=20'
+		attr :query_string
+		
+		# A fragment identifier, the part after the '#'
 		attr :fragment
-		attr :query
+		
+		# User supplied parameters that will be appended to the query part.
+		attr :parameters
 		
 		def append(buffer)
-			buffer << escape_path(@base)
-			
-			if @query&.any?
-				buffer << query_separator << query_part
+			if @query_string
+				buffer << escape_path(@path) << '?' << query_string
+				buffer << '&' << query_parameters if @parameters
+			else
+				buffer << escape_path(@path)
+				buffer << '?' << query_parameters if @parameters
 			end
 			
 			if @fragment
@@ -52,10 +63,12 @@ module Trenni
 		
 		private
 		
-		# Escapes a path string, using percent encoding, but additionally ignoring "/" and substituting spaces with "+".
+		# According to https://tools.ietf.org/html/rfc3986#section-3.3, we escape non-pchar.
+		NON_PCHAR = /([^ a-zA-Z0-9\-\.~!$&'()*+,;=:@\/]+)/.freeze
+		
 		def escape_path(path)
 			encoding = path.encoding
-			path.b.gsub(/([^ a-zA-Z0-9_.\-\/:]+)/) do |m|
+			path.b.gsub(NON_PCHAR) do |m|
 				'%' + m.unpack('H2' * m.bytesize).join('%').upcase
 			end.tr(' ', '+').force_encoding(encoding)
 		end
@@ -68,18 +81,16 @@ module Trenni
 			end.force_encoding(encoding)
 		end
 		
-		def query_part
-			@query.map{|k,v| "#{escape(k.to_s)}=#{escape(v.to_s)}"}.join('&')
-		end
-		
-		def query_separator
-			@base.include?('?') ? '&' : '?'
+		def query_parameters
+			@parameters.map{|k,v| "#{escape(k.to_s)}=#{escape(v.to_s)}"}.join('&')
 		end
 	end
 	
-	def self.URI(path = '', query = nil)
+	# Generate a URI from a path and user parameters. The path may contain a `#fragment` or `?query=parameters`.
+	def self.URI(path = '', parameters = nil)
 		base, fragment = path.split('#', 2)
+		path, query_string = base.split('?', 2)
 		
-		URI.new(base, fragment, query)
+		URI.new(path, query_string, fragment, parameters)
 	end
 end

data/lib/trenni/version.rb

@@ -19,5 +19,5 @@
 # THE SOFTWARE.
 
 module Trenni
-	VERSION = "3.1.0"
+	VERSION = "3.1.1"
 end

data/spec/trenni/uri_spec.rb

@@ -52,4 +52,17 @@ RSpec.describe Trenni::URI do
 		
 		expect(tag.to_s).to be == '<img src="image.jpg?x=10"/>'
 	end
+	
+	describe Trenni::URI("foo?bar=10&baz=20", yes: 'no') do
+		it "can use existing query parameters" do
+			expect(subject.to_s).to be == "foo?bar=10&baz=20&yes=no"
+		end
+	end
+	
+	describe Trenni::URI('foo#frag') do
+		it "can use existing fragment" do
+			expect(subject.fragment).to be == "frag"
+			expect(subject.to_s).to be == 'foo#frag'
+		end
+	end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: trenni
 version: !ruby/object:Gem::Version
-  version: 3.1.0
+  version: 3.1.1
 platform: ruby
 authors:
 - Samuel Williams