converted the rest!

lib/squib/api/shapes.rb

@@ -7,117 +7,9 @@ require_relative '../args/coords'
 module Squib
   class Deck
 
-    # DSL method. See http://squib.readthedocs.io
-    def rect(opts = {})
-      range = Args::CardRange.new(opts[:range], deck_size: size)
-      box   = Args::Box.new(self).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      draw  = Args::Draw.new(custom_colors).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      trans  = Args::Transform.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      range.each { |i| @cards[i].rect(box[i], draw[i], trans[i]) }
-    end
-
-    # DSL method. See http://squib.readthedocs.io
-    def circle(opts = {})
-      range  = Args::CardRange.new(opts[:range], deck_size: size)
-      coords = Args::Coords.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      draw   = Args::Draw.new(custom_colors).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      range.each { |i| @cards[i].circle(coords[i], draw[i]) }
-    end
-
-    # DSL method. See http://squib.readthedocs.io
-    def ellipse(opts = {})
-      range = Args::CardRange.new(opts[:range], deck_size: size)
-      draw  = Args::Draw.new(custom_colors).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      box   = Args::Box.new(self, { width: '0.25in', height: '0.25in' }).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      trans  = Args::Transform.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      range.each { |i| @cards[i].ellipse(box[i], draw[i], trans[i]) }
-    end
-
-    # DSL method. See http://squib.readthedocs.io
-    # def grid(opts = {})
-    #   range = Args::CardRange.new(opts[:range], deck_size: size)
-    #   draw  = Args::Draw.new(custom_colors).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-    #   box   = Args::Box.new(self).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-    #   range.each { |i| @cards[i].grid(box[i], draw[i]) }
-    # end
-
-    # DSL method. See http://squib.readthedocs.io
-    def triangle(opts = {})
-      range  = Args::CardRange.new(opts[:range], deck_size: size)
-      draw   = Args::Draw.new(custom_colors).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      coords = Args::Coords.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      range.each { |i| @cards[i].triangle(coords[i], draw[i]) }
-    end
-
-    # DSL method. See http://squib.readthedocs.io
-    def line(opts = {})
-      range   = Args::CardRange.new(opts[:range], deck_size: size)
-      draw    = Args::Draw.new(custom_colors).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      coords  = Args::Coords.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      range.each { |i| @cards[i].line(coords[i], draw[i]) }
-    end
-
-    # DSL method. See http://squib.readthedocs.io
-    def curve(opts = {})
-      range  = Args::CardRange.new(opts[:range], deck_size: size)
-      draw   = Args::Draw.new(custom_colors).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      coords = Args::Coords.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      range.each { |i| @cards[i].curve(coords[i], draw[i]) }
-    end
-
-    # DSL method. See http://squib.readthedocs.io
-    def star(opts = {})
-      range  = Args::CardRange.new(opts[:range], deck_size: size)
-      draw   = Args::Draw.new(custom_colors).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      coords = Args::Coords.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      trans  = Args::Transform.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      range.each { |i| @cards[i].star(coords[i], trans[i], draw[i]) }
-    end
-
-    # DSL method. See http://squib.readthedocs.io
-    def polygon(opts = {})
-      range  = Args::CardRange.new(opts[:range], deck_size: size)
-      draw   = Args::Draw.new(custom_colors).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      coords = Args::Coords.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      trans  = Args::Transform.new.load!(opts, expand_by: size, layout: layout, dpi: dpi)
-      range.each { |i| @cards[i].polygon(coords[i], trans[i], draw[i]) }
-    end
-
-    # DSL method. See http://squib.readthedocs.io
-    def safe_zone(opts = {})
-      safe_defaults = {
-        margin: '0.25in',
-        radius: '0.125in',
-        stroke_color: :blue,
-        fill_color: '#0000',
-        stroke_width: 1.0,
-        dash: '3 3',
-      }
-      new_opts = safe_defaults.merge(opts)
-      margin = Args::UnitConversion.parse new_opts[:margin]
-      new_opts[:x] = margin
-      new_opts[:y] = margin
-      new_opts[:width] = width - (2 * margin)
-      new_opts[:height] = height - (2 * margin)
-      rect new_opts
-    end
-
     # DSL method. See http://squib.readthedocs.io
     def cut_zone(opts = {})
