I am encountering a problem when trying to .build a new appointment. But first a look at my Model(s)<\/p>\n
class Patient < ActiveRecord::Base\n attr_accessible :address1, :address2, :city, :comment, :email, :first_name, :init_date, :init_time, :last_name, :mobile, :notes, :phone, :state, :zip\n before_validation :upcase_patient\n before_save { self.email.downcase! }\n has_many :appointments, dependent: :destroy\n has_many :procedures, through: :appointments\n<\/code><\/pre>\nclass Procedure < ActiveRecord::Base\n attr_accessible :comment, :occurence, :procedure, :procedure_code, :procedure_price, :procedure_time, :visits\n before_validation :uppercase_procedure\n has_many :appointments\n has_many :patients, through: :appointments\n<\/code><\/pre>\nclass Appointment < ActiveRecord::Base\n attr_accessible :appointment_date, :appointment_notes, :appointment_time, :procedure_id\n belongs_to :patient\n belongs_to :procedure\n<\/code><\/pre>\nclass AppointmentsController < ApplicationController\ninclude PatientsHelper\nbefore_filter :signed_in_user\n\ndef create\n @patient = current_patient\n @appointment = @patient.appointments.build(params[:appointment])\n if @appointment.save\n flash[:success] = \"Appointment scheduled!\"\n redirect_to patient_path(@patient)\n else\n render patient_path(@patient)\n end\nend\n<\/code><\/pre>\nmodule PatientsHelper\n\n def current_patient=(patient)\n @current_patient = patient\n end\n\n def current_patient\n @current_patient\n end\n\n def current_patient?(patient)\n patient == current_patient\n end\nend\n<\/code><\/pre>\nThese items including “PatientsHelper” which helps define current_patient make up the associations. I have created the form successfullly within the patients_controller<\/p>\n
class PatientsController < ApplicationController\nbefore_filter :signed_in_user\nbefore_filter :admin_user, only: [:destroy]\ndef show\n @patient = Patient.find(params[:id])\n @procedures = Procedure.all\n @appointments = @patient.appointments.paginate(page: params[:page])\n @appointment = @patient.appointments.build\nend\n<\/code><\/pre>\nI would like to use the patients resources to create new appointments. Here is where I am fouled up. I continue to receive via rspec the following error:<\/p>\n
NoMethodError:\n undefined method `appointments' for nil:NilClass\n<\/code><\/pre>\nWhere am I going wrong? Any help would be greatly appreciated. If clarification is required, I would like to:<\/p>\n
a_ associate<\/p>\n
Patient model -> has_many [:appointments, :procedures (through :appointments)]<\/p>\n
Procedure model -> has_many [:appointments, :patients (through :appointments)]<\/p>\n
Appointment model -> belongs_to [:patients, :procedures]<\/p>\n
b_ create new :appointments through the patients controller rather than create a new controller specifically for appointments model<\/p>\n
my mistake! here is the spec test.<\/p>\n
require 'spec_helper'\n\ndescribe \"Appointment Pages\" do\n subject { page }\n\n let(:user) { FactoryGirl.create(:user) }\n let(:patient) { FactoryGirl.create(:patient) }\n let(:procedure) { FactoryGirl.create(:procedure) } \n before { sign_in user }\n\n describe \"appointment creation\" do\n before { visit patient_path(patient) }\n\n describe \"with invalid information\" do\n\n it \"should not create an appointment\" do\n expect { click_button \"Schedule procedure\" }.not_to change(Appointment,\n :count)\n end\n\n describe \"error messages\" do\n before { click_button \"Schedule procedure\" }\n it { should have_content('error') }\n end\nend\n\ndescribe \"with valid information\" do\n before do\n procedure_id = procedure.id\n patient_id = patient.id\n fill_in 'appointment_appointment_date', with: \"2013-04-04\"\n fill_in 'appointment_appointment_time', with: \"12:45:00\"\n fill_in 'appointment_appointment_notes', with: \"Test the notes\"\n end\n it \"should create a micropost\" do\n expect { click_button \"Schedule procedure\" }.to change(Appointment, \n :count).by(1)\n end\n end\n end\nend\n<\/code><\/pre>\nerrors show up on the following<\/p>\n
Failures:\n\n1) Appointment Pages appointment creation with invalid information should not create an appointment\n Failure\/Error: expect { click_button \"Schedule procedure\" }.not_to change(Appointment,\n NoMethodError:\n undefined method `appointments' for nil:NilClass\n # .\/app\/controllers\/appointments_controller.rb:7:in `create'\n # (eval):2:in `click_button'\n # .\/spec\/requests\/appointment_pages_spec.rb:17:in `block (5 levels) in '\n # .\/spec\/requests\/appointment_pages_spec.rb:17:in `block (4 levels) in '\n\n2) Appointment Pages appointment creation with invalid information error messages \n Failure\/Error: before { click_button \"Schedule procedure\" }\n NoMethodError:\n undefined method `appointments' for nil:NilClass\n # .\/app\/controllers\/appointments_controller.rb:7:in `create'\n # (eval):2:in `click_button'\n # .\/spec\/requests\/appointment_pages_spec.rb:22:in `block (5 levels) in '\n\n3) Appointment Pages appointment creation with valid information should create a micropost\n Failure\/Error: expect { click_button \"Schedule procedure\" }.to change(Appointment,\n NoMethodError:\n undefined method `appointments' for nil:NilClass\n # .\/app\/controllers\/appointments_controller.rb:7:in `create'\n # (eval):2:in `click_button'\n # .\/spec\/requests\/appointment_pages_spec.rb:36:in `block (5 levels) in '\n # .\/spec\/requests\/appointment_pages_spec.rb:36:in `block (4 levels) in '\n\nFinished in 1.08 seconds\n23 examples, 3 failures\n\nFailed examples:\n\nrspec .\/spec\/requests\/appointment_pages_spec.rb:16 # Appointment Pages appointment creation with invalid information should not create an appointment\nrspec .\/spec\/requests\/appointment_pages_spec.rb:23 # Appointment Pages appointment creation with invalid information error messages \nrspec .\/spec\/requests\/appointment_pages_spec.rb:35 # Appointment Pages appointment creation with valid information should create a micropost\n<\/code><\/pre>\n\n- \n
Not exactly what I was looking for but made this work-around.<\/p>\n
Using the following was able to get this to save accordingly. Still cannot find out why I am unable to pull the Patient from the URL — example would be app.dev\/patients\/2 — where patient information is pulling id 2<\/em>… but now I am able to save the Appointment using what I suspect is a rather crude method.<\/p>\n\n- In the form I added a hidden field which pulls the appropriate patient_id<\/li>\n
- In Appointments controller, added to attr_accessible :patient_id<\/li>\n
- \n
In appointments_controller<\/p>\n
def create\n@appointment = Appointment.new(params[:appointment])\n@current_patient = @appointment.patient_id\nif @appointment.save\n flash[:success] = \"Appointment scheduled!\"\n redirect_to patient_path(@current_patient)\nelse\n render 'patients\/show'\nend\nend\n<\/code><\/pre>\n<\/li>\n<\/ol>\nrender 'patients\/show'<\/code> remains broken but will leave that for another thread. Thank you all for your help and guidance.<\/p>\n<\/li>\n<\/ol>\nOriginally posted 2013-11-09 22:49:22. <\/small><\/p>","protected":false},"excerpt":{"rendered":"
I am encountering a problem when trying to .build a new appointment. But first a look at my Model(s) class Patient < ActiveRecord::Base attr_accessible :address1, :address2, :city, :comment, :email, :first_name, :init_date, :init_time, :last_name, :mobile, :notes, :phone, :state, :zip before_validation :upcase_patient before_save { self.email.downcase! } has_many :appointments, dependent: :destroy has_many :procedures, through: :appointments class Procedure < […]<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"closed","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[1],"tags":[],"class_list":["post-861","post","type-post","status-publish","format-standard","hentry","category-uncategorized"],"_links":{"self":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/posts\/861","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/comments?post=861"}],"version-history":[{"count":0,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/posts\/861\/revisions"}],"wp:attachment":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/media?parent=861"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/categories?post=861"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/tags?post=861"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}