From 42540e7645eaa4abd2bbae767149e07ddb2ab482 Mon Sep 17 00:00:00 2001 From: Luke Shumaker Date: Mon, 7 Apr 2014 11:33:12 -0400 Subject: fix indent --- app/controllers/matches_controller.rb | 11 ++++----- app/models/tournament.rb | 42 +++++++++++++++++------------------ 2 files changed, 27 insertions(+), 26 deletions(-) diff --git a/app/controllers/matches_controller.rb b/app/controllers/matches_controller.rb index 69ea23b..0704e5d 100644 --- a/app/controllers/matches_controller.rb +++ b/app/controllers/matches_controller.rb @@ -22,8 +22,8 @@ class MatchesController < ApplicationController tournament_matches_path(@tournament) end - def get_riot_info - if signed_in? + def get_riot_info + if signed_in? pull = "Kaceytron" #current user information @@ -107,7 +107,7 @@ class MatchesController < ApplicationController purple.merge!("#{place}" => info["games"][0]["stats"]) end sleep(1) - end + end if 100 == recent["games"][0]["stats"]["team"] blue.merge!("#{players[9]}" => recent["games"][0]["stats"]) @@ -118,8 +118,9 @@ class MatchesController < ApplicationController @purp = purple @blue = blue - end #end if - end #end def + end #end if + end #end def + # GET /matches/1 # GET /matches/1.json diff --git a/app/models/tournament.rb b/app/models/tournament.rb index 4cbbf95..d3ef12a 100644 --- a/app/models/tournament.rb +++ b/app/models/tournament.rb @@ -34,27 +34,27 @@ class Tournament < ActiveRecord::Base :range => 4 } - def initialize(tournament) - @tournament = tournament - end + def initialize(tournament) + @tournament = tournament + end - def [](preference) - p = @tournament.preferences_raw.find_by_name(preference) - if p.nil? - return nil - else - return p.value - end - end + def [](preference) + p = @tournament.preferences_raw.find_by_name(preference) + if p.nil? + return nil + else + return p.value + end + end - def []=(preference, val) - p = @tournament.preferences_raw.find_by_name(preference) - if p.nil? + def []=(preference, val) + p = @tournament.preferences_raw.find_by_name(preference) + if p.nil? TournamentPreference.create( tournament_id: @tournament.id, vartype: get_type(val), name: preference, value: val ) - else - p.value = val - end - end + else + p.value = val + end + end def get_type(val) case val @@ -71,9 +71,9 @@ class Tournament < ActiveRecord::Base end end - def keys - @tournament.preferences_raw.all.collect { |x| x.name } - end + def keys + @tournament.preferences_raw.all.collect { |x| x.name } + end def method_missing(name, *args) if name.to_s.ends_with?('=') -- cgit v1.2.3-54-g00ecf From 8f06eb768d3f674c04767f020b8bef52b8f3a879 Mon Sep 17 00:00:00 2001 From: Luke Shumaker Date: Mon, 7 Apr 2014 11:50:07 -0400 Subject: MstchesController: tidy up comments --- app/controllers/matches_controller.rb | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/app/controllers/matches_controller.rb b/app/controllers/matches_controller.rb index 0704e5d..10c45b0 100644 --- a/app/controllers/matches_controller.rb +++ b/app/controllers/matches_controller.rb @@ -1,12 +1,12 @@ class MatchesController < ApplicationController before_action :set_tournament, only: [:index, :update] - # GET /matches - # GET /matches.json require 'httparty' require 'json' require 'delayed_job' + # GET /tournaments/1/matches + # GET /tournaments/1/matches.json def index @matches = @tournament.matches # depth of SVG tree @@ -121,9 +121,6 @@ class MatchesController < ApplicationController end #end if end #end def - # GET /matches/1 - # GET /matches/1.json - def is_match_over response = HTTParty.get("https://prod.api.pvp.net/api/lol/na/v1.3/summoner/by-name/#{@first}?api_key=ad539f86-22fd-474d-9279-79a7a296ac38") riot_id = response["#{@first}"]['id'] @@ -144,6 +141,8 @@ class MatchesController < ApplicationController end handle_asynchronously :is_match_over + # GET /tournaments/1/matches/1 + # GET /tournaments/1/matches/1.json def show if Tournament.find_by_id(@match.tournament_id).game_id == 1 file_blue = "blue.yaml" @@ -153,6 +152,8 @@ class MatchesController < ApplicationController end end + # PATCH/PUT /tournaments/1/matches/1 + # PATCH/PUT /tournaments/1/matches/1.json def update case params[:update_action] when "start" -- cgit v1.2.3-54-g00ecf From d6b32cbd1234634740fda22f43c1dd1b31b20d35 Mon Sep 17 00:00:00 2001 From: Luke Shumaker Date: Mon, 7 Apr 2014 11:52:21 -0400 Subject: MatchesController: skip peer review if there aren't enough players/team --- app/controllers/matches_controller.rb | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/app/controllers/matches_controller.rb b/app/controllers/matches_controller.rb index 10c45b0..b6fde0a 100644 --- a/app/controllers/matches_controller.rb +++ b/app/controllers/matches_controller.rb @@ -168,8 +168,6 @@ class MatchesController < ApplicationController end end when "finish" - @match.status = 2 - # Individual scores scores = params["scores"] scores.each do |user_name, score| @@ -193,6 +191,15 @@ class MatchesController < ApplicationController @match.winner.matches.push(@tournament.matches_ordered[cur_match_num/2]) end + # Skip peer evaluation if there aren't enough players per team + peer = false + @match.teams.each do |team| + if team.users.count > 2 + peer = true + end + end + @match.status = peer ? 2 : 3 + respond_to do |format| if @match.save format.html { redirect_to tournament_match_path(@tournament, @match), notice: 'Peer evaluation started.' } -- cgit v1.2.3-54-g00ecf From 87d69d44d9d7f56071f34597fd46bd26b59ab9fd Mon Sep 17 00:00:00 2001 From: Luke Shumaker Date: Mon, 7 Apr 2014 12:43:23 -0400 Subject: stuff --- app/controllers/tournaments_controller.rb | 7 ++- app/views/tournaments/index.html.erb | 100 ++++++++++++++---------------- 2 files changed, 53 insertions(+), 54 deletions(-) diff --git a/app/controllers/tournaments_controller.rb b/app/controllers/tournaments_controller.rb index 2fc82ed..f572248 100644 --- a/app/controllers/tournaments_controller.rb +++ b/app/controllers/tournaments_controller.rb @@ -14,8 +14,11 @@ class TournamentsController < ApplicationController case @tournament.status when 0 render action: 'show' - when 1..2 - redirect_to "/tournaments/" + @tournament.id.to_s + "/matches" #tournament_matches_page(@tournament) + when 1 + #redirect_to tournament_matches_page(@tournament) + redirect_to "/tournaments/" + @tournament.id.to_s + "/matches" + when 2 + redirect_to tournaments_page end } format.json { diff --git a/app/views/tournaments/index.html.erb b/app/views/tournaments/index.html.erb index 72eacba..7957042 100644 --- a/app/views/tournaments/index.html.erb +++ b/app/views/tournaments/index.html.erb @@ -1,56 +1,52 @@