-      safe_defaults = {
-        margin: '0.125in',
-        radius: '0.125in',
-        stroke_color: :red,
-        fill_color: '#0000',
-        stroke_width: 2.0,
-      }
-      new_opts = safe_defaults.merge(opts)
-      margin = Args::UnitConversion.parse new_opts[:margin]
-      new_opts[:x] = margin
-      new_opts[:y] = margin
-      new_opts[:width] = width - (2 * margin)
-      new_opts[:height] = height - (2 * margin)
-      rect new_opts
+      
     end
 
   end

lib/squib/args/coords.rb

@@ -1,7 +1,7 @@
 require_relative 'arg_loader'
 
 module Squib::Args
-  module_function def extract_coord(opts, deck)
+  module_function def extract_coords(opts, deck)
     Coords.new.extract!(opts, deck)
   end
 

lib/squib/deck.rb

@@ -102,11 +102,9 @@ module Squib
     ##################
     ### PUBLIC API ###
     ##################
-    require_relative 'dsl/background'
     require_relative 'api/data'
     require_relative 'api/groups'
     require_relative 'api/settings'
-    require_relative 'api/shapes'
     require_relative 'api/text'
     require_relative 'api/units'
 
@@ -114,15 +112,25 @@ module Squib
     ### DSL METHODS ###
     ###################
     require_relative 'dsl/background'
+    require_relative 'dsl/circle'
+    require_relative 'dsl/curve'
+    require_relative 'dsl/cut_zone'
+    require_relative 'dsl/ellipse'
     require_relative 'dsl/grid'
     require_relative 'dsl/hand'
+    require_relative 'dsl/line'
     require_relative 'dsl/png'
+    require_relative 'dsl/polygon'
+    require_relative 'dsl/rect'
+    require_relative 'dsl/safe_zone'
     require_relative 'dsl/save_pdf'
     require_relative 'dsl/save_png'
     require_relative 'dsl/save_sheet'
     require_relative 'dsl/save'
     require_relative 'dsl/showcase'
+    require_relative 'dsl/star'
     require_relative 'dsl/svg'
+    require_relative 'dsl/triangle'
 
   end
 end

lib/squib/dsl/circle.rb

