app/views/votes/_voting_box.html.erb
db/migrate/20080701085004_add_hiring_stage_to_votes.rb
@@ -1,6 +1,34 @@
class VotesController < ApplicationController
- # GET /votes
- # GET /votes.xml
+
+
+ def vote
+ hiring_stage = HiringStage.find_by_hiring_stage_id( params[:hiring_stage_id] )
+ vote_value = params[:vote]
+
+ # Is the current_user allowed to vote on this hiring_stage?
+ if current_user.positions.include?( hiring_stage.position )
+ logger.debug ">> #{vote_value}"
+
+ # Does this user already have a vote for this hiring stage for this position?
+ vote = Vote.find_vote_by_hiring_stage( hiring_stage )
+ if vote.nil?
+ vote = Vote.new.construct_from_basic_elements( current_user, hiring_stage, vote_value )
+ else
+ vote.update_attribute( :vote_value, vote_value )
+ end
+ end
+
+ respond_to do |wants|
+ wants.js do
+ render :update do |page|
+ logger.debug ">> Boo"
+ #page.visual_effect :puff, "comment_#{params[:id]}".to_sym
+ end
+ end
+ end
+ end
+
+=begin
def index
@votes = Vote.find(:all)
@@ -82,4 +110,5 @@ class VotesController < ApplicationController
format.xml { head :ok }
end
end
+=end
end
app/controllers/votes_controller.rb
@@ -31,9 +31,7 @@ module PositionsHelper
def render_vetting_stage_cell( hiring_stage, v_stage )
hiring_stage.vetting_stage.is_me_or_is_my_parent?( v_stage ) ? hilite_class = 'hilited' : hilite_class = ''
- cell = "<td class='t c " + hilite_class + "'>"
- cell += render_voting_box_cell( hiring_stage, v_stage )
- cell += "</td>"
+ cell = "<td class='t c " + hilite_class + "'>" + render_voting_box_cell( hiring_stage, v_stage ) + "</td>"
v_stage.children.each do |child|
cell += render_vetting_stage_cell( hiring_stage, child )
@@ -44,22 +42,39 @@ module PositionsHelper
# TODO: Finish this function
+ # If a user needs to vote on this applicant, display the two voting options.
+ # If a user has voted on this applicant, display the box in the colour of their vote with a delete circle in the top-left
+ # If a user is not allowed to vote on this position, show the user icon
def render_voting_box_cell( hiring_stage, v_stage )
- return hiring_stage.vetting_stage == v_stage ? image_tag( 'applicant.png' ) : ""
+ cell = ""
+
+ if hiring_stage.vetting_stage == v_stage
+ if current_user.can_vote_on_position?( hiring_stage.position )
+ vote = Vote.find_vote_by_hiring_stage( hiring_stage )
+
+ cell += "<div class='vote_value_#{vote.vote_value.to_s}'>" if vote
+ cell += render( :partial => 'votes/voting_box', :locals => { :hiring_stage => hiring_stage } )
+ cell += "</div>" if vote
+ else
+ cell = image_tag( 'applicant.png' )
+ end
+ end
+
+ return cell
end
def render_vetting_stage_controls( hiring_stage, applicant )
- cell = '<td>'
+ cell = '<td class="t c">'
if hiring_stage.vetting_stage.parent
- cell += link_to( ( image_tag( 'applicant_og.png', :alt => 'retreat', :title => 'Retreat this applicant to the prev stage' ) ), :controller => 'positions', :action => 'retreat', :id => @position, :applicant_id => applicant.id )
+ cell += link_to( ( image_tag( 'applicant_og.png', :alt => 'retreat', :title => 'Retreat this applicant to the prev stage', :class => 'controls_button' ) ), :controller => 'positions', :action => 'retreat', :id => @position, :applicant_id => applicant.id )
else
- cell += image_tag( 'applicant_blank.png' )
+ cell += image_tag( 'applicant_blank.png', :class => 'controls_button' )
end
cell += " "
- cell += link_to( image_tag( 'applicant_go.png', :alt => 'advance', :title => 'Advance this applicant to the next stage' ), :controller => 'positions', :action => 'advance', :id => @position, :applicant_id => applicant.id )
+ cell += link_to( image_tag( 'applicant_go.png', :alt => 'advance', :title => 'Advance this applicant to the next stage', :class => 'controls_button' ), :controller => 'positions', :action => 'advance', :id => @position, :applicant_id => applicant.id )
cell += '</td>'
return cell
app/helpers/positions_helper.rb
@@ -5,6 +5,7 @@ class HiringStage < ActiveRecord::Base
belongs_to :position
belongs_to :vetting_stage
+ has_many :votes
def construct_from_basic_elements( applicant, position, vetting_stage )
self.applicant = applicant
app/models/hiring_stage.rb
@@ -21,10 +21,10 @@ class User < ActiveRecord::Base
validates_inclusion_of :is_admin, :in => [false, true]
has_many :comments, :as => :commentable, :order => 'comments.created_at DESC', :dependent => :destroy
- has_many :votes, :dependent => :nullify
has_many :open_positions, :class_name => 'Position', :conditions => 'is_closed = 0'
has_many :closed_positions, :class_name => 'Position', :conditions => 'is_closed = 1'
-
+ has_many :votes, :dependent => :nullify
+
has_and_belongs_to_many :positions, :order => 'title ASC'
@@ -61,6 +61,11 @@ class User < ActiveRecord::Base
return false
end
+
+ def can_vote_on_position?( position )
+ return positions.include?( position )
+ end
+
protected
app/models/user.rb
@@ -5,6 +5,28 @@ class Vote < ActiveRecord::Base
belongs_to :applicant
belongs_to :vetting_stage
belongs_to :position
+ belongs_to :hiring_stage
- validates_presence_of :vote
+
+ # ------------------------------------------------------------ Class Methods ------------------------------------------------------------ #
+
+
+ def self.find_vote_by_hiring_stage( hiring_stage )
+ return self.find( :first, :conditions => "hiring_stage_id = #{hiring_stage.id}" )
+ end
+
+
+ # ------------------------------------------------------------ Instance Methods ------------------------------------------------------------ #
+
+
+ def construct_from_basic_elements( current_user, hiring_stage, vote_value )
+ self.user = current_user
+ self.applicant = hiring_stage.applicant
+ self.vetting_stage = hiring_stage.vetting_stage
+ self.position = hiring_stage.position
+ self.hiring_stage = hiring_stage
+ self.vote_value = vote_value
+
+ self.save!
+ end
end
app/models/vote.rb
@@ -8,10 +8,12 @@
<% for applicant in @applicants %>
<tr class="separated selectable">
- <td class="t l"><%= link_to( sanitize( applicant.name ), applicant_path( applicant ), :class => 'new_link' ) %></td>
+ <td class="t l">
+ <%= link_to( sanitize( applicant.name ), applicant_path( applicant ), :class => 'new_link' ) %>
+
+ </td>
<td class="t l" width="50%">
<%= truncate( sanitize( applicant.contact_info ), 128 ) %> <br />
- <span class="display_date"><%= applicant.updated_at.friendly_date %></span> <br />
<span class="small"><%= "Positions: " + applicant.positions.map { |pos| link_to( pos.title, position_path( pos ) ) }.to_sentence if ( applicant.positions.count > 0 ) %></span>
</td>
<td class="t c">
app/views/applicants/index.html.erb
@@ -31,6 +31,8 @@ ActionController::Routing::Routes.draw do |map|
map.connect 'positions/skip/:id', :controller => 'positions', :action => 'skip'
+ map.connect 'votes/vote', :controller => 'votes', :action => 'vote'
+
# The priority is based upon order of creation: first created -> highest priority.
# Sample of regular route:
config/routes.rb
@@ -5,7 +5,7 @@ class CreateVotes < ActiveRecord::Migration
t.column :applicant_id, :integer, :null => false
t.column :position_id, :integer, :null => false
t.column :vetting_stage_id, :integer, :null => false
- t.column :vote, :boolean, :null => false, :default => 0
+ t.column :vote_value, :boolean, :null => false, :default => 0
t.timestamps
end
db/migrate/20080623221948_create_votes.rb
@@ -492,6 +492,28 @@ a.new_link img {
}
+.controls_button {
+ margin: 0px 4px 0px 4px;
+}
+
+.voting_button {
+ cursor: pointer;
+ margin-right: 8px;
+}
+
+
+.vote_value_true {
+ border: 1px solid green;
+ padding: 2px;
+}
+
+
+.vote_value_false {
+ border: 1px solid red;
+ padding: 2px;
+}
+
+
/* ------------------------------------------------------------ Rounded Rect ------------------------------------------------------------ */
.rounded_rect_btn_light, rounded_rect_btn_dark {
public/stylesheets/application.css
app/views/positions/_voting_box.html.erb
8ea73d463c5072aa00d88d277bf18a0b89fd1682
Chris
chris@ChoOyu.local
http://github.com/senorprogrammer/applican/commit/56d286bd036bf2f85fc5901995ad40be51b19113
56d286bd036bf2f85fc5901995ad40be51b19113
2008-07-01T02:53:35-07:00
2008-07-01T02:53:35-07:00
Voting is now in place (though it is aesthetically ugly a wee bit)
4b4745db2c9956777d7b716dc81d3487198f9c93
Chris
chris@ChoOyu.local