Remove usage of openstruct for config object

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 * `Unreleased` - [View Diff](https://github.com/cortiz/prawn-rails/compare/v1.4.2...master)
   - [#52](https://github.com/cortiz/prawn-rails/pull/52) - Add active_support as a dependency and use the active_support lazy load hooks
+  - [#51](https://github.com/cortiz/prawn-rails/pull/51) - Remove usage of openstruct for config object
   - Remove unnecessary method `get_metadata` from `PrawnRailsHelper` which was getting included into `ActionView::Base`
   - Remove unnecessary rescue in `PrawnRails.config`
   - Remove redundant method `PrawnRails::Document.extensions`

lib/prawn-rails/config.rb

@@ -1,12 +1,29 @@
-require 'ostruct'
+require "active_support/hash_with_indifferent_access"
 
 module PrawnRails
-  extend self
+  class Config < HashWithIndifferentAccess
+    def method_missing(method_name, *args)
+      # to replace behaviour of previous openstruct-based configuration object
 
-  @config = OpenStruct.new(page_layout: :portrait, page_size: "A4", skip_page_creation: false)
+      method_name = method_name.to_s
+
+      if method_name.end_with?("=")
+        key = method_name.sub(/=$/, "")
+        self[key] = args.first
+      else
+        self[method_name]
+      end
+    end
+  end
+
+  @config = Config.new.merge(
+    page_layout: :portrait,
+    page_size: "A4",
+    skip_page_creation: false,
+  )
 
   def config(&block)
     block_given? ? yield(@config) : @config
   end
-
+  module_function :config
 end

lib/prawn-rails/document.rb

@@ -4,7 +4,7 @@
 module PrawnRails
   class Document < Prawn::Document
     def initialize(options = {})
-      options = PrawnRails.config.marshal_dump.merge(options) ### For Ruby 1.9.3 support, use `marshal_dump` instead of `to_h`
+      options = PrawnRails.config.merge(options)
 
       super(options)
     end

test/unit/config_test.rb

@@ -19,7 +19,7 @@ def teardown
   end
 
   test "has a default config" do
-    assert_equal PrawnRails.config.to_h, {page_layout: :portrait, page_size: "A4", skip_page_creation: false}
+    assert_equal PrawnRails.config.to_h.symbolize_keys, {page_layout: :portrait, page_size: "A4", skip_page_creation: false}
   end
 
   test "config can be set with block syntax" do
@@ -28,7 +28,7 @@ def teardown
       config.page_size = "A8"
     end
 
-    assert_equal PrawnRails.config.to_h, {page_layout: :landscape, page_size: "A8", skip_page_creation: false}
+    assert_equal PrawnRails.config.to_h.symbolize_keys, {page_layout: :landscape, page_size: "A8", skip_page_creation: false}
   end
 
 end