release_group.rb in musicbrainz-0.5.0

- old
+ new
@@ -1,30 +1,30 @@
 module MusicBrainz
-  class ReleaseGroup
+  class ReleaseGroup < MusicBrainz::Base
     attr_accessor :id, :type, :title, :disambiguation, :first_release_date
     @releases
   
     def releases
       if @releases.nil? and not self.id.nil?
         @releases = []
-        Nokogiri::XML(MusicBrainz.load('http://musicbrainz.org/ws/2/release/?release-group=' + self.id + '&inc=media&limit=100')).css('release').each do |r|
+        Nokogiri::XML(MusicBrainz.load(:release, :release_group => self.id, :inc => [:media], :limit => 100)).css('release').each do |r|
           @releases << MusicBrainz::Release.parse_xml(r)
         end
       end
       @releases.sort{ |a, b| a.date <=> b.date }
     end
   
     def self.find mbid
-      xml = Nokogiri::XML(MusicBrainz.load('http://musicbrainz.org/ws/2/release-group/' + mbid)).css('release-group').first
+      xml = Nokogiri::XML(MusicBrainz.load(:release_group, :id => mbid)).css('release-group').first
       self.parse_xml(xml) unless xml.nil?
     end
   
     def self.parse_xml xml
       @release_group = MusicBrainz::ReleaseGroup.new
-      @release_group.id = xml.attr('id')
-      @release_group.type = xml.attr('type')
-      @release_group.title = xml.css('title').text
-      @release_group.disambiguation = xml.css('disambiguation').empty? ? '' : xml.css('disambiguation').text
+      @release_group.id = self.safe_get_attr(xml, nil, 'id')
+      @release_group.type = self.safe_get_attr(xml, nil, 'type')
+      @release_group.title = self.safe_get_value(xml, 'title')
+      @release_group.disambiguation = self.safe_get_value(xml, 'disambiguation')
       date = xml.css('first-release-date').empty? ? '2030-12-31' : xml.css('first-release-date').text
       if date.length == 0
         date = '2030-12-31'
       elsif date.length == 4
         date += '-12-31'