Listing Tournaments

-<% if @tournaments.length > 0 %> -
- - <%# Each tournament has a div for its listing %> - <% @tournaments.each do |t| %> -
-
<%= image_tag 'http://www.gravatar.com/avatar/' + Digest::MD5.hexdigest(t.hosts.first.email) + '?s=100&d=mm' %>
- +
+ <% if @tournaments.length > 0 %> + <%# Each tournament has a div for its listing %> + <% @tournaments.each do |t| %> +
+
<%= image_tag 'http://www.gravatar.com/avatar/' + Digest::MD5.hexdigest(t.hosts.first.email) + '?s=100&d=mm' %>
+
+ <%# "header" %> + <%= link_to(t) do %>

<%= t.name %>

<% end %> +
+
+ Hosted by: <%= t.hosts.first.name %> +
+
+

Players per team: <%= t.min_players_per_team %>

+

Players signed up: <%= t.players.count %>

+
+
+

<%= (t.randomized_teams)? "Teams are Random" : "Teams are Chosen" %>

+

Players signed up: <%= t.players.count %>

+
+
+
+
+ <% if signed_in? %> + <% if !t.players.include?(current_user) %> + <%= form_tag(tournament_path(t), method: "put") do %> + + <%= submit_tag("Join") + %> + <% end %> + <% else %> +

You've signed up for this tournament!

+ <% end %> + <% end %> +
+
+ <% end %> + <% else %> +

No tournaments going on right now... + <% if current_user.can?(:create_tournament) %> + Why not start your own? + <% end %> +

+ <% end %> +
-
- <%# "header" %> - <%= link_to(t) do %> -

<%= t.name %>

- <% end %> - -
-
- Hosted by: <%= t.hosts.first.name %> -
-
-

Players per team: <%= t.min_players_per_team %>

-

Players signed up: <%= t.players.count %>

-
-
-

<%= (t.randomized_teams)? "Teams are Random" : "Teams are Chosen" %>

-

Players signed up: <%= t.players.count %>

-
-
- -
- - -
- <%# If this guy is logged in AND not in the tournament %> - <% if signed_in? && !t.players.include?(current_user) %> - <%= form_tag(tournament_path(t), method: "put") do %> - - <%= submit_tag("Join") %> - <% end %> - <% else %> -

You've signed up for this tournament!

- <% end %> -
- -
- <% end %> - -
-<% else %> -

No tournaments going on right now... Why not start your own?

+<% if current_user.can?(:create_tournament) %> +
<%= link_to 'New Tournament', new_tournament_path, :class => "btn btn-warning btn-lg" %>
<% end %> -
- -<%= link_to 'New Tournament', new_tournament_path, :class => "btn btn-warning btn-lg" %> -- cgit v1.2.3-54-g00ecf