sirportly 1.1.7 → 1.2.0

data/lib/sirportly.rb

@@ -16,7 +16,7 @@ require 'sirportly/data_objects/department'
 require 'sirportly/data_objects/escalation_path'
 require 'sirportly/data_objects/filter'
 require 'sirportly/data_objects/knowledge_base'
-require 'sirportly/data_objects/page'
+require 'sirportly/data_objects/knowledge_base_page'
 require 'sirportly/data_objects/priority'
 require 'sirportly/data_objects/sla'
 require 'sirportly/data_objects/status'
@@ -26,7 +26,7 @@ require 'sirportly/data_objects/ticket_update'
 require 'sirportly/data_objects/user'
 
 module Sirportly
-  VERSION = '1.1.7'
+  VERSION = '1.2.0'
   
   class << self
     

data/lib/sirportly/data_objects/knowledge_base.rb

@@ -1,31 +1,29 @@
 module Sirportly
-  class KnowledgeBase < DataObject    
+  class KnowledgeBase < DataObject
+    
     self.collection_path = 'knowledge/list'
-    self.member = {:path => 'knowledge/tree', :param => :kb}
-    self.maps = {'children' => 'Page'}
     
-    ## Workaround for a Sirportly bug
-    ## Currently SP wraps the result in an array
     def self.find(client, query)
-      raise Sirportly::Error, "This object does not support finding objects" unless member.is_a?(Hash)
-      result = client.request(member[:path], {member[:param] => query})
-      ## Notice .first here
-      self.new(client, result.first)
+      result = client.request("knowledge/list").select { |i| i['id'] == query.to_i}.first
+      self.new(client, result)
+    end
+    
+    def page(path)
+      KnowledgeBasePage.new(client, client.request('knowledge/page', :kb => attributes['id'], :path => path))
     end
     
-    def pages
-      self.children
+    def tree
+      client.request('knowledge/tree', :kb => attributes['id']).map { |i| KnowledgeBasePage.new(client, i) }
     end
     
-    ## Creates a new page associated with this knowledge base
-    ## Returns a Page object
     def create_page(params = {})
       params.merge!({:kb => self.attributes['id']})
       if req = @client.request('knowledge/add_page', params)
-        Page.new @client, req
+        KnowledgeBasePage.new(@client, req)
       else
         false
       end
     end
+    
   end
 end

data/lib/sirportly/data_objects/knowledge_base_page.rb

@@ -0,0 +1,10 @@
+module Sirportly
+  class KnowledgeBasePage < DataObject
+    self.maps = {'children' => 'KnowledgeBasePage'}
+    
+    def has_children?
+      !self.children.empty?
+    end
+    
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sirportly
 version: !ruby/object:Gem::Version
-  version: 1.1.7
+  version: 1.2.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-05-14 00:00:00.000000000Z
+date: 2012-06-15 00:00:00.000000000Z
 dependencies: []
 description: A Ruby library for interacting with the Sirportly API.
 email: adam@atechmedia.com
@@ -27,7 +27,7 @@ files:
 - lib/sirportly/data_objects/escalation_path.rb
 - lib/sirportly/data_objects/filter.rb
 - lib/sirportly/data_objects/knowledge_base.rb
-- lib/sirportly/data_objects/page.rb
+- lib/sirportly/data_objects/knowledge_base_page.rb
 - lib/sirportly/data_objects/priority.rb
 - lib/sirportly/data_objects/sla.rb
 - lib/sirportly/data_objects/status.rb

data/lib/sirportly/data_objects/page.rb

@@ -1,9 +0,0 @@
-module Sirportly
-  class Page < DataObject
-    self.maps = {'children' => 'Page'}
-    
-    def has_children?
-      !self.children.empty?
-    end
-  end
-end