plist4r 0.2.2 → 1.0.0

data/lib/plist4r/docs/Backends.html

@@ -0,0 +1,59 @@
+<h1>Backends</h1>
+<div>
+  <p>Documentation for the Plist4r backend modules - please see <tt><a href="Plist4r/Backend.html" target="_self" title="Plist4r::Backend (class)">Plist4r::Backend</a></tt></p>
+  <h3>Performance</h3>
+  <p>Time elapsed for encoding / decoding a non-flat (nested) plist structure of 1024 keys</p>
+  <p>Real elapsed time based on - 2GHz Intel Core Duo with 2GB Ram</p>
+  <p>Ruby Enterprise Edition (REE) 1.8.7 p248, Mac OS-X 10.6.3</p>
+  <table>
+    <thead>
+      <tr>
+        <th>&nbsp;</th>
+        <th style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #F5F5FF; text-align: center'>:from_xml</th>
+        <th style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #F5F5FF; text-align: center'>:to_xml</th>
+        <th style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #F5F5FF; text-align: center'>:from_binary</th>
+        <th style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #F5F5FF; text-align: center'>:to_binary</th>
+      </tr>
+    </thead>
+    <tbody>
+      <tr>
+        <th style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #F5F5FF; text-align: right'>c_f_property_list</th>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>18.2 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>19.5 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>33.4 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>34.9 ms</td>
+      </tr>
+      <tr>
+        <th style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #F5F5FF; text-align: right'>haml</th>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>n/a</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>126.8 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>n/a</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>n/a</td>
+      </tr>
+      <tr>
+        <th style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #F5F5FF; text-align: right'>libxml4r</th>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>41.4 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>n/a</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>n/a</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>n/a</td>
+      </tr>
+      <tr>
+        <th style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #F5F5FF; text-align: right'>osx_plist</th>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>4.8 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>5.0 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>4.6 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>4.6 ms</td>
+      </tr>
+      <tr>
+        <th style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #F5F5FF; text-align: right'>ruby_cocoa</th>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>320.4 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>318.8 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>319.7 ms</td>
+        <td style='padding: 5px; padding-left: 15px; padding-right: 15px; background-color: #FAFAFA; text-align: center'>319.2 ms</td>
+      </tr>
+    </tbody>
+  </table>
+  <p>To re-run the backend tests</p>
+  <pre class='code'>$ cd plist4r && rake backend:tests</pre>
+</div>
+<p></p>

data/lib/plist4r/docs/DeveloperGuide.rdoc

