inversion 0.10.0 → 0.10.1

data/ChangeLog

@@ -1,8 +1,41 @@
+2012-05-07  Michael Granger  <ged@FaerieMUD.org>
+
+	* .hgtags:
+	Added tag v0.10.0 for changeset 64cf302354aa
+	[79bc44fab294] [tip]
+
+	* .hgsigs:
+	Added signature for changeset 9d9c49d532be
+	[64cf302354aa] [v0.10.0]
+
+	* .rvm.gems, History.rdoc, Manifest.txt, Rakefile, bin/inversion,
+	experiments/benchmark-parser.rb, experiments/demo.rb,
+	experiments/dumptemplate.rb, lib/inversion.rb,
+	lib/inversion/logging.rb, lib/inversion/mixins.rb,
+	lib/inversion/parser.rb, lib/inversion/renderstate.rb,
+	lib/inversion/template.rb, lib/inversion/template/begintag.rb,
+	lib/inversion/template/codetag.rb,
+	lib/inversion/template/commenttag.rb,
+	lib/inversion/template/configtag.rb,
+	lib/inversion/template/elsetag.rb,
+	lib/inversion/template/elsiftag.rb,
+	lib/inversion/template/endtag.rb, lib/inversion/template/fortag.rb,
+	lib/inversion/template/iftag.rb,
+	lib/inversion/template/includetag.rb,
+	lib/inversion/template/node.rb, lib/inversion/template/rescuetag.rb,
+	lib/inversion/template/tag.rb, lib/inversion/template/textnode.rb,
+	lib/inversion/template/unlesstag.rb,
+	lib/inversion/template/yieldtag.rb, spec/inversion/logging_spec.rb,
+	spec/inversion/mixins_spec.rb, spec/inversion_spec.rb,
+	spec/lib/helpers.rb:
+	Convert to Loggability for logging.
+	[9d9c49d532be]
+
 2012-04-24  Michael Granger  <ged@FaerieMUD.org>
 
 	* .hgtags:
 	Added tag v0.9.0 for changeset 27f082ef7985
-	[dd18f3255985] [tip]
+	[dd18f3255985] [github/master]
 
 	* .hgsigs:
 	Added signature for changeset b208fa9154b1
@@ -21,7 +54,7 @@
 
 	* manual/src/tags.page:
 	Fix the include tag example in the manual.
-	[b5fcbae36ff2] [github/master]
+	[b5fcbae36ff2]
 
 2012-04-02  Michael Granger  <ged@FaerieMUD.org>
 

data/History.rdoc

@@ -1,3 +1,12 @@
+== v0.10.1 [2012-06-22] Michael Granger <ged@FaerieMUD.org>
+
+- Bugfix: duplicated templates get distinct copies of their attributes.
+
+
+== v0.10.0 [2012-05-07] Michael Granger <ged@FaerieMUD.org>
+
+- Added signature for changeset 9d9c49d532be
+
 == v0.10.0 [2012-05-07] Michael Granger <ged@FaerieMUD.org>
 
 - Convert to Loggability for logging.

data/Rakefile

@@ -20,7 +20,7 @@ hoespec = Hoe.spec 'inversion' do
 	self.developer 'Michael Granger', 'ged@FaerieMUD.org'
 	self.developer 'Mahlon E. Smith', 'mahlon@martini.nu'
 
-	self.dependency 'loggability',   '~> 0.0'
+	self.dependency 'loggability',   '~> 0.2'
 
 	self.dependency 'rdoc',          '~> 3.12', :development
 	self.dependency 'rspec',         '~> 2.8',  :development

data/lib/inversion.rb

@@ -25,10 +25,10 @@ module Inversion
 	warn ">>> Inversion requires Ruby 1.9.2 or later. <<<" if RUBY_VERSION < '1.9.2'
 
 	# Library version constant
-	VERSION = '0.10.0'
+	VERSION = '0.10.1'
 
 	# Version-control revision constant
-	REVISION = %q$Revision: 9d9c49d532be $
+	REVISION = %q$Revision: 432abb56d593 $
 
 
 	### Get the Inversion version.

data/lib/inversion/mixins.rb

@@ -196,5 +196,37 @@ module Inversion
 
 	end # module MethodUtilities
 
+
+	# A collection of data-manipulation functions.
+	module DataUtilities
+
+		### Recursively copy the specified +obj+ and return the result.
+		def deep_copy( obj )
+			# self.log.debug "Deep copying: %p" % [ obj ]
+
+			# Handle mocks during testing
+			return obj if obj.class.name == 'RSpec::Mocks::Mock'
+
+			return case obj
+				when NilClass, Numeric, TrueClass, FalseClass, Symbol
+					obj
+
+				when Array
+					obj.map {|o| deep_copy(o) }
+
+				when Hash
+					newhash = {}
+					obj.each do |k,v|
+						newhash[ deep_copy(k) ] = deep_copy( v )
+					end
+					newhash
+
+				else
+					obj.clone
+				end
+		end
+
+	end # module DataUtilities
+
 end # module Inversion
 

data/lib/inversion/renderstate.rb

@@ -8,6 +8,8 @@ require 'inversion' unless defined?( Inversion )
 # An object that provides an encapsulation of the template's state while it is rendering.
 class Inversion::RenderState
 	extend Loggability
+	include Inversion::DataUtilities
+
 
 	# Loggability API -- set up logging through the Inversion module's logger
 	log_to :inversion
@@ -436,35 +438,5 @@ class Inversion::RenderState
 	end
 
 
