opera-mobile-store-sdk 0.1.1 → 0.1.2

data/lib/opera/mobile_store/product_image.rb

@@ -1,43 +1,41 @@
-module Opera
-  module MobileStore
-    class ProductImage
-
-      include ActiveModel::Model
-      include ActiveModel::Serialization
-
-      # All attributes are Read-Only...
-      attr_accessor :type, :width, :height, :url
-
-      def attributes
-        [:type, :width, :height, :url].inject({}) do |hash, field_name|
-          field_value = self.public_send field_name
-          hash[field_name.to_s] = field_value if field_value.present?
-          hash
-        end
-      end
+require "active_model"
 
-      def self.build_from_nokogiri_node(node)
+module Opera::MobileStore
+  class ProductImage
 
-        data = {
-          type: node.name,
-          url: node.text.strip
-        }
+    include ActiveModel::Model
+    include ActiveModel::Serialization
 
-        # Extract width + height data:
-        width = node.xpath("string(@width)")
-        height = node.xpath("string(@height)")
-        data[:width] = width.to_i if width.present?
-        data[:height] = width.to_i if height.present?
+    include Opera::MobileStore::InspectableAttributes
 
-        self.new data
-      end
+    # All attributes are Read-Only...
+    attr_accessor :type, :width, :height, :url
 
-      def inspect
-        "<#{self.class.name} #{attributes.inspect}>"
+    def attributes
+      [:type, :width, :height, :url].inject({}) do |hash, field_name|
+        field_value = self.public_send field_name
+        hash[field_name.to_s] = field_value if field_value.present?
+        hash
       end
+    end
+
+    def self.build_from_nokogiri_node(node)
 
-      self.singleton_class.send :alias_method, :deserialize, :new
+      data = {
+        type: node.name,
+        url: node.text.strip
+      }
 
+      # Extract width + height data:
+      width = node.xpath("string(@width)")
+      height = node.xpath("string(@height)")
+      data[:width] = width.to_i if width.present?
+      data[:height] = width.to_i if height.present?
+
+      self.new data
     end
+
+    self.singleton_class.send :alias_method, :deserialize, :new
+
   end
 end

data/lib/opera/mobile_store/product_localization.rb

@@ -1,32 +1,30 @@
-module Opera
-  module MobileStore
-    class ProductLocalization
+require "active_model"
 
-      include ActiveModel::Model
-      include ActiveModel::Serialization
+module Opera::MobileStore
+  class ProductLocalization
 
-      # All attributes are Read-Only...
-      attr_accessor :language_code,
-                    :title,
-                    :short_description,
-                    :long_description
+    include ActiveModel::Model
+    include ActiveModel::Serialization
 
-      def attributes
-        [
-          :language_code, :title, :short_description, :long_description
-        ].inject({}) do |hash, field_name|
-          field_value = self.public_send field_name
-          hash[field_name.to_s] = field_value if field_value.present?
-          hash
-        end
-      end
+    include Opera::MobileStore::InspectableAttributes
+
+    # All attributes are Read-Only...
+    attr_accessor :language_code,
+                  :title,
+                  :short_description,
+                  :long_description
 
-      def inspect
-        "<#{self.class.name} #{attributes.inspect}>"
+    def attributes
+      [
+        :language_code, :title, :short_description, :long_description
+      ].inject({}) do |hash, field_name|
+        field_value = self.public_send field_name
+        hash[field_name.to_s] = field_value if field_value.present?
+        hash
       end
+    end
 
-      self.singleton_class.send :alias_method, :deserialize, :new
+    self.singleton_class.send :alias_method, :deserialize, :new
 
-    end
   end
 end

data/lib/opera/mobile_store_sdk/identity_mapable.rb

@@ -8,8 +8,16 @@ module Opera
       def initialize(params={})
         super
 
-        # Register in the identity map:
-        self.class.identity_map[self.id] = self
+        # Upsert in the identity map:
+        if self.class.identity_map.key? self.id
+          params.each do |field_name, field_value|
+            self.class.identity_map[self.id].public_send(
+              "#{field_name}=", field_value
+            )
+          end
+        else
+          self.class.identity_map[self.id] = self
+        end
       end
 
       module ClassMethods
@@ -26,7 +34,7 @@ module Opera
           end
         end
 
-        def registered?(object_id)
+        def identity_mapped?(object_id)
           identity_map.key? object_id
         end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opera-mobile-store-sdk
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Roberto Quintanilla
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2015-04-22 00:00:00.000000000 Z
+date: 2015-04-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -127,11 +127,11 @@ files:
 - bin/setup
 - lib/opera-mobile-store-sdk.rb
 - lib/opera-mobile-store-sdk/version.rb
-- lib/opera/mobile_store/author.rb
 - lib/opera/mobile_store/build.rb
 - lib/opera/mobile_store/build_file.rb
 - lib/opera/mobile_store/category.rb
 - lib/opera/mobile_store/developer.rb
+- lib/opera/mobile_store/inspectable_attributes.rb
 - lib/opera/mobile_store/payment_info.rb
 - lib/opera/mobile_store/product.rb
 - lib/opera/mobile_store/product_image.rb

data/lib/opera/mobile_store/author.rb

@@ -1,29 +0,0 @@
-module Opera
-  module MobileStore
-    class Author
-
-      include ActiveModel::Model
-      include ActiveModel::Serialization
-
-      include Opera::MobileStoreSDK::IdentityMapable
-
-      # All attributes are Read-Only...
-      attr_accessor :id, :name, :email
-
-      def attributes
-        [:id, :name, :email].inject({}) do |hash, field_name|
-          value = self.public_send field_name
-          hash[field_name.to_s] = value unless value.nil?
-          hash
-        end
-      end
-
-      def inspect
-        "<#{self.class.name} #{attributes.inspect}>"
-      end
-
-      self.singleton_class.send :alias_method, :deserialize, :new
-
-    end
-  end
-end