@@ -0,0 +1,53 @@
+== Plist4r Backends
+
+There are now a number of ruby libraries which can read / write plist files. The aim of plist4r is to utilize the individual best features from all of those libraries, as a series of "backends". And hide those behind a "frontend" that is easy to work with.
+
+Backends often only need to be a single ruby file, which implements the Plist4r API methods and calls out to other (existing) ruby code. No single backend has to provide the whole API. Instead, Plist4r simply iterates over all of the backends it knows about, and then calls the first backend that can responds to the API method.
+
+There are only 3 generally recognized Plist file formats. (the gnustep plist format also goes by the names openstep, and nextstep)
+
+  FileFormats = %w[ binary xml gnustep ]
+
+A plist4r backend can implement any number of these 6 supported API methods
+
+  ApiMethods = %w[ from_xml from_binary from_gnustep to_xml to_binary to_gnustep ]
+
+Other API methods are "generated" methods, which translate the call onto real backend API methods. A good example is +from_string()+, which gets resolved to +from_binary+ or +from_xml+. Generally speaking, those {Plist4r::Backend.PrivateApiMethods} methods are cached and should not be provided by a Plist4r backend
+
+  PrivateApiMethods = %w[ from_string open save ]
+
+For backends performance data see {file:Backends}.
+
+== Plist4r Types
+
+A Plist type can be one of +%w[plist info launchd]+, and is the data type for the whole plist file. A PlistType can provide special convenience methods for its Type-specific data structures. For example {Plist4r::PlistType::Launchd#socket}.
+
+We re-use common support objects when writing a new PlistType
+* {Plist4r::PlistType}
+* {Plist4r::ArrayDict}
+* {Plist4r::DataMethods}
+
+== Contributing to Plist4r
+
+* Fork the project, and create a topic branch as per {these instructions}[http://wiki.opscode.com/display/opscode/Working+with+Git]
+* Make the appropriate source code changes
+* Raise a new issue in Github Issues, with a name that corresponds to the topic branch name
+
+For a change in the {Plist4r} Core library
+
+* Please update the inline source code documentation, generate locally with +rake yard+
+* Please run the regression tests with +rake spec+ and +rake backend:tests+
+* Please update the existing regression tests (in RSpec) or write new ones
+
+For a {Plist4r::Backend}
+
+* Please include attribution with an @author tag in the yard inline comments
+* If requires new gem dependencies, be add to the Rakefile and run +rake gemspec+
+
+For a {Plist4r::PlistType}
+
+* Conform to the conventions used by {Plist4r::PlistType::Info} and {Plist4r::PlistType::Launchd}
+* Use {Plist4r::OrderedHash} instead of +Hash+ or +{}+
+* Try to use {Plist4r::DataMethods} and {Plist4r::ArrayDict} for storing the data
+* Use yard to help document any custom key functions and +rake yard+ afterward
+

data/lib/plist4r/docs/EditingPlistFiles.rdoc

@@ -0,0 +1,88 @@
+= Editing Plist Files
+
+== Editing
+
+In this example, we will be editing a Launchd plist file.
+
+When we wish to perform an edit operation on a plist object, we (almost always) are calling an accessor method on the Plist Object. We may call the method directly on the object, like this
+
+  launchd_plist.watch_paths ["/path1", "/path2", ...]
+
+However it gets a bit repetitive when there are many such plist keys to set
+
+  launchd_plist.label "com.mydomain.myapp"
+  launchd_plist.program "/path/to/myapp"
+  launchd_plist.launch_only_once true
+  # etc...
+  launchd_plist.save
+
+=== plist.edit do
+
+So instead we can invoke an convenience edit block on our plist object, which will just instance_eval(&blk) the block.
+
+  launchd_plist.edit do
+    label "com.mydomain.myapp"
+    program "/path/to/myapp"
+    launch_only_once true
+    # etc...
+    save
+  end
+
+=== plist.<< do
+
+The << operator can alternatively be used, interchangeably. Its just another way of writing plist.edit. 
+
+  launchd_plist.<< do
+    label "com.mydomain.myapp"
+    program "/path/to/myapp"
+    launch_only_once true
+  end
+  launchd_plist.save
+
+== Editing operations
+
+Certain kinds of edit operation are available on the plist keys. These are useful when we want to treat the plist keys like we would an Array object or a Hash object. Methods like {Plist4r::Plist#select}, {Plist4r::Plist#map}, {Plist4r::Plist#delete_if}, {Plist4r::Plist#clear} and similar. Such methods are all documented in {Plist4r::Plist}.
+
+== Plist Data (CFData / NSData)
+
+A plist file supports key-value pairs in several types, including Base64 encoded binary data. For example, a plist key that stores binary data might look something like this
+
+  <data>
+      PEKBpYGlmYFCPA==
+  </data>
+
+When its decoded, this is a byte stream (8-bit bytes), of some finite length. In Ruby we have no dedicated class to represent this, but instead can use a ruby +String+ as a binary string.
+
+In Plist4r, we extend {String} with 2 methods, {String#blob=} and {String#blob?}. This allows us to identify and differentiate a binary string from a regular string object.
+
+So to store binary data into a plist data key...
+
+  @plist = Plist4r.new
+  bstr = "my_binary_data"
+  bstr.blob = true # mark as a binary string
+  @plist.store "MyData", bstr
+
+...and to read or inspect the binary data in Ruby from a {Plist4r::Plist} object
+
+  @plist.my_data
+  => "my_binary_data"
+  @plist.my_data.blob?
+  => true
+
+If we forget the +blob?+ and +blob=+ methods, then we simple are storing and reading our data as a regular textual +String+. So heh, just remember to set +blob=+ true at some point.
+  
+If we want to perform byte stream IO operations on our binary string...
+
+  # wrap it in a StringIO object
+  stream = StringIO.new(@plist.my_data)
+  next_byte = stream.getc # or stream.putc(int) to write a byte
+
+See http://www.ruby-doc.org/core/classes/IO.html and http://www.ruby-doc.org/core/classes/StringIO.html for more information about the IO object classes.
+
+== Plist Types
+
+The class {Plist4r::PlistType} allows you to save those more complex data structures to specific plist keys in a plist file. For example the method {Plist4r::PlistType::Launchd#socket} will construct a Socket entry for a launchd plist file.
+
+If you are developing a custom application, and intend to exchange data in a custom plist file format, it may be worth writing a custom Plist Type. In which case please see the {file:DeveloperGuide} for more info.
+
+

data/lib/plist4r/docs/InfoPlistExample.rdoc

@@ -0,0 +1,33 @@
+= Info Plist Example
+
+In this example, we edit an Apple Info.plist, to modify a couple of the keys and write them back to the same file. For setting the plist file attributes see {Plist4r::Plist}. For more information specifically about Info Plists, see {Plist4r::PlistType::Info}.
+
+  # standard method
+  info_plist = Plist4r.open "/Applications/MyApp.app/Contents/Info.plist"
+  info_plist[:c_f_bundle_signature] # => "????"
+  info_plist["LSUIElement"] = true
+  info_plist.save
+
+  # block method
+  info_plist = Plist4r.open "/Applications/MyApp.app/Contents/Info.plist" do
+
+    # plist attributes
+    filename    # => "/Applications/MyApp.app/Contents/Info.plist"
+    file_format # => :xml
+    plist_type  # => :info
+
+    # read
+    c_f_bundle_executable # => "MyApp"
+    c_f_bundle_icon_file  # => "Icon"
+    c_f_bundle_identifier # => "com.mydomain.MyApp"
+    c_f_bundle_name       # => "MyApp"
+    c_f_bundle_signature  # => "????"
+
+    # write
+    c_f_bundle_signature "MYAP"
+    store "LSUIElement" true
+    
+    # call info_plist.save
+    save
+  end
+

data/lib/plist4r/docs/LaunchdPlistExample.rdoc

@@ -0,0 +1,33 @@
+= Launchd Plist Example
+
+In this example, we edit an Apple Launchd plist, to modify a couple of the keys and write them back to the same file. For setting the plist file attributes see {Plist4r::Plist}. For more information specifically about Launchd Plists, see {Plist4r::PlistType::Launchd}.
+
+  # standard method
+  launchd_plist = Plist4r.open "/Library/LaunchDaemons/com.mydomain.MyApp.plist"
+  launchd_plist[:label] # => "com.mydomain.Myapp"
+  launchd_plist[:start_on_mount] = true
+  launchd_plist.save
+
+  # block method
+  launchd_plist = Plist4r.open "/Library/LaunchDaemons/com.mydomain.MyApp.plist" do
+
+    # plist attributes
+    filename    # => "/Library/LaunchDaemons/com.mydomain.MyApp.plist"
+    file_format # => :xml
+    plist_type  # => :launchd
+
+    # read
+    label             # => "com.mydomain.Myapp"
+    program_arguments # => ["/Applications/MyApp.app/Contents/MacOS/MyApp"]
+    queue_directories # => ["/dir/to/watch/1","/dir/to/watch/2","etc..."]
+    run_at_load    # => false
+    start_on_mount # => false
+
+    # write
+    run_at_load true
+    store "StartOnMount" true
+    
+    # call launchd_plist.save
+    save
+  end
+

data/lib/plist4r/docs/PlistKeyNames.rdoc

@@ -0,0 +1,47 @@
+= Plist Key Names
+
+To read from or write to a plist key, we must specify the key name, plus an optional value. There are several ways we can specify key names in Plist4r. The precise method syntax you choose will depend mostly upon convenience. However there are certain limitations to method names in Ruby in regards to uppercase / lowercase. And plist key names are usually capitalized (such as "WatchPaths", and "CFBundleSignature"). To get around this we "CamelCase" and "snake_case" the key accessor methods, by default. Dont forget that many Info plist keys start with 2 capital letters. So "CFBundleSignature" should be written with an extra underscore between the "c" and the "f"
+
+=== CF/NSKeyNames
+
+  info_plist.<< do
+    c_f_bundle_signature "com.mydomain.myapp"
+  end
+  info_plist.to_hash
+  # => { "CFBundleSignature" => "com.mydomain.myapp" }
+
+=== Corrected Key Names
+
+Sometimes, the associated PlistType will come with a "tweaked" method name, is the true keyname cannot be expressed in terms of snake_case. For example
+
+  launchd_plist.<< do
+    inetd_compatibility :wait => true
+  end
+  launchd_plist.to_hash
+  # => { "inetdCompatability" => { "Wait" => true } }
+  # (the CamelCased key name would otherwise start with a capital letter, "InetdCompatability")
+
+=== CamelCasing of Key Names
+
+For an unknown key name which isnt specifically supported, these are all passed through method_missing, and automatically CamelCased. This is the default behaviour. Therefore
+
+  plist.<< do
+    some_arbitrary_key ["some","arbitrary","data"]
+  end
+  plist.to_hash
+  # => { "SomeArbitraryKey" => ["some", "arbitrary", "data"] }
+
+=== store(), [] and []=
+
+Sometimes a plist key name will be in lowercase, and we cannot use method_missing(). To store or read the plist key we call the regular Hash accessor methods with the literal key name. See {Plist4r::Plist#store}, {Plist4r::Plist#[]}, and {Plist4r::Plist#[]=}.
+
+  # writing with plist.store("key",value)
+  plist.<< do
+    store "keyname" "value"
+  end
+
+  # reading and writing with plist["key"], and plist["key"]="value"
+  plist["keyname"] = "value"
+  plist["keyname"] # => "value"
+
+

data/lib/plist4r/mixin/array_dict.rb

@@ -0,0 +1,61 @@
+
+require 'plist4r/mixin/data_methods'
+
+module Plist4r
+  # Abstract Base class. Represents some nested data structure within an open {Plist4r::Plist}.
+  # Typically, a {Plist4r::PlistType} will create and build upon nested instances of this class.
+  class ArrayDict
+    include Plist4r::DataMethods
+
+    # The initializer for this object. Here we set a reference to our raw data structure,
+    # which typically is a nested hash within the plist root hash object.
+    # Or an Array type structure if index is set.
+    # @param [OrderedHash] orig The nested hash object which this structure represents.
+    # @param [Fixnum] index The Array index (if representing an Array structure)
+    def initialize orig, index=nil, &blk
+      @orig = orig
+      @orig = @orig[index] if index
+      @hash = ::Plist4r::OrderedHash.new
+
+      @block = blk
+      instance_eval(&@block) if block_given?
+    end
+
+    # The raw data object
+    # @return [Plist4r::OrderedHash] @hash
+    def hash
+      @hash
+    end
+
+    # Select (keep) plist keys from the object.
+    # Copy them to the resultant object moving forward.
+    # @param [Array, *args] keys The list of Plist Keys to keep
+    def select *keys
+      keys.flatten.each do |k|
+        k = k.to_s.camelcase if k.class == Symbol
+        @hash[k] = @orig[k] if @orig[k]
+      end
+    end
+
+    # Unselect (delete) plist keys from the object.
+    # @param [Array, *args] keys The list of Plist Keys to delete
+    def unselect *keys
+      @hash = @orig
+      keys.flatten.each do |k|
+        k = k.to_s.camelcase if k.class == Symbol
+        @hash.delete k
+      end
+    end
+
+    # Unselect (delete) all plist keys from the object.
+    def unselect_all
+      @hash = Plist4r::OrderedHash.new
+    end
+
+    # Select (keep) all plist keys from the object.
+    # Copy them to the resultant object moving forward.
+    def select_all
+      @hash = @orig
+    end
+  end
+end

data/lib/plist4r/mixin/data_methods.rb

@@ -5,99 +5,223 @@ require 'plist4r/mixin/ordered_hash'
 module Plist4r
   module DataMethods
 
-    def classes_for_key_type
-      {
-        :string => [String], 
-        :bool => [TrueClass,FalseClass],
-        :integer => [Fixnum], 
-        :array_of_strings => [Array],
-        :hash_of_bools => [Hash],
-        :hash => [Hash],
-        :bool_or_string_or_array_of_strings => [TrueClass,FalseClass,String,Array]
-      }
-    end
+    # Return those Class constants which the value of a Plist Key must conform to.
+    # This is class-based data validation.
+    ClassesForKeyType = 
+    {
+      :string => [String], 
+      :bool => [TrueClass,FalseClass],
+      :bool_or_string => [TrueClass,FalseClass,String],
+      :integer => [Fixnum], 
+      :array_of_strings => [Array],
+      :array_of_hashes => [Array],
+      :array => [Array],
+      :array_or_integer => [Array,Fixnum],
+      :array_or_hash => [Array, Hash],
+      :hash_of_bools => [Hash],
+      :hash_of_strings => [Hash],
+      :hash_of_arrays => [Hash],
+      :hash_of_arrays_of_strings => [Hash],
+      :hash => [Hash],
+      :bool_or_string_or_array_of_strings => [TrueClass,FalseClass,String,Array]
+    }
+
+    # A Hash Array of the supported plist keys for this type. These are only those plist keys 
+    # recognized as belonging to a specific plist datatype. Used in validation, part of DataMethods.
+    # We usually overload this method in subclasses. To see how to use these Plist Keys, go to {file:PlistKeyNames}
+    # @example
+    #  class Plist4r::PlistType::MyPlistType < PlistType
+    #    ValidKeys =
+    #    {
+    #       :string => %w[PlistKeyS1 PlistKeyS2 ...],
+    #       :bool => %w[PlistKeyB1 PlistKeyB2 ...],
+    #       :integer => %w[PlistKeyI1 PlistKeyI2 ...],
+    #       :method_defined => %w[CustomPlistKey1 CustomPlistKey2 ...]
+    #    }
+    #  end
+    #
+    #  plist.plist_type :my_plist_type
+    #  plist.plist_key_s1 "some string"
+    #  plist.plist_key_b1 true
+    #  plist.plist_key_i1 08
+    #  plist.custom_plist_key1 MyClass.new(opts)
+    # @see ClassesForKeyType
+    ValidKeys = {}
 
-    def valid_keys
-      {}
+    # A template for creating new plist types
+    ValidKeysTemplate =
+    {
+      :string => %w[
+        
+        ],
+      :bool => %w[
+        
+        ],
+      :integer => %w[
+        
+        ],
+      :array_of_strings => %w[
+        
+        ],
+      :array_of_hashes => %w[
+        
+        ],
+      :array => %w[
+        
+        ],
+      :hash_of_bools => %w[
+        
+        ],
+      :hash_of_strings => %w[
+        
+        ],
+      :hash_of_arrays => %w[
+        
+        ],
+      :hash_of_arrays_of_strings => %w[
+        
+        ],
+      :hash => %w[
+        
+        ],
+      :could_be_method_defined   => %w[
+        
+        ]
+    }
+
+    # Defining respond_to? conflicts with rspec 1.3.0. @object.stub() blows up.
+    # To as a consequence we dont implement respond_to? for method_missing()
+    # @return true
+    def _respond_to? method_sym
+      true
     end
 
+    # Call {#set_or_return} with the appropriate arguments. If {Plist4r::Plist#strict_keys} is enabled, 
+    # then raise an error on any unrecognised Plist Keys.
     def method_missing method_symbol, *args, &blk
-      # puts "method_missing: #{method_symbol.inspect}, args: #{args.inspect}"
-      # puts "@hash = #{@hash.inspect}"
-      valid_keys.each do |key_type, valid_keys_of_those_type|
-        if valid_keys_of_those_type.include?(method_symbol.to_s.camelcase)
-          puts "key_type = #{key_type}, method_symbol.to_s.camelcase = #{method_symbol.to_s.camelcase}, args = #{args.inspect}"
-          # return eval("set_or_return key_type, method_symbol.to_s.camelcase, *args, &blk")
-          return set_or_return key_type, method_symbol.to_s.camelcase, *args, &blk
+      set_or_return method_symbol.to_s.camelcase, *args, &blk
+    end
+
+    # Set or return value data
+    # @param [Symbol, String] key The plist key name, either a snake-cased symbol, or literal string
+    # @param [nil, Object] value If present, store Object under the plist key "key". If nil, return the stored object
+    # @return The Object stored under key
+    def set_or_return key, value=nil
+      key = key.to_s.camelcase if key.class == Symbol
+
+      self.class::ValidKeys.each do |key_type, valid_keys_of_those_type|
+        if valid_keys_of_those_type.include?(key)
+          return set_or_return_of_type key_type, key, value
         end
       end
-      # puts @plist.inspect
+
       unless @plist.strict_keys
         key_type = nil
-        # return eval("set_or_return key_type, method_symbol.to_s.camelcase, *args, &blk")
-        return set_or_return key_type, method_symbol.to_s.camelcase, *args, &blk
+        return set_or_return_of_type key_type, key, value
       else
-        raise "Unrecognized key for class: #{self.class.inspect}. Tried to set_or_return #{method_symbol.inspect}, with: #{args.inspect}"
+        raise "Unrecognized key for class: #{self.class.inspect}. Tried to set_or_return #{key.inspect}, with: #{value.inspect}"
       end
-      # puts "bob"
     end
 
+    # This method is called when setting a value to a plist key. (or some value within a nested plist sub-structure).
+    # @param [Symbol] key_type This Symbol is resolved to a class constant, by looking it up in {ClassesForKeyType}
+    # @param value The value to validate. We just check that the value conforms to key_type.
+    # @see ClassesForKeyType
+    # @raise Class mistmatch
+    # @example
+    #  plist.validate_value :string, "CFBundleIdentifier", "com.apple.myapp"
+    #  # => Okay, no error raised
+    #  plist.validate_value :string, "CFBundleIdentifier", ["com.apple.myapp"]
+    #  # => Raises Class mismatch. Value is of type Array, should be String
     def validate_value key_type, key, value
-      unless classes_for_key_type[key_type].include? value.class
-        raise "Key: #{key}, value: #{value.inspect} is of type #{value.class}. Should be: #{classes_for_key_type[key_type].join ", "}"
+      unless ClassesForKeyType[key_type].include? value.class
+        raise "Key: #{key}, value: #{value.inspect} is of type #{value.class}. Should be: #{ClassesForKeyType[key_type].join ", "}"
       end
       case key_type
       when :array_of_strings, :bool_or_string_or_array_of_strings
         if value.class == Array
           value.each_index do |i|
             unless value[i].class == String
-              raise "Element: #{key}[#{i}], value: #{value[i].inspect} is of type #{value[i].class}. Should be: #{classes_for_key_type[:string].join ", "}"
+              raise "Element: #{key}[#{i}], value: #{value[i].inspect} is of type #{value[i].class}. Should be: #{ClassesForKeyType[:string].join ", "}"
             end
           end
         end
+      when :array_of_hashes
+        value.each_index do |i|
+          unless value[i].class == Hash
+            raise "Element: #{key}[#{i}], value: #{value[i].inspect} is of type #{value[i].class}. Should be: #{ClassesForKeyType[:hash].join ", "}"
+          end
+        end
       when :hash_of_bools
         value.each do |k,v|
           unless [TrueClass,FalseClass].include? v.class
-            raise "Key: #{key}[#{k}], value: #{v.inspect} is of type #{v.class}. Should be: #{classes_for_key_type[:bool].join ", "}"
+            raise "Key: #{key}[#{k}], value: #{v.inspect} is of type #{v.class}. Should be: #{ClassesForKeyType[:bool].join ", "}"
+          end
+        end
+      when :hash_of_strings
+        value.each do |k,v|
+          unless v.class == String
+            raise "Key: #{key}[#{k}], value: #{v.inspect} is of type #{v.class}. Should be: #{ClassesForKeyType[:string].join ", "}"
+          end
+        end
+      when :hash_of_arrays
+        value.each do |k,v|
+          unless v.class == Array
+            raise "Key: #{key}[#{k}], value: #{v.inspect} is of type #{v.class}. Should be: #{ClassesForKeyType[:array].join ", "}"
+          end
+        end
+      when :hash_of_arrays_of_strings
+        value.each do |k,v|
+          unless v.class == Array
+            raise "Key: #{key}[#{k}], value: #{v.inspect} is of type #{v.class}. Should be: #{ClassesForKeyType[:array].join ", "}"
+          end
+          v.each_index do |i|
+            unless v[i].class == String
+              raise "Element: #{key}[#{k}][#{i}], value: #{v[i].inspect} is of type #{v[i].class}. Should be: #{ClassesForKeyType[:string].join ", "}"
+            end
           end
         end
       end
     end
 
-    # Set a plist key to a specific value
-    # @param [String] The Plist key, as-is
-    # @param value A ruby object (Hash, String, Array, etc) to set as the value of the plist key
-    # @example
-    # plist.set "CFBundleIdentifier", "com.apple.myapp"
-    # @see #set_or_return
-    def set key, value
-      set_or_return nil, key, value
-    end
+    # # Set a plist key to a specific value
+    # # @param [String] The Plist key, as-is
+    # # @param value A ruby object (Hash, String, Array, etc) to set as the value of the plist key
+    # # @see #set_or_return
+    # # @example
+    # #  plist.set "CFBundleIdentifier", "com.apple.myapp"
+    # def set key, value
+    #   set_or_return key, value
+    # end
 
-    # Return the value of an existing plist key
-    # @return The key's current value, at the time this method was called 
-    # @see #set_or_return
-    def value_of key
-      set_or_return nil, key
-    end
+    # # Return the value of an existing plist key
+    # # @return The key's current value, at the time this method was called 
+    # # @see #set_or_return
+    # # @example
+    # #  plist.value_of "CFBundleIdentifier"
+    # #  # => "com.apple.myapp"
+    # def value_of key
+    #   set_or_return key
+    # end
 
-    # Set or return a plist key, value pair
+    # Set or return a plist key, value pair, and verify the value type
     # @param [Symbol, nil] key_type The type of class which the value of the key must belong to. Used for validity check.
     # If key_type is set to nil, then skip value data check
     # @return the key's value
     # @see #validate_value
-    def set_or_return key_type, key, value=nil
-      # puts "#{method_name}, key_type: #{key_type.inspect}, key: #{key.inspect}, value: #{value.inspect}"
-      if value
+    # @example
+    #  plist.set_or_return_of_type :string, "CFBundleIdentifier", "com.apple.myapp"
+    #  
+    #  plist.set_or_return_of_type nil, "SomeUnknownKey", [[0],1,2,{ 3 => true}]
+    #  # Skips validation
+    def set_or_return_of_type key_type, key, value=nil
+      case value
+      when nil
+        @orig[key]
+      else
         validate_value key_type, key, value unless key_type == nil
         @hash[key] = value
-      else
-        @orig[key]
       end
     end
   end
 end
-
-
-
-