-	#######
-	private
-	#######
-
-	### Recursively copy the specified +obj+ and return the result.
-	def deep_copy( obj )
-		# self.log.debug "Deep copying: %p" % [ obj ]
-
-		# Handle mocks during testing
-		return obj if obj.class.name == 'RSpec::Mocks::Mock'
-
-		return case obj
-			when NilClass, Numeric, TrueClass, FalseClass, Symbol
-				obj
-
-			when Array
-				obj.map {|o| deep_copy(o) }
-
-			when Hash
-				newhash = {}
-				obj.each do |k,v|
-					newhash[ deep_copy(k) ] = deep_copy( v )
-				end
-				newhash
-
-			else
-				obj.clone
-			end
-	end
-
 end # class Inversion::RenderState
 

data/lib/inversion/template.rb

@@ -18,6 +18,8 @@ end
 # can be used to populate it when rendered.
 class Inversion::Template
 	extend Loggability
+	include Inversion::DataUtilities
+
 
 	# Loggability API -- set up logging through the Inversion module's logger
 	log_to :inversion
@@ -140,14 +142,9 @@ class Inversion::Template
 			self.log.debug "Parse state is: %p" % [ parsestate ]
 		end
 
-		# carry global template options to the parser.
-		opts = self.class.config.merge( opts )
-
 		@source       = source
-		@parser       = Inversion::Parser.new( self, opts )
 		@node_tree    = [] # Parser expects this to always be an Array
-		@init_options = opts
-		@options      = nil
+		@options      = opts
 		@attributes   = {}
 		@source_file  = nil
 		@created_at   = Time.now
@@ -157,6 +154,14 @@ class Inversion::Template
 	end
 
 
+	### Copy constructor -- make copies of some internal data structures, too.
+	def initialize_copy( other )
+		@options    = deep_copy( other.options )
+		@node_tree  = deep_copy( other.node_tree )
+		@attributes = deep_copy( other.attributes )
+	end
+
+
 	######
 	public
 	######
@@ -223,13 +228,19 @@ class Inversion::Template
 	### Return a human-readable representation of the template object suitable
 	### for debugging.
 	def inspect
-		return "#<%s:%08x (loaded from %s) attributes: %p, node_tree: %p, options: %p>" % [
+		nodemap = if $DEBUG
+				", node_tree: %p" % [ self.node_tree.map(&:as_comment_body) ]
+			else
+				''
+			end
+
+		return "#<%s:%08x (loaded from %s) attributes: %p, options: %p%s>" % [
 			self.class.name,
 			self.object_id / 2,
 			self.source_file ? self.source_file : "memory",
-			self.attributes,
-			self.node_tree.map(&:as_comment_body),
+			self.attributes.keys,
 			self.options,
+			nodemap
 		]
 	end
 
@@ -252,9 +263,11 @@ class Inversion::Template
 
 	### Parse the given +source+ into the template node tree.
 	def parse( source, parsestate=nil )
-		@options = self.class.config.merge( @init_options )
+		opts = self.class.config.merge( self.options )
+		parser = Inversion::Parser.new( self, opts )
+
 		@attributes.clear
-		@node_tree = @parser.parse( source, parsestate )
+		@node_tree = parser.parse( source, parsestate )
 		@source = source
 
 		self.define_attribute_accessors

data/spec/inversion/template_spec.rb

@@ -62,10 +62,16 @@ describe Inversion::Template do
 	end
 
 	it "carries its global configuration to the parser" do
-		Inversion::Template.configure( :i_exist => true )
-		tmpl = Inversion::Template.new( '' )
-		parser = tmpl.instance_variable_get( :@parser )
-		parser.options.should have_key( :i_exist )
+		begin
+			orig_config = Inversion::Template.config
+			Inversion::Template.configure( :ignore_unknown_tags => false )
+
+			expect {
+				Inversion::Template.new( '<?rumple an unknown tag ?>' )
+			}.to raise_error( Inversion::ParseError, /unknown tag/i )
+		ensure
+			Inversion::Template.config = orig_config
+		end
 	end
 
 	it "can make an human-readable string version of itself suitable for debugging" do
@@ -74,7 +80,19 @@ describe Inversion::Template do
 		tmpl.inspect.should =~ /Inversion::Template/
 		tmpl.inspect.should =~ %r{/tmp/inspect.tmpl}
 		tmpl.inspect.should =~ /attributes/
-		tmpl.inspect.should =~ /node_tree/
+		tmpl.inspect.should_not =~ /node_tree/
+	end
+
+	it "includes the node tree in the inspected object if debugging is enabled" do
+		begin
+			debuglevel = $DEBUG
+			$DEBUG = true
+
+			tmpl = Inversion::Template.new( '<?attr something ?>' )
+			tmpl.inspect.should =~ /node_tree/
+		ensure
+			$DEBUG = debuglevel
+		end
 	end
 
 	it "provides accessors for attributes that aren't identifiers in the template" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inversion
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.10.1
   prerelease: 
 platform: ruby
 authors:
@@ -37,7 +37,7 @@ cert_chain:
   YUhDS0xaZFNLai9SSHVUT3QrZ2JsUmV4OEZBaDhOZUEKY21saFhlNDZwWk5K
   Z1dLYnhaYWg4NWpJang5NWhSOHZPSStOQU01aUg5a09xSzEzRHJ4YWNUS1Bo
   cWo1UGp3RgotLS0tLUVORCBDRVJUSUZJQ0FURS0tLS0tCg==
-date: 2012-05-07 00:00:00.000000000 Z
+date: 2012-06-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: loggability
@@ -46,7 +46,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '0.0'
+        version: '0.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -54,7 +54,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '0.0'
+        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: hoe-mercurial
   requirement: !ruby/object:Gem::Requirement