hubstats 0.7.2 → 0.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2d38206085aaf51c050cb61f3487f8083a2a8945
-  data.tar.gz: 75dabcc2b4a07fed0f0afbf35ae19b1129107041
+  metadata.gz: 5688b322dcd8b475e84a566cd74904508d6f845c
+  data.tar.gz: 50009dbc401e9a134fd99cc49ae7198c0c7a7621
 SHA512:
-  metadata.gz: e19fee330d5d16f76a0792b40f97028131862289c2b27fc84b4a6f3234a69f5b55000113cad273081b758a3b6451755efb3210a12c8e6cc0da76f08275db3178
-  data.tar.gz: fc5b81cc305b27650bb3f07ba52336bd66374659194808bd20d1b7fb32460af860e753755cf9f92f9515738cecc509a6e8174f2a376e0609923dd148eaa6d6c7
+  metadata.gz: 5817e679b851a94179e217f14a3f982e21e83a82d58a73e6f36169c237c57707d42b527fdd37eed68d680cc699ed3f5ef3110aad39173e971e6ed6cc22c1d3b1
+  data.tar.gz: 0695c7e533e556ae0ba1006dc84aff3ed898ece7a9e4cb91c25508aae50693364af38de7b1c6da694891de8489fc04bea4e109da528d45a71289faac5ab459e6

data/CHANGELOG.markdown

@@ -1,3 +1,8 @@
+#### v0.7.3
+* Fixing bug that allowed users to be shown multiple times on teams show page
+
+  > Emma Sax: Brian Bergstrom: https://github.com/sportngin/hubstats/pull/102
+
 #### v0.7.2
 * Upgrading Hubstats to Ruby-2.3.0
 

data/app/controllers/hubstats/teams_controller.rb

@@ -33,7 +33,7 @@ module Hubstats
       @team = Hubstats::Team.where(id: params[:id]).first
       @pull_requests = Hubstats::PullRequest.belonging_to_team(@team.id).merged_in_date_range(@start_date, @end_date).order("updated_at DESC").limit(20)
       @pull_count = Hubstats::PullRequest.belonging_to_team(@team.id).merged_in_date_range(@start_date, @end_date).count(:all)
-      @users = @team.users.where("login NOT IN (?)", Hubstats.config.github_config["ignore_users_list"]).order("login ASC")
+      @users = @team.users.where("login NOT IN (?)", Hubstats.config.github_config["ignore_users_list"]).order("login ASC").distinct
       @active_user_count = @users.length
       @comment_count = Hubstats::Comment.belonging_to_team(@users.pluck(:id)).created_in_date_range(@start_date, @end_date).count(:all)
       repos_pr = @pull_requests.pluck(:repo_id)

data/lib/hubstats/version.rb

@@ -1,3 +1,3 @@
 module Hubstats
-  VERSION = "0.7.2"
+  VERSION = "0.7.3"
 end

data/spec/controllers/hubstats/teams_controller_spec.rb

@@ -26,6 +26,7 @@ module Hubstats
         repo = create(:repo, :updated_at => Date.today)
         team.users << user1
         team.users << user2
+        team.users << user2
         pull1 = create(:pull_request, :user => user1, :id => 303030, :team => team, :repo_id => repo.id, :updated_at => Date.today)
         pull2 = create(:pull_request, :user => user2, :id => 404040, :team => team, :repo => pull1.repo, :updated_at => Date.today)
         allow(Hubstats).to receive_message_chain(:config, :github_config, :[]).with("ignore_users_list") { ["user"] }
@@ -33,7 +34,9 @@ module Hubstats
         expect(assigns(:team)).to eq(team)
         expect(pull1.team_id).to eq(team.id)
         expect(pull2.team_id).to eq(team.id)
-        expect(assigns(:team).users).to contain_exactly(user1, user2)
+        expect(assigns(:team).users).to contain_exactly(user1, user2, user2)
+        expect(assigns(:users)).to contain_exactly(user1, user2)
+        expect(assigns(:active_user_count)).to eq(2)
         expect(response).to have_http_status(200)
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hubstats
 version: !ruby/object:Gem::Version
-  version: 0.7.2
+  version: 0.7.3
 platform: ruby
 authors:
 - Elliot Hursh
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-14 00:00:00.000000000 Z
+date: 2016-01-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails