feat: allow booking appointments against department / service_unit ba…

…sed on appointment type

healthcare/healthcare/doctype/patient_appointment/patient_appointment.js

@@ -196,6 +196,88 @@ frappe.ui.form.on('Patient Appointment', {
  });
  },
 
+ appointment_for: function(frm) {
+ if (frm.doc.appointment_for == 'Practitioner') {
+ if (!frm.doc.practitioner) {
+ frm.set_value('department', '');
+ }
+ frm.set_value('service_unit', '');
+ frm.trigger('set_check_availability_action');
+ } else if (frm.doc.appointment_for == 'Service Unit') {
+ frm.set_value({
+ 'practitioner': '',
+ 'practitioner_name': '',
+ 'department': '',
+ });
+ frm.trigger('set_book_action');
+ } else if (frm.doc.appointment_for == 'Department') {
+ frm.set_value({
+ 'practitioner': '',
+ 'practitioner_name': '',
+ 'service_unit': '',
+ });
+ frm.trigger('set_book_action');
+ } else {
+ if (frm.doc.appointment_for == 'Department') {
+ frm.set_value('service_unit', '');
+ }
+ frm.set_value({
+ 'practitioner': '',
+ 'practitioner_name': '',
+ 'department': '',
+ 'service_unit': '',
+ });
+ frm.page.clear_primary_action();
+ }
+ },
+
+ set_book_action: function(frm) {
+ frm.page.set_primary_action(__('Book'), function() {
+ frm.enable_save();
+ frm.save();
+ });
+ },
+
+ set_check_availability_action: function(frm) {
+ frm.page.set_primary_action(__('Check Availability'), function() {
+ if (!frm.doc.patient) {
+ frappe.msgprint({
+ title: __('Not Allowed'),
+ message: __('Please select Patient first'),
+ indicator: 'red'
+ });
+ } else {
+ frappe.call({
+ method: 'healthcare.healthcare.doctype.patient_appointment.patient_appointment.check_payment_fields_reqd',
+ args: { 'patient': frm.doc.patient },
+ callback: function(data) {
+ if (data.message == true) {
+ if (frm.doc.mode_of_payment && frm.doc.paid_amount) {
+ check_and_set_availability(frm);
+ }
+ if (!frm.doc.mode_of_payment) {
+ frappe.msgprint({
+ title: __('Not Allowed'),
+ message: __('Please select a Mode of Payment first'),
+ indicator: 'red'
+ });
+ }
+ if (!frm.doc.paid_amount) {
+ frappe.msgprint({
+ title: __('Not Allowed'),
+ message: __('Please set the Paid Amount first'),
+ indicator: 'red'
+ });
+ }
+ } else {
+ check_and_set_availability(frm);
+ }
+ }
+ });
+ }
+ });
+ },
+
  patient: function(frm) {
  if (frm.doc.patient) {
  frm.trigger('toggle_payment_fields');

healthcare/healthcare/doctype/patient_appointment/patient_appointment.py

@@ -37,7 +37,7 @@ class OverlapError(frappe.ValidationError):
 class PatientAppointment(Document):
  def validate(self):
  self.validate_overlaps()
- self.validate_based_on_appointments_for()
+ self.validate_appointments_without_practitioner()
  self.validate_service_unit()
  self.set_appointment_datetime()
  self.validate_customer_created()
@@ -166,8 +166,8 @@ def validate_overlaps(self):
  OverlapError,
  )
 
- def validate_based_on_appointments_for(self):
- if self.appointment_for:
+ def validate_appointments_without_practitioner(self):
+ if not self.practitioner and self.appointment_type:
  # fieldname: practitioner / department / service_unit
  appointment_for_field = frappe.scrub(self.appointment_for)
 
@@ -178,12 +178,7 @@ def validate_based_on_appointments_for(self):
  frappe.MandatoryError,
  )
 
- if self.appointment_for == "Practitioner":
- # appointments for practitioner are validated separately,
- # based on practitioner schedule
- return
-
- # validate if patient already has an appointment for the day
+ # validate if patient already has an appointment
  booked_appointment = frappe.db.exists(
  "Patient Appointment",
  {
@@ -197,7 +192,7 @@ def validate_based_on_appointments_for(self):
 
  if booked_appointment:
  frappe.throw(
- _("Patient already has an appointment {} booked for {} on {}").format(
+ _("Patient already has an appointment {} booked at {} on {}").format(
  get_link_to_form("Patient Appointment", booked_appointment),
  frappe.bold(self.get(appointment_for_field)),
  frappe.bold(format_date(self.appointment_date)),