amee 2.2.1 → 2.3.0

data/CHANGELOG

@@ -1,5 +1,9 @@
 = Changelog
 
+== 2.3.0
+ * Remove misleading ItemValueDefinition#type function. Use profile?, data? and
+   drill? separately instead.
+
 == 2.2.0
 
 * Add SSL support. SSL connections are now used by default.

data/lib/amee/exceptions.rb

@@ -38,4 +38,8 @@ module AMEE
 
   class NotSupported < Exception
   end
+
+  class Deprecated < Exception
+  end
+  
 end

data/lib/amee/item_value_definition.rb

@@ -30,8 +30,10 @@ module AMEE
         data[:unit] = p['unit']
         data[:perunit] = p['perunit']
         data[:valuetype] = p['valueDefinition']['valueType']
-        data=ItemValueDefinition.parsetype(data,p['drillDown'],p['fromProfile'],p['fromData'])
-        
+        data[:drill]=p['drillDown']
+        data[:from_profile]=p['fromProfile']
+        data[:from_data]=p['fromData']
+        data
       end
       def parse_xml(p)
         data = {}
@@ -42,11 +44,10 @@ module AMEE
         data[:unit] = x 'Unit',:doc => p
         data[:perunit] = x 'PerUnit',:doc => p
         data[:valuetype] =x 'ValueDefinition/ValueType',:doc => p
-        drill=(x('DrillDown',:doc => p)=='true')
-        profile=(x('FromProfile',:doc => p)=='true')
-        ldata=(x('FromData',:doc => p)=='true')
-        data=ItemValueDefinition.parsetype(data,drill,profile,ldata)
-        
+        data[:drill]=(x('DrillDown',:doc => p)=='true')
+        data[:from_profile]=(x('FromProfile',:doc => p)=='true')
+        data[:from_data]=(x('FromData',:doc => p)=='true')
+        data
       end
     end
 
@@ -57,28 +58,35 @@ module AMEE
         @name = data[:name]
         @uid = data[:uid]
         @path = data[:path]
-        @type = data[:type]
         @unit = data[:unit]
         @perunit = data[:perunit]
         @valuetype = data[:valuetype]
         @default = data[:default] == "" ? nil : data[:default]
         @choices = data[:choices] || []
         @versions = data[:versions] || []
+        @drill = data[:drill]
+        @from_data = @drill ? false : data[:from_data] # Drills are also data, but we don't want them to show as such for clarity
+        @from_profile = data[:from_profile]
         super
       end
 
-      attr_reader :name,:uid,:path,:itemdefuid, :type, :valuetype, :unit, :perunit, :type, :default, :choices, :versions
+      attr_reader :name,:uid,:path,:itemdefuid, :type, :valuetype, :unit, :perunit, :default, :choices, :versions
+
+      # This function has bee deprecated in favour of the separate profile? data? and drill? functions
+      def type
+        raise AMEE::Deprecated
+      end
 
       def profile?
-        type=='profile'
+        @from_profile
       end
 
       def data?
-        type=='data'
+        @from_data
       end
 
       def drill?
-        type=='drill'
+        @drill
       end
 
       def self.list(connection)
@@ -115,19 +123,17 @@ module AMEE
         data[:default] = p['value']
         data[:choices] = p['choices'].split(',')
         data[:versions] = p['apiVersions'].map{|v| v['apiVersion']}
-        data=ItemValueDefinition.parsetype(data,p['drillDown'],p['fromProfile'],p['fromData'])
+        data[:drill]=p['drillDown']
+        data[:from_profile]=p['fromProfile']
+        data[:from_data]=p['fromData']
         # Create object
         ItemValueDefinition.new(data)
       rescue
         raise AMEE::BadData.new("Couldn't load ItemValueDefinition from JSON. Check that your URL is correct.\n#{json}")
       end
 
-      def self.parsetype(data,drill,profile,ldata)
-        ( (profile && !(drill||ldata) ) || ldata) or raise 'inconsistent type'
-        data[:type]='profile' if profile
-        data[:type]='data' if ldata
-        data[:type]='drill' if drill
-        data
+      def self.xmlpathpreamble
+        "/Resources//ItemValueDefinition/"
       end
 
       def self.from_xml(xml, is_list = true)
@@ -149,10 +155,9 @@ module AMEE
         data[:default] = REXML::XPath.first(doc, prefix + "ItemValueDefinition/Value").text
         data[:choices] = REXML::XPath.first(doc, prefix + "ItemValueDefinition/Choices").text.split(',') rescue nil
         data[:versions] = REXML::XPath.each(doc, prefix + "ItemValueDefinition/APIVersions/APIVersion").map{|v| v.text}
-        drill=(REXML::XPath.first(doc, prefix + "ItemValueDefinition/DrillDown").text=='true')
-        profile=(REXML::XPath.first(doc, prefix + "ItemValueDefinition/FromProfile").text=='true')
-        ldata=(REXML::XPath.first(doc, prefix + "ItemValueDefinition/FromData").text=='true')
-        data=parsetype(data,drill,profile,ldata)
+        data[:drill]=(x('DrillDown',:doc => doc)=='true')
+        data[:from_profile]=(x('FromProfile',:doc => doc)=='true')
+        data[:from_data]=(x('FromData',:doc => doc)=='true')
         # Create object
         ItemValueDefinition.new(data)
       rescue

data/lib/amee/version.rb

@@ -2,8 +2,8 @@ module AMEE
   
   module VERSION #:nodoc:
     MAJOR = 2
-    MINOR = 2
-    TINY  = 1
+    MINOR = 3
+    TINY  = 0
     STRING = [MAJOR, MINOR, TINY].join('.')
   end
   

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: amee
 version: !ruby/object:Gem::Version 
-  hash: 5
+  hash: 3
   prerelease: false
   segments: 
   - 2
-  - 2
-  - 1
-  version: 2.2.1
+  - 3
+  - 0
+  version: 2.3.0
 platform: ruby
 authors: 
 - James Smith
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-08-24 00:00:00 +01:00
+date: 2010-09-23 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency