xml-mixup 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b5d422e3cab772172b48772aa775c71c7362f3459c4315b28a48180832160dca
-  data.tar.gz: 12d50f74beb943f993fbd52b89913e9696c9240893fb14bcca81a434bc569d0c
+  metadata.gz: d4939bcfe6e304b5b71251451f59982f747f43b6e31cdd5337d95ed30dfa37d3
+  data.tar.gz: e97756ea84870da09ac8fbc82d35b74d0a856eb74ed0fe826d4153d520f16398
 SHA512:
-  metadata.gz: 1896aa7cec980baf9779ca87aaad386de815879406f24ca04cc92d9a5b00b995e4a785edbd7d625b4954435f3b248b69f997f429eb55d76d1dbef4538f38f0da
-  data.tar.gz: 1648c19729fcd0a56504c046d11e4c2def65c2cc287ee371fbb24e837c8fe499991ad763c51ec9abaa060d3e34718c8db69d74bb16c3299c6da8c5266532eee0
+  metadata.gz: bed9c8b8a0f064d533192bee3f883f9b19c00e45dfb4e569c0b34fbfa7622124087a8fb39c5a36aefdbafab2e5c3d412af0d8911262111f89893c110354cd737
+  data.tar.gz: 870ebed923f4b9a6fb5c83924115ad598a4b1293ed03a481ef9cd5a3409e53bca68609e61baa23f4549f92f5f7d4a7551dc375b27d769007d2f59df9c48e64e9

data/README.md

@@ -44,20 +44,20 @@ Ruby, and found a lot of the same things:
 
 Granted it's a lot nicer to do this sort of thing in Ruby, but at the
 end of the day, the thing generating the XML is a nested list of
-method calls---_not_ a declarative data structure.
+method calls — _not_ a declarative data structure.
 
 ### Document has to be generated all in one shot
 
 It's not super-easy to generate a piece of the target document and
 then go back and generate some more (although
-{Nokogiri::XML::Builder.with} is a nice start). This plus the last
+`Nokogiri::XML::Builder.with` is a nice start). This plus the last
 point leads to all sorts of cockamamy constructs which are almost as
 life-sucking as writing raw DOM routines.
 
 ### Hard to do surgery on existing documents
 
 This comes up a lot: you have an existing document and you want to add
-even just a single node to it---say, in between two nodes just for
+even just a single node to it — say, in between two nodes just for
 fun. Good luck with that.
 
 ### Enter `XML::Mixup`
@@ -77,7 +77,7 @@ among other things, takes a `:spec`. The spec can be any composite
 
 ### Hashes
 
-The principal construct in `XML::Mixup` is the {Hash}. You can
+The principal construct in `XML::Mixup` is the `Hash`. You can
 generate pretty much any node with it:
 
 #### Elements
@@ -115,7 +115,7 @@ Attributes are sorted lexically. Composite attribute values get
 flattened like this:
 
 ```ruby
-{ nil => :foo, array: [:a :b], hash: { e: :f, c: :d } }
+{ nil => :foo, array: [:a, :b], hash: { e: :f, c: :d } }
 # => <foo array="a b" hash="c: d e: f"/>
 ```
 

data/lib/xml/mixup/version.rb

@@ -1,5 +1,5 @@
 module XML
   module Mixup
-    VERSION = "0.1.0"
+    VERSION = "0.1.1"
   end
 end

data/lib/xml/mixup.rb

@@ -112,10 +112,10 @@ module XML::Mixup
     ADJACENT.keys do |k|
       if nodes[k]
         if adj
-          raise
+          raise "Cannot bind to #{k}: #{adj} is already present"
         end
         unless nodes[k].is_a? Nokogiri::XML::Node
-          raise
+          raise "#{k} must be an XML node"
         end
         adj = k
       end
@@ -126,7 +126,7 @@ module XML::Mixup
       doc ||= nodes[adj].document
       unless adj == 'parent'
         unless (nodes[:parent] = nodes[adj].parent)
-          raise
+          raise "#{adj} node must have a parent node!"
         end
       end
     else
@@ -222,7 +222,7 @@ module XML::Mixup
         elsif name == '#pi' or name == '#processing-instruction'
           # now processing instructions
           if children.empty?
-            raise
+            raise "Processing instruction must have at least a target"
           end
           target  = children[0]
           content = ''
@@ -245,7 +245,7 @@ module XML::Mixup
         elsif name == '#dtd' or name == '#doctype'
           # now doctype declarations
           if children.empty?
-            raise
+            raise "DTD node must have a root element declaration"
           end
 
           # assign as if these are args
@@ -255,7 +255,7 @@ module XML::Mixup
           sys ||= attr[:system] if attr[:system]
 
           # XXX for some reason this is an *internal* subset?
-          node = doc.create_internal_subset(root, pub, sys)
+          node = doc.create_internal_subset(root, pub.to_s, sys.to_s)
 
           # at any rate it doesn't have to be explicitly attached
 
@@ -319,7 +319,7 @@ module XML::Mixup
 
           # there should be no nil namespace declarations now
           if ns.has_value? nil
-            raise
+            raise 'INTERNAL ERROR: nil namespace declaration'
           end
 
           # generate the node
@@ -466,7 +466,7 @@ module XML::Mixup
   private
 
   def element tag, doc: nil, ns: {}, attr: {}, args: []
-    raise unless doc
+    raise 'Document node must be present' unless doc
     prefix = local = nil
     if tag.respond_to? :to_a
       prefix, local = tag

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: xml-mixup
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Dorian Taylor