django_q2 no longer needs to be installed even if not used

Author: adamspd

README.md

@@ -68,6 +68,11 @@ see their [release notes](https://github.com/adamspd/django-appointment/tree/mai
    ```bash
    pip install django-appointment
    ```
+   Optionally installing django_q2 if you need email reminders:
+
+   ```bash
+   pip install django_q2
+   ```
 
 2. Add "appointment" (& "django_q" if you want to enable email reminders) to your `INSTALLED_APPS` setting like so:
 

appointment/email_sender/email_sender.py

@@ -3,13 +3,21 @@
 
 from django.core.mail import mail_admins, send_mail
 from django.template import loader
-from django_q.tasks import async_task
 
 from appointment.logger_config import get_logger
 from appointment.settings import APP_DEFAULT_FROM_EMAIL, check_q_cluster
 
 logger = get_logger(__name__)
 
+try:
+    from django_q.tasks import async_task
+
+    DJANGO_Q_AVAILABLE = True
+except ImportError:
+    async_task = None
+    DJANGO_Q_AVAILABLE = False
+    logger.warning("django-q is not installed. Email will be send synchronously.")
+
 
 def has_required_email_settings():
     """Check if all required email settings are configured and warn if any are missing."""
@@ -45,7 +53,7 @@ def send_email(recipient_list, subject: str, template_url: str = None, context:
     from_email = from_email or APP_DEFAULT_FROM_EMAIL
     html_message = render_email_template(template_url, context)
 
-    if get_use_django_q_for_emails() and check_q_cluster():
+    if get_use_django_q_for_emails() and check_q_cluster() and DJANGO_Q_AVAILABLE:
         # Asynchronously send the email using Django-Q
         async_task(
                 "appointment.tasks.send_email_task", recipient_list=recipient_list, subject=subject,
@@ -69,7 +77,7 @@ def notify_admin(subject: str, template_url: str = None, context: dict = None, m
 
     html_message = render_email_template(template_url, context)
 
-    if get_use_django_q_for_emails() and check_q_cluster():
+    if get_use_django_q_for_emails() and check_q_cluster() and DJANGO_Q_AVAILABLE:
         # Enqueue the task to send admin email asynchronously
         async_task('appointment.tasks.notify_admin_task', subject=subject, message=message, html_message=html_message)
     else:

appointment/settings.py

@@ -37,29 +37,15 @@ def check_q_cluster():
     logger.info("Checking missing configuration for django q cluster")
     # Check if Django Q is installed
     if 'django_q' not in settings.INSTALLED_APPS:
-        missing_conf.append("Django Q is not in settings.INSTALLED_APPS. Please add it to the list.\n"
-                            "Example: \n\n"
-                            "INSTALLED_APPS = [\n"
-                            "    ...\n"
-                            "    'appointment',\n"
-                            "    'django_q',\n"
-                            "]\n")
+        missing_conf.append("Django Q is not in settings.INSTALLED_APPS. Please add it to the list. "
+                            "See https://django-appt-doc.adamspierredavid.com/getting-started/#installation "
+                            "for more information")
 
     # Check if Q_CLUSTER configuration is defined
     if not hasattr(settings, 'Q_CLUSTER'):
-        missing_conf.append("Q_CLUSTER is not defined in settings. Please define it.\n"
-                            "Example: \n\n"
-                            "Q_CLUSTER = {\n"
-                            "    'name': 'DjangORM',\n"
-                            "    'workers': 4,\n"
-                            "    'timeout': 90,\n"
-                            "    'retry': 120,\n"
-                            "    'queue_limit': 50,\n"
-                            "    'bulk': 10,\n"
-                            "    'orm': 'default',\n"
-                            "}\n"
-                            "Then run 'python manage.py qcluster' to start the worker.\n"
-                            "See https://django-q.readthedocs.io/en/latest/configure.html for more information.")
+        missing_conf.append("Q_CLUSTER is not defined in settings. Please define it. "
+                            "See https://django-appt-doc.adamspierredavid.com/project-structure/#configuration "
+                            "for more information.")
 
     # Log warnings if any configurations are missing
     if missing_conf:

appointment/tests/test_settings.py

@@ -22,13 +22,9 @@ def test_check_q_cluster_with_django_q_missing(self, mock_logger, mock_settings)
         self.assertFalse(result)
         # Verify logger was called with the expected warning about 'django_q' not being installed
         mock_logger.warning.assert_called_with(
-            "Django Q is not in settings.INSTALLED_APPS. Please add it to the list.\n"
-            "Example: \n\n"
-            "INSTALLED_APPS = [\n"
-            "    ...\n"
-            "    'appointment',\n"
-            "    'django_q',\n"
-            "]\n")
+                "Django Q is not in settings.INSTALLED_APPS. Please add it to the list. "
+                "See https://django-appt-doc.adamspierredavid.com/getting-started/#installation "
+                "for more information")
 
     @patch('appointment.settings.settings')
     @patch('appointment.settings.logger')

appointment/tests/utils/test_db_helpers.py

@@ -241,7 +241,7 @@ def test_create_and_save_appointment_without_django_q(self):
         self.assertIsNotNone(appointment)
         self.assertEqual(appointment.client.email, client_data['email'])
         mock_logger_warning.assert_called_with(
-            f"Email reminder requested for appointment {appointment.id}, but django-q is not available.")
+                f"Email reminder requested for appointment {appointment.id}, but django-q is not available.")
 
 
 def get_mock_reverse(url_name, **kwargs):

appointment/utils/db_helpers.py

@@ -11,13 +11,12 @@
 from urllib.parse import urlparse
 
 from django.apps import apps
+from django.conf import settings
 from django.contrib.auth import get_user_model
 from django.core.cache import cache
 from django.core.exceptions import FieldDoesNotExist
 from django.urls import reverse
 from django.utils import timezone
-from django_q.models import Schedule
-from django_q.tasks import schedule
 
 from appointment.logger_config import get_logger
 from appointment.settings import (
@@ -26,6 +25,23 @@
 )
 from appointment.utils.date_time import combine_date_and_time, get_weekday_num
 
+logger = get_logger(__name__)
+
+# Check if django-q is installed in settings
+DJANGO_Q_AVAILABLE = 'django_q' in settings.INSTALLED_APPS
+
+# Check if django-q is installed as a dependency
+try:
+    from django_q.models import Schedule
+    from django_q.tasks import schedule
+
+    DJANGO_Q_AVAILABLE = True
+except ImportError:
+    DJANGO_Q_AVAILABLE = False
+    Schedule = None
+    schedule = None
+    logger.warning("django-q is not installed. Email reminders will not be scheduled.")
+
 Appointment = apps.get_model('appointment', 'Appointment')
 AppointmentRequest = apps.get_model('appointment', 'AppointmentRequest')
 WorkingHours = apps.get_model('appointment', 'WorkingHours')
@@ -37,8 +53,6 @@
 EmailVerificationCode = apps.get_model('appointment', 'EmailVerificationCode')
 AppointmentRescheduleHistory = apps.get_model('appointment', 'AppointmentRescheduleHistory')
 
-logger = get_logger(__name__)
-
 
 def calculate_slots(start_time, end_time, buffer_time, slot_duration):
     """Calculate the available slots between the given start and end times using the given buffer time and slot duration
@@ -112,16 +126,17 @@ def create_and_save_appointment(ar, client_data: dict, appointment_data: dict, r
     logger.info(f"New appointment created: {appointment.to_dict()}")
     if appointment.want_reminder:
         logger.info(f"User wants a reminder for appointment {appointment.id}, scheduling it...")
-        schedule_email_reminder(appointment, request)
+        if DJANGO_Q_AVAILABLE:
+            schedule_email_reminder(appointment, request)
+        else:
+            logger.warning(f"Email reminder requested for appointment {appointment.id}, but django-q is not available.")
     return appointment
 
 
 def schedule_email_reminder(appointment, request, appointment_datetime=None):
     """Schedule an email reminder for the given appointment."""
-    # Check if the Django-Q cluster is running
-    from appointment.settings import check_q_cluster
-    if not check_q_cluster():
-        logger.warning("Django-Q cluster is not running. Email reminder will not be scheduled.")
+    if not DJANGO_Q_AVAILABLE:
+        logger.warning("Django-Q is not available. Email reminder will not be scheduled.")
         return
 
     # Calculate reminder datetime if not provided
@@ -155,6 +170,9 @@ def update_appointment_reminder(appointment, new_date, new_start_time, request,
     Updates or cancels the appointment reminder based on changes to the start time or date,
     and the user's preference for receiving a reminder.
     """
+    if not DJANGO_Q_AVAILABLE:
+        logger.warning("Django-Q is not available. Appointment reminder cannot be updated.")
+        return
     # Convert new date and time strings to datetime objects for comparison
     new_datetime = combine_date_and_time(new_date, new_start_time)
 
@@ -179,7 +197,7 @@ def update_appointment_reminder(appointment, new_date, new_start_time, request,
         cancel_existing_reminder(appointment.id_request)
 
         # If a reminder is still desired and the appointment is in the future, schedule a new one
-        if want_reminder and new_datetime > timezone.now() and DJANGO_Q_AVAILABLE:
+        if want_reminder and new_datetime > timezone.now():
             schedule_email_reminder(appointment, request, new_datetime)
         else:
             logger.info(
@@ -195,6 +213,9 @@ def cancel_existing_reminder(appointment_id_request):
     """
     Cancels any existing reminder for the appointment.
     """
+    if not DJANGO_Q_AVAILABLE:
+        logger.warning("Django-Q is not available. Appointment reminder cannot be updated.")
+        return
     task_name = f"reminder_{appointment_id_request}"
     Schedule.objects.filter(name=task_name).delete()
 

appointments/settings.py

@@ -42,7 +42,6 @@
     "django.contrib.messages",
     "django.contrib.staticfiles",
     'appointment.apps.AppointmentConfig',
-    'django_q',
 ]
 
 MIDDLEWARE = [
@@ -150,13 +149,3 @@
 ADMINS = [
     (os.getenv('ADMIN_NAME'), os.getenv('ADMIN_EMAIL')),
 ]
-
-Q_CLUSTER = {
-    'name': 'DjangORM',
-    'workers': 4,
-    'timeout': 90,
-    'retry': 120,
-    'queue_limit': 50,
-    'bulk': 10,
-    'orm': 'default',
-}

setup.cfg

@@ -27,5 +27,4 @@ install_requires =
     phonenumbers==8.13.42
     django-phonenumber-field==8.0.0
     babel==2.15.0
-    django-q2==1.6.2
     colorama~=0.4.6