@@ -0,0 +1,39 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+require_relative '../args/card_range'
+require_relative '../args/coords'
+require_relative '../args/draw'
+
+module Squib
+  class Deck
+    def circle(opts = {})
+      DSL::Circle.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class Circle
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(x y
+           radius arc_start arc_end arc_direction arc_close
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        range = Args.extract_range opts, deck
+        coords = Args.extract_coords opts, deck
+        draw  = Args.extract_draw opts, deck
+        range.each { |i| deck.cards[i].circle(coords[i], draw[i]) }
+      end
+    end
+  end
+end

lib/squib/dsl/curve.rb

@@ -0,0 +1,35 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+
+module Squib
+  class Deck
+    def curve(opts = {})
+      DSL::Curve.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class Curve
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(x1 y1 x2 y2 cx1 cy1 cx2 cy2
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        range = Args.extract_range opts, deck
+        draw  = Args.extract_draw opts, deck
+        coords   = Args.extract_coords opts, deck
+        range.each { |i| deck.cards[i].curve(coords[i], draw[i]) }
+      end
+    end
+  end
+end

lib/squib/dsl/cut_zone.rb

@@ -0,0 +1,47 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+
+module Squib
+  class Deck
+    def cut_zone(opts = {})
+      DSL::CutZone.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class CutZone
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(x y width height margin angle
+           x_radius y_radius radius
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        cut_defaults = {
+          margin: '0.125in',
+          radius: '0.125in',
+          stroke_color: :red,
+          fill_color: '#0000',
+          stroke_width: 2.0,
+        }
+        new_opts = cut_defaults.merge(opts)
+        margin = Args::UnitConversion.parse new_opts[:margin]
+        new_opts[:x] = margin
+        new_opts[:y] = margin
+        new_opts[:width] = deck.width - (2 * margin)
+        new_opts[:height] = deck.height - (2 * margin)
+        new_opts.delete :margin
+        deck.rect new_opts
+      end
+    end
+  end
+end

lib/squib/dsl/ellipse.rb

@@ -0,0 +1,37 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+
+module Squib
+  class Deck
+    def ellipse(opts = {})
+      DSL::Ellipse.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class Ellipse
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(x y width height
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           angle
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        range = Args.extract_range opts, deck
+        draw  = Args.extract_draw opts, deck
+        box   = Args.extract_box opts, deck
+        trans = Args.extract_transform opts, deck
+        range.each { |i| deck.cards[i].ellipse(box[i], draw[i], trans[i]) }
+      end
+    end
+  end
+end

lib/squib/dsl/grid.rb

@@ -19,7 +19,7 @@ module Squib
 
       def self.accepted_params
         %i(x y width height
-           fill_color stroke_color stroke_width stroke_strategy dash cap
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
            range layout)
       end
 

lib/squib/dsl/line.rb

@@ -0,0 +1,35 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+
+module Squib
+  class Deck
+    def line(opts = {})
+      DSL::Line.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class Line
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(x1 y1 x2 y2
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        range = Args.extract_range opts, deck
+        draw  = Args.extract_draw opts, deck
+        coords   = Args.extract_coords opts, deck
+        range.each { |i| deck.cards[i].line(coords[i], draw[i]) }
+      end
+    end
+  end
+end

lib/squib/dsl/polygon.rb

@@ -0,0 +1,36 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+
+module Squib
+  class Deck
+    def polygon(opts = {})
+      DSL::Polygon.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class Polygon
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(n x y radius angle
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        range = Args.extract_range opts, deck
+        draw = Args.extract_draw opts, deck
+        coords = Args.extract_coords opts, deck
+        trans = Args.extract_transform opts, deck
+        range.each { |i| deck.cards[i].polygon(coords[i], trans[i], draw[i]) }
+      end
+    end
+  end
+end

lib/squib/dsl/rect.rb

@@ -0,0 +1,37 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+
+module Squib
+  class Deck
+    def rect(opts = {})
+      DSL::Rect.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class Rect
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(x y width height angle
+           x_radius y_radius radius
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        range = Args.extract_range opts, deck
+        draw  = Args.extract_draw opts, deck
+        box   = Args.extract_box opts, deck
+        trans   = Args.extract_transform opts, deck
+        range.each { |i| deck.cards[i].rect(box[i], draw[i], trans[i]) }
+      end
+    end
+  end
+end

lib/squib/dsl/safe_zone.rb

@@ -0,0 +1,48 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+
+module Squib
+  class Deck
+    def safe_zone(opts = {})
+      DSL::SafeZone.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class SafeZone
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(x y width height margin angle
+           x_radius y_radius radius
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        safe_defaults = {
+          margin: '0.25in',
+          radius: '0.125in',
+          stroke_color: :blue,
+          fill_color: '#0000',
+          stroke_width: 1.0,
+          dash: '3 3',
+        }
+        new_opts = safe_defaults.merge(opts)
+        margin = Args::UnitConversion.parse new_opts[:margin]
+        new_opts[:x] = margin
+        new_opts[:y] = margin
+        new_opts[:width] = deck.width - (2 * margin)
+        new_opts[:height] = deck.height - (2 * margin)
+        new_opts.delete :margin
+        deck.rect(new_opts)
+      end
+    end
+  end
+end

lib/squib/dsl/star.rb

@@ -0,0 +1,37 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+
+module Squib
+  class Deck
+    def star(opts = {})
+      DSL::Star.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class Star
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(n x y
+           inner_radius outer_radius angle
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        range  = Args.extract_range opts, deck
+        draw   = Args.extract_draw opts, deck
+        coords = Args.extract_coords opts, deck
+        trans  = Args.extract_transform opts, deck
+        range.each { |i| deck.cards[i].star(coords[i], trans[i], draw[i]) }
+      end
+    end
+  end
+end

lib/squib/dsl/triangle.rb

@@ -0,0 +1,35 @@
+require_relative '../errors_warnings/warn_unexpected_params'
+
+module Squib
+  class Deck
+    def triangle(opts = {})
+      DSL::Triangle.new(self, __callee__).run(opts)
+    end
+  end
+
+  module DSL
+    class Triangle
+      include WarnUnexpectedParams
+      attr_reader :dsl_method, :deck
+
+      def initialize(deck, dsl_method)
+        @deck = deck
+        @dsl_method = dsl_method
+      end
+
+      def self.accepted_params
+        %i(x1 y1 x2 y2 x3 y3
+           fill_color stroke_color stroke_width stroke_strategy join dash cap
+           range layout)
+      end
+
+      def run(opts)
+        warn_if_unexpected opts
+        range = Args.extract_range opts, deck
+        draw  = Args.extract_draw opts, deck
+        coords   = Args.extract_coords opts, deck
+        range.each { |i| deck.cards[i].triangle(coords[i], draw[i]) }
+      end
+    end
+  end
+end