On Mon, 2013-01-28 at 12:32 +0100, mfoj...@redhat.com wrote:
> From: Michal Fojtik <mfoj...@redhat.com>
> 
> 
> Signed-off-by: Michal fojtik <mfoj...@redhat.com>

ACK, though

> --- a/server/lib/cimi/models/collection.rb
> +++ b/server/lib/cimi/models/collection.rb
> @@ -24,11 +24,15 @@ module CIMI::Model
>      self.schema
>  
>      def initialize(values = {})
> -      if values[:entries]
> -        values[self.class.entry_name] = values.delete(:entries)
> +      if values.kind_of?(Hash) 
> +        if values[:entries]
> +          values[self.class.entry_name] = values.delete(:entries)
> +        end
> +        values[self.class.entry_name] ||= []
> +        super(values)
> +      else
> +        super

This still needs a comment about what besides a Hash can be passed in.

David


Reply via email to