github-api-client 0.1.2.4 → 0.1.2.6

data/db/migrate/002_create_user_followings.rb

@@ -0,0 +1,12 @@
+class CreateUserFollowings < ActiveRecord::Migration
+  def self.up
+    create_table :user_followings, :id => false, :force => true do |t|
+      t.references :follower
+      t.references :following
+    end
+  end
+  
+  def self.down
+    drop_table :user_followings
+  end
+end

data/db/migrate/004_create_repo_watchings.rb

@@ -0,0 +1,12 @@
+class CreateRepoWatchings < ActiveRecord::Migration
+  def self.up
+    create_table :repo_watchings, :id => false, :force => true do |t|
+      t.references :watcher
+      t.references :repo
+    end
+  end
+  
+  def self.down
+    drop_table :repo_watchings
+  end
+end

data/lib/github_api/base.rb

@@ -38,8 +38,8 @@ module GitHub
     def self.parse_attributes(resource, attributes)
       hash = case resource
         when :user_get then {:public_repo_count => :nil, :public_gist_count => :nil, :created => :nil, :permission => :nil, :followers_count => :nil, :following_count => :nil}
-        when :user_search then {:name => :login, :username => :login, :fullname => :name, :followers => :nil, :repos => :public_repo_count, :created => :nil, :permission => :nil}
-        when :repo_get then {:fork => :b_fork}
+        when :user_search then {:name => :login, :username => :login, :fullname => :name, :followers => :nil, :repos => :nil, :created => :nil, :permission => :nil}
+        when :repo_get then {:fork => :b_fork, :watchers => :nil}
       end
       hash.each do |k, v|
         unless v == :nil

data/lib/github_api/repo.rb

@@ -2,6 +2,7 @@ module GitHub
   class Repo < ActiveRecord::Base
     belongs_to :owner, :class_name => 'GitHub::User'
     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 self.get(information)
       #FIXME: permalink column must be present, comparing url's is surely not the most efficient way for the db
@@ -10,6 +11,13 @@ module GitHub
           YAML::load(GitHub::Browser.get("/repos/show/#{information}"))['repository']))
     end
     
+    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))
+      end
+      self
+    end
     
     def owner=(user)
       self.owner_id = GitHub::User.find_or_create_by_login(user).id if user.class == String
@@ -30,7 +38,7 @@ module GitHub
       "#{owner.login}/#{name}"
     end
     
-    define_method(:fork?) do
+    def fork?
       b_fork
     end
   end

data/lib/github_api/user.rb

@@ -1,8 +1,8 @@
 module GitHub
   # Basic model, stores retrieved user and his associations
   class User < ActiveRecord::Base
-    has_and_belongs_to_many :followers, :foreign_key => 'follower_id', :association_foreign_key => 'following_id', :join_table => 'followings', :class_name => 'User'
-    has_and_belongs_to_many :followings, :foreign_key => 'following_id', :association_foreign_key => 'follower_id', :join_table => 'followings', :class_name => 'User'
+    has_and_belongs_to_many :followers, :foreign_key => 'follower_id', :association_foreign_key => 'following_id', :join_table => 'user_followings', :class_name => 'User'
+    has_and_belongs_to_many :followings, :foreign_key => 'following_id', :association_foreign_key => 'follower_id', :join_table => 'user_followings', :class_name => 'User'
     has_many :repos, :class_name => 'GitHub::Repo', :foreign_key => 'owner_id'
     
     # Fetches info about current_user from GitHub
@@ -35,7 +35,7 @@ module GitHub
     def self.search(login)
       users = []
       YAML::load(GitHub::Browser.get("/user/search/#{login}"))['users'].each do |user|
-        users << GitHub::User.find_or_create_by_login(GitHub::Base.parse_attributes(:user, user))
+        users << GitHub::User.find_or_create_by_login(GitHub::Base.parse_attributes(:user_search, user))
       end
       users
     end
@@ -79,8 +79,6 @@ module GitHub
         i = i + 1
         u = GitHub::User.find_or_create_by_login(user)
         self.followers.find_or_create u
-        # Realized it was stupid as it's a bi-directional relation :D
-        #u.followings.find_or_create self
       end
       self
     end

metadata

@@ -6,8 +6,8 @@ version: !ruby/object:Gem::Version
   - 0
   - 1
   - 2
-  - 4
-  version: 0.1.2.4
+  - 6
+  version: 0.1.2.6
 platform: ruby
 authors: 
 - "Jakub Oko\xC5\x84ski"
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-12-02 00:00:00 +01:00
+date: 2010-12-03 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -77,8 +77,9 @@ files:
 - lib/github_api/browser.rb
 - lib/github_api/config.rb
 - lib/github_api/user.rb
+- db/migrate/004_create_repo_watchings.rb
 - db/migrate/003_create_repos.rb
-- db/migrate/002_create_followings.rb
+- db/migrate/002_create_user_followings.rb
 - db/migrate/001_create_users.rb
 - lib/core_ext/habtm.rb
 has_rdoc: false

data/db/migrate/002_create_followings.rb

@@ -1,12 +0,0 @@
-class CreateFollowings < ActiveRecord::Migration
-  def self.up
-    create_table :followings, :id => false, :force => true do |t|
-      t.references :follower
-      t.references :following
-    end
-  end
-  
-  def self.down
-    drop_table :followings
-  end
-end