I believe this has resolved the dependent destruction and we can close issue #18

This commit is contained in:
Ken Johnson
2013-06-02 13:08:56 -04:00
parent 4e445375fa
commit 06dce1f8b2
3 changed files with 8 additions and 6 deletions
+3 -1
View File
@@ -34,7 +34,9 @@ class AdminController < ApplicationController
def delete_user
user = User.find_by_user_id(params[:admin_id])
if user && !(current_user.user_id == user.user_id)
user.delete
# Call destroy here so that all association records w/ user_id are destroyed as well
# Example user.retirement records would be destroyed
user.destroy
message = true
end
respond_to do |format|
+1 -1
View File
@@ -1,7 +1,7 @@
class PaidTimeOff < ActiveRecord::Base
attr_accessible :pto_earned, :pto_taken, :sick_days_earned, :sick_days_taken
belongs_to :user
has_many :schedule, :foreign_key => :user_id, :primary_key => :user_id
has_many :schedule, :foreign_key => :user_id, :primary_key => :user_id, :dependent => :destroy
def sick_days_remaining
self.sick_days_earned - self.sick_days_taken
+4 -4
View File
@@ -10,10 +10,10 @@ class User < ActiveRecord::Base
validates_format_of :email, :with => /.+@.+\..+/i
attr_accessor :skip_user_id_assign
before_save :assign_user_id, :on => :create
has_one :retirement, :foreign_key => :user_id, :primary_key => :user_id
has_one :paid_time_off, :foreign_key => :user_id, :primary_key => :user_id
has_one :work_info, :foreign_key => :user_id, :primary_key => :user_id
has_many :performance, :foreign_key => :user_id, :primary_key => :user_id
has_one :retirement, :foreign_key => :user_id, :primary_key => :user_id, :dependent => :destroy
has_one :paid_time_off, :foreign_key => :user_id, :primary_key => :user_id, :dependent => :destroy
has_one :work_info, :foreign_key => :user_id, :primary_key => :user_id, :dependent => :destroy
has_many :performance, :foreign_key => :user_id, :primary_key => :user_id, :dependent => :destroy
def full_name