github-api-client 0.1.2.6 → 0.1.3.0

data/lib/github_api/repo.rb

@@ -4,21 +4,49 @@ module GitHub
     belongs_to :parent, :class_name => 'GitHub::Repo'
     has_and_belongs_to_many :watchers, :class_name => 'GitHub::User', :join_table => 'repo_watchings', :foreign_key => 'repo_id', :association_foreign_key => 'watcher_id'
     
+    def get
+      self.update_attributes GitHub::Base.parse_attributes(:repo_get, 
+        YAML::load(
+          GitHub::Browser.get("/repos/show/#{self.permalink}"))['repository'])
+      self
+    end
+    
     def self.get(information)
       #FIXME: permalink column must be present, comparing url's is surely not the most efficient way for the db
-      GitHub::Repo.find_or_create_by_url(
-        GitHub::Base.parse_attributes(:repo_get,
-          YAML::load(GitHub::Browser.get("/repos/show/#{information}"))['repository']))
+      conditions = {:name => information.split('/').last, :owner_id => GitHub::User.find_or_create_by_login(information.split('/').first).id}
+      if r = GitHub::Repo.where(conditions).first
+        r.get
+      else
+        r = GitHub::Repo.new(conditions).get
+      end
+    end
+    
+    def fetch(*things)
+      things.each do |thing|
+        case thing
+          when :self then get
+          when :watchers then get_watchers
+        end
+      end
+      self
     end
     
+    private
     def get_watchers
-      YAML::load(GitHub::Browser.get("/repos/show/#{self.permalink}/watchers"))['watchers'].each do |watcher|
-        attr = {:login => watcher}
-        self.watchers.find_or_create(GitHub::User.find_or_create_by_login(attr))
+      watchers = YAML::load(GitHub::Browser.get("/repos/show/#{self.permalink}/watchers"))['watchers']
+      i = 1
+      self.transaction do
+        watchers.each do |watcher|
+          puts "#{watchers.length.to_s.color(:green).bright} / #{i.to_s.color(:blue).bright} - Fetching watchers"
+          i = i + 1
+          attr = {:login => watcher}
+          self.watchers.find_or_create(GitHub::User.find_or_create_by_login(attr))
+        end
       end
       self
     end
     
+    public
     def owner=(user)
       self.owner_id = GitHub::User.find_or_create_by_login(user).id if user.class == String
       if user.class == GitHub::User
@@ -38,6 +66,11 @@ module GitHub
       "#{owner.login}/#{name}"
     end
     
+    # For future, when sql will be find_or_create_by_permalink
+    def permalink=(anything)
+      @permalink = permalink
+    end
+    
     def fork?
       b_fork
     end

data/lib/github_api/user.rb

@@ -74,11 +74,13 @@ module GitHub
     def get_followers
       users = YAML::load(GitHub::Browser.get "/user/show/#{login}/followers")['users']
       i = 1
-      users.each do |user|
-        puts "#{users.length.to_s.color(:green).bright} / #{i.to_s.color(:blue).bright} - Fetching followers"
-        i = i + 1
-        u = GitHub::User.find_or_create_by_login(user)
-        self.followers.find_or_create u
+      self.transaction do
+        users.each do |user|
+          puts "#{users.length.to_s.color(:green).bright} / #{i.to_s.color(:blue).bright} - Fetching followers"
+          i = i + 1
+          u = GitHub::User.find_or_create_by_login(user)
+          self.followers.find_or_create u
+        end
       end
       self
     end
@@ -86,11 +88,13 @@ module GitHub
     def get_followings
       users = YAML::load(GitHub::Browser.get "/user/show/#{login}/following")['users']
       i = 1
-      users.each do |user|
-        puts "#{users.length.to_s.color(:green).bright} / #{i.to_s.color(:blue).bright} - Fetching followings"
-        i = i + 1
-        u = GitHub::User.find_or_create_by_login(user)
-        self.followings.find_or_create u
+      self.transaction do
+        users.each do |user|
+          puts "#{users.length.to_s.color(:green).bright} / #{i.to_s.color(:blue).bright} - Fetching followings"
+          i = i + 1
+          u = GitHub::User.find_or_create_by_login(user)
+          self.followings.find_or_create u
+        end
       end
       self
     end

metadata

@@ -5,9 +5,9 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 2
-  - 6
-  version: 0.1.2.6
+  - 3
+  - 0
+  version: 0.1.3.0
 platform: ruby
 authors: 
 - "Jakub Oko\xC5\x84ski"