NEMO-CE 7.3.5__tar.gz → 7.3.7__tar.gz

{nemo_ce-7.3.5 → nemo_ce-7.3.7}/NEMO/admin.py

@@ -287,7 +287,7 @@ class ToolAdmin(admin.ModelAdmin):
                     "parent_tool",
                     "_category",
                     "_operation_mode",
-                    "_qualifications_never_expire",
+                    "qualified_users",
                     "_problem_shutdown_enabled",
                     "_reservation_required",
                     "_logout_grace_period",
@@ -342,6 +342,17 @@ class ToolAdmin(admin.ModelAdmin):
                 )
             },
         ),
+        (
+            "Qualification expiration",
+            {
+                "fields": (
+                    "_qualification_reminder_days",
+                    "_qualification_expiration_days",
+                    "_qualification_expiration_never_used_days",
+                    "_qualification_notification_email",
+                )
+            },
+        ),
         (
             "Area Access",
             {

nemo_ce-7.3.7/NEMO/migrations/0141_toolqualificationexpiration.py

@@ -0,0 +1,103 @@
+# Generated by Django 4.2.27 on 2026-01-26 20:42
+
+import re
+
+import django.core.validators
+import django.db.models.deletion
+from django.db import migrations, models
+
+import NEMO.fields
+
+
+def migrate_tool_qualification_expiration_forward(apps, schema_editor):
+    Tool = apps.get_model("NEMO", "Tool")
+    Customization = apps.get_model("NEMO", "Customization")
+    tool_qualification_reminder_days = Customization.objects.filter(name="tool_qualification_reminder_days").first()
+    tool_qualification_expiration_days = Customization.objects.filter(name="tool_qualification_expiration_days").first()
+    tool_qualification_expiration_never_used_days = Customization.objects.filter(
+        name="tool_qualification_expiration_never_used_days"
+    ).first()
+    tool_qualification_notification = Customization.objects.filter(name="tool_qualification_cc").first()
+    for tool in Tool.objects.filter(_qualifications_never_expire=False, parent_tool__isnull=True):
+        if tool_qualification_expiration_days or tool_qualification_expiration_never_used_days:
+            if tool_qualification_reminder_days and tool_qualification_reminder_days.value:
+                tool._qualification_reminder_days = tool_qualification_reminder_days.value
+            if tool_qualification_expiration_days and tool_qualification_expiration_days.value:
+                tool._qualification_expiration_days = tool_qualification_expiration_days.value
+            if tool_qualification_expiration_never_used_days and tool_qualification_expiration_never_used_days.value:
+                tool._qualification_expiration_never_used_days = tool_qualification_expiration_never_used_days.value
+            if tool_qualification_notification and tool_qualification_notification.value:
+                tool._qualification_notification_email = tool_qualification_notification.value
+            tool.save()
+    if tool_qualification_reminder_days:
+        tool_qualification_reminder_days.delete()
+    if tool_qualification_expiration_days:
+        tool_qualification_expiration_days.delete()
+    if tool_qualification_expiration_never_used_days:
+        tool_qualification_expiration_never_used_days.delete()
+    if tool_qualification_notification:
+        tool_qualification_notification.delete()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("NEMO", "0140_alter_user_options"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="tool",
+            name="_qualification_expiration_days",
+            field=models.PositiveIntegerField(
+                db_column="qualification_expiration_days",
+                blank=True,
+                help_text="The number of days from the user’s last tool use until the qualification expires.",
+                null=True,
+            ),
+        ),
+        migrations.AddField(
+            model_name="tool",
+            name="_qualification_expiration_never_used_days",
+            field=models.PositiveIntegerField(
+                db_column="qualification_expiration_never_used_days",
+                blank=True,
+                help_text="Number of days from the user's first qualification until the qualification expires (if the user never used the tool).",
+                null=True,
+            ),
+        ),
+        migrations.AddField(
+            model_name="tool",
+            name="_qualification_notification_email",
+            field=NEMO.fields.MultiEmailField(
+                db_column="qualification_notification_email",
+                blank=True,
+                help_text="The email addresses to cc on tool qualification expiration and on reminders. Separate multiple emails with commas.",
+                max_length=2000,
+                null=True,
+            ),
+        ),
+        migrations.AddField(
+            model_name="tool",
+            name="_qualification_reminder_days",
+            field=models.CharField(
+                db_column="qualification_reminder_days",
+                blank=True,
+                help_text="The (optional) number of days to send a reminder prior to the user's tool qualification expiration (below). A comma-separated list can be used for multiple reminders. This applies to both expiration cases.",
+                max_length=255,
+                null=True,
+                validators=[
+                    django.core.validators.RegexValidator(
+                        re.compile("^\\d+(?:,\\d+)*\\Z"),
+                        code="invalid",
+                        message="Enter only digits separated by commas.",
+                    )
+                ],
+            ),
+        ),
+        migrations.RunPython(migrate_tool_qualification_expiration_forward, migrations.RunPython.noop),
+        migrations.RemoveField(
+            model_name="tool",
+            name="_qualifications_never_expire",
+        ),
+    ]

{nemo_ce-7.3.5 → nemo_ce-7.3.7}/NEMO/models.py

@@ -1349,10 +1349,32 @@ class Tool(SerializationByNameModel):
     _interlock = models.OneToOneField(
         "Interlock", db_column="interlock_id", blank=True, null=True, on_delete=models.SET_NULL
     )
-    _qualifications_never_expire = models.BooleanField(
-        default=False,
-        db_column="qualifications_never_expire",
-        help_text="Check this box if qualifications for this tool should never expire (even if the tool qualification expiration feature is enabled).",
+    # Qualification expiration fields
+    _qualification_reminder_days = models.CharField(
+        db_column="qualification_reminder_days",
+        null=True,
+        blank=True,
+        max_length=CHAR_FIELD_MEDIUM_LENGTH,
+        validators=[validate_comma_separated_integer_list],
+        help_text="The (optional) number of days to send a reminder prior to the user's tool qualification expiration (below). A comma-separated list can be used for multiple reminders. This applies to both expiration cases.",
+    )
+    _qualification_expiration_days = models.PositiveIntegerField(
+        db_column="qualification_expiration_days",
+        null=True,
+        blank=True,
+        help_text="The number of days from the user’s last tool use until the qualification expires.",
+    )
+    _qualification_expiration_never_used_days = models.PositiveIntegerField(
+        db_column="qualification_expiration_never_used_days",
+        null=True,
+        blank=True,
+        help_text="Number of days from the user's first qualification until the qualification expires (if the user never used the tool).",
+    )
+    _qualification_notification_email = fields.MultiEmailField(
+        db_column="qualification_notification_email",
+        null=True,
+        blank=True,
+        help_text="The email addresses to cc on tool qualification expiration and on reminders. Separate multiple emails with commas.",
     )
     # Policy fields:
     _requires_area_access = TreeForeignKey(
@@ -1530,17 +1552,6 @@ class Tool(SerializationByNameModel):
         self.raise_setter_error_if_child_tool("category")
         self._category = value
 
-    @property
-    def qualifications_never_expire(self):
-        return (
-            self.parent_tool.qualifications_never_expire if self.is_child_tool() else self._qualifications_never_expire
-        )
-
-    @qualifications_never_expire.setter
-    def qualifications_never_expire(self, value):
-        self.raise_setter_error_if_child_tool("qualifications_never_expire")
-        self._qualifications_never_expire = value
-
     @property
     def description(self):
         return self.parent_tool.description if self.is_child_tool() else self._description
@@ -1689,6 +1700,56 @@ class Tool(SerializationByNameModel):
         self.raise_setter_error_if_child_tool("interlock")
         self._interlock = value
 
+    @property
+    def qualification_reminder_days(self):
+        return (
+            self.parent_tool.qualification_reminder_days if self.is_child_tool() else self._qualification_reminder_days
+        )
+
+    @qualification_reminder_days.setter
+    def qualification_reminder_days(self, value):
+        self.raise_setter_error_if_child_tool("qualification_reminder_days")
+        self._qualification_reminder_days = value
+
+    @property
+    def qualification_expiration_days(self):
+        return (
+            self.parent_tool.qualification_expiration_days
+            if self.is_child_tool()
+            else self._qualification_expiration_days
+        )
+
+    @qualification_expiration_days.setter
+    def qualification_expiration_days(self, value):
+        self.raise_setter_error_if_child_tool("qualification_expiration_days")
+        self._qualification_expiration_days = value
+
+    @property
+    def qualification_expiration_never_used_days(self):
+        return (
+            self.parent_tool.qualification_expiration_never_used_days
+            if self.is_child_tool()
+            else self._qualification_expiration_never_used_days
+        )
+
+    @qualification_expiration_never_used_days.setter
+    def qualification_expiration_never_used_days(self, value):
+        self.raise_setter_error_if_child_tool("qualification_expiration_never_used_days")
+        self._qualification_expiration_never_used_days = value
+
+    @property
+    def qualification_notification_email(self):
+        return (
+            self.parent_tool.qualification_notification_email
+            if self.is_child_tool()
+            else self._qualification_notification_email
+        )
+
+    @qualification_notification_email.setter
+    def qualification_notification_email(self, value):
+        self.raise_setter_error_if_child_tool("qualification_notification_email")
+        self._qualification_notification_email = value
+
     @property
     def requires_area_access(self):
         return self.parent_tool.requires_area_access if self.is_child_tool() else self._requires_area_access
@@ -2379,6 +2440,11 @@ class Tool(SerializationByNameModel):
         else:
             raise ValueError(f"A {'project' if user else 'user'} must be provided for usage questions")
 
+    def get_qualification_reminder_days(self) -> List[int]:
+        if not self.qualification_reminder_days:
+            return []
+        return [int(days) for days in self.qualification_reminder_days.split(",") if days]
+
     def clean(self):
         errors = {}
         if self.parent_tool_id:

{nemo_ce-7.3.5 → nemo_ce-7.3.7}/NEMO/templates/calendar/calendar.html

@@ -646,7 +646,7 @@ function create_calendar()
 	{
 		"header":
 		{
-			left: "prev,next today{% if user.is_staff or user.is_tool_staff %} proxyReservation scheduledOutage{% endif %}{% if customizations|get_item:"training_module_enabled" == "enabled" %}{% if user|is_trainer %} scheduleTraining{% endif %} requestTraining{% endif %} login logout",
+			left: "prev,next today{% if user.is_staff or user.is_tool_staff %} proxyReservation scheduledOutage{% endif %}{% if customizations|get_item:"training_module_enabled" == "enabled" %}{% if user|is_trainer %} scheduleTraining{% endif %} {% if customizations|get_item:"training_requests_enabled" == "enabled" %}requestTraining{% endif %}{% endif %} login logout",
 			center: "title",
 			right: "agendaDay,agendaWeek,month"
 		},
@@ -752,6 +752,7 @@ function reserve_for_someone_else_callback(response, status, xml_http_request)
 
 function request_training()
 {
+	{% if customizations|get_item:"training_requests_enabled" == "enabled" %}
 	let item = get_selected_item();
 	if (item && item !== "personal_schedule" && item !== "all_tools" && item !== "all_areas" && item !== "all_areastools")
 	{
@@ -779,6 +780,7 @@ function request_training()
 		}
 		ajax_get(url, event_changes, function(response, status, xml_http_request) { return event_creation_success_callback(response, status, xml_http_request, post_data, callback) }, [ajax_failure_callback("Training request creation failed")])
 	}
+    {% endif %}
 }
 
 function login_to_area()

{nemo_ce-7.3.5 → nemo_ce-7.3.7}/NEMO/templates/customizations/customizations_tool.html

@@ -623,97 +623,14 @@
             </div>
         </div>
         <div class="customization-separation" style="margin-bottom: 15px"></div>
-        <h3 class="customization-section-title">Tool qualification expiration</h3>
-        <p>
-            If active, this feature will remove tool qualification from a user if the user has not used the tool after a while or never used it since qualified (configured separately).
-            <p>
-                The <a href="{% url 'customization' 'templates' %}?#tool_qualification_expiration_email_id">user tool qualification expiration email</a> need to be set to enable this feature.
-            </p>
-            <br />
-            <div class="form-group {% if errors.tool_qualification_reminder_days %}has-error{% endif %}">
-                <label class="control-label col-md-3" for="tool_qualification_reminder_days">Reminder days</label>
-                <div class="col-md-5">
-                    <input type="text"
-                           id="tool_qualification_reminder_days"
-                           name="tool_qualification_reminder_days"
-                           class="form-control"
-                           value="{% if errors.tool_qualification_reminder_days %}{{ errors.tool_qualification_reminder_days.value }}{% else %}{{ tool_qualification_reminder_days }}{% endif %}" />
-                </div>
-                <div class="col-md-offset-3 col-md-9 help-block light-grey">
-                    {% if errors.tool_qualification_reminder_days %}
-                        {{ errors.tool_qualification_reminder_days.error }}
-                    {% else %}
-                        The (optional) number of days to send a reminder prior to the user's tool qualification expiration (below). A comma-separated list can be used for multiple reminders. This applies to both expiration cases.
-                    {% endif %}
-                </div>
-            </div>
-            <div class="form-group {% if errors.tool_qualification_expiration_days %}has-error{% endif %}">
-                <label class="control-label col-md-3" for="tool_qualification_expiration_days">
-                    Expiration days (previous tool usage)
-                </label>
-                <div class="col-md-5">
-                    <input type="number"
-                           step="1"
-                           id="tool_qualification_expiration_days"
-                           name="tool_qualification_expiration_days"
-                           class="form-control"
-                           value="{% if errors.tool_qualification_expiration_days %}{{ errors.tool_qualification_expiration_days.value }}{% else %}{{ tool_qualification_expiration_days }}{% endif %}" />
-                </div>
-                <div class="col-md-offset-3 col-md-9 help-block light-grey">
-                    {% if errors.tool_qualification_expiration_days %}
-                        {{ errors.tool_qualification_expiration_days.error }}
-                    {% else %}
-                        The number of days before the user's tool qualification expires since the user last used the tool.
-                    {% endif %}
-                </div>
-            </div>
-            <div class="form-group {% if errors.tool_qualification_expiration_never_used_days %}has-error{% endif %}">
-                <label class="control-label col-md-3" for="tool_qualification_expiration_never_used_days">
-                    Expiration days (no tool usage)
-                </label>
-                <div class="col-md-5">
-                    <input type="number"
-                           step="1"
-                           id="tool_qualification_expiration_never_used_days"
-                           name="tool_qualification_expiration_never_used_days"
-                           class="form-control"
-                           value="{% if errors.tool_qualification_expiration_never_used_days %}{{ errors.tool_qualification_expiration_never_used_days.value }}{% else %}{{ tool_qualification_expiration_never_used_days }}{% endif %}" />
-                </div>
-                <div class="col-md-offset-3 col-md-9 help-block light-grey">
-                    {% if errors.tool_qualification_expiration_never_used_days %}
-                        {{ errors.tool_qualification_expiration_never_used_days.error }}
-                    {% else %}
-                        The number of days before the user's tool qualification expires since the user qualified for the first time.
-                    {% endif %}
-                </div>
-            </div>
-            <div class="form-group {% if errors.tool_qualification_cc %}has-error{% endif %}">
-                <label class="control-label col-md-3" for="tool_qualification_cc">Reminder/expiration CC</label>
-                <div class="col-md-5">
-                    <input type="text"
-                           id="tool_qualification_cc"
-                           name="tool_qualification_cc"
-                           class="form-control"
-                           value="{% if errors.tool_qualification_cc %}{{ errors.tool_qualification_cc.value }}{% else %}{{ tool_qualification_cc }}{% endif %}"
-                           placeholder="information@example.org" />
-                </div>
-                <div class="col-md-offset-3 col-md-9 help-block light-grey">
-                    {% if errors.tool_qualification_cc %}
-                        {{ errors.tool_qualification_cc.error }}
-                    {% else %}
-                        Extra email address to copy when a user's tool qualification reminder/expiration email is sent. A comma-separated list can be used.
-                    {% endif %}
-                </div>
-            </div>
-            <div class="customization-separation" style="margin-bottom: 15px"></div>
-            <div class="text-center">{% button type="save" value="Save settings" %}</div>
-        </div>
-        <script type="text/javascript">
+        <div class="text-center">{% button type="save" value="Save settings" %}</div>
+    </div>
+    <script type="text/javascript">
         $("#tool-tab-link").click(function() {setTimeout(on_tool_tab_show, 50)});
         function on_tool_tab_show()
         {
             auto_size_textarea(document.getElementById('tool_control_configuration_setting_template'))
         }
         on_tool_tab_show();
-        </script>
-    </form>
+    </script>
+</form>

{nemo_ce-7.3.5 → nemo_ce-7.3.7}/NEMO/templates/customizations/customizations_training.html

@@ -82,6 +82,15 @@
                         Enable training requests/sessions scheduling
                     </label>
                     <br />
+                    <input type="hidden" name="training_requests_enabled" value="off" />
+                    <label>
+                        <input type="checkbox"
+                               name="training_requests_enabled"
+                               {% if training_requests_enabled == "enabled" %}checked{% endif %}
+                               value="enabled">
+                        Allow users to request training (via calendar or training module)
+                    </label>
+                    <br />
                     <label>
                         <input type="checkbox"
                                name="training_show_in_user_requests"

{nemo_ce-7.3.5 → nemo_ce-7.3.7}/NEMO/templates/tool_control/tool_status.html

@@ -17,7 +17,7 @@
                         History</a>
                     </li>
                 {% endif %}
-                {% if show_usage_data_tab and not hide_data_history or show_usage_data_tab and user.is_any_part_of_staff %}
+                {% if not hide_data_history or user.is_any_part_of_staff %}
                     <li style="width: 48%; text-align: center">
                         <a href="#usage_data" style="padding: 10px 5px;" onclick="load_usage_data('{{ tool.id }}');">Usage Data History</a>
                     </li>
@@ -50,7 +50,7 @@
                         <a href="#config" onclick="load_config_history('{{ tool.id }}');">Config History</a>
                     </li>
                 {% endif %}
-                {% if show_usage_data_tab and not hide_data_history or show_usage_data_tab and user.is_any_part_of_staff %}
+                {% if not hide_data_history or user.is_any_part_of_staff %}
                     <li>
                         <a href="#usage_data" onclick="load_usage_data('{{ tool.id }}');">Run Data History</a>
                     </li>

{nemo_ce-7.3.5 → nemo_ce-7.3.7}/NEMO/templates/tool_control/usage_data.html

@@ -98,21 +98,61 @@
         </div>
     </div>
 </form>
-{% if run_data_table.rows or pre_run_data_table.rows %}
+{% if data_table.rows or run_data_table.rows or pre_run_data_table.rows %}
     <ul class="nav nav-tabs" id="usage-tabs">
+        {% if data_table.rows %}
+            <li class="active">
+                <a data-toggle="tab" href="#run-data-tab">Usage history</a>
+            </li>
+        {% endif %}
         {% if pre_run_data_table.rows %}
-            <li class="{% if pre_run_data_table.rows %}active{% endif %}">
+            <li>
                 <a data-toggle="tab" href="#pre-run-data-tab">Pre-Run Data</a>
             </li>
         {% endif %}
         {% if run_data_table.rows %}
-            <li class="{% if not pre_run_data_table.rows %}active{% endif %}">
-                <a data-toggle="tab" href="#run-data-tab">Run Data</a>
+            <li>
+                <a data-toggle="tab" href="#post-run-data-tab">Post-Run Data</a>
             </li>
         {% endif %}
     </ul>
     <div class="tab-content" style="margin-bottom: 0;">
-        <div class="tab-pane {% if pre_run_data_table.rows %}active{% endif %}" id="pre-run-data-tab">
+        <div class="tab-pane active" id="run-data-tab">
+            <div class="col-md-12">
+                <div class="form-group pull-right extra-side-padding" style="padding-top: 10px">
+                    {% button type="export" value="Export" onclick="load_usage_data('"|concat:tool_id|concat:"', 'all');" %}
+                </div>
+            </div>
+            <div class="panel-body">
+                {% if data_table.rows %}
+                    <table class="table table-bordered table-condensed" style="margin-top: 10px">
+                        <thead>
+                            <tr>
+                                {% for header in data_table.flat_headers %}<th>{{ header }}</th>{% endfor %}
+                            </tr>
+                        </thead>
+                        <tbody>
+                            {% for row in data_table.flat_rows %}
+                                <tr>
+                                    {% for item in row %}
+                                        <td>
+                                            {% if item|class_name == "list" %}
+                                                {{ item|join:", " }}
+                                            {% else %}
+                                                {{ item|default_if_none:"" }}
+                                            {% endif %}
+                                        </td>
+                                    {% endfor %}
+                                </tr>
+                            {% endfor %}
+                        </tbody>
+                    </table>
+                {% else %}
+                    <span class="italic">No run data was found between these dates</span>
+                {% endif %}
+            </div>
+        </div>
+        <div class="tab-pane" id="pre-run-data-tab">
             <div class="col-md-12">
                 <div class="form-group pull-right extra-side-padding" style="padding-top: 10px">
                     {% button type="export" value="Export" onclick="load_usage_data('"|concat:tool_id|concat:"', 'pre-run');" %}
@@ -147,7 +187,7 @@
                 {% endif %}
             </div>
         </div>
-        <div class="tab-pane {% if not pre_run_data_table.rows %}active{% endif %}" id="run-data-tab">
+        <div class="tab-pane" id="post-run-data-tab">
             <div class="col-md-12">
                 <div class="form-group pull-right extra-side-padding" style="padding-top: 10px">
                     {% button type="export" value="Export" onclick="load_usage_data('"|concat:tool_id|concat:"', 'run');" %}

{nemo_ce-7.3.5 → nemo_ce-7.3.7}/NEMO/templates/training_new/training_base.html

@@ -1,5 +1,6 @@
 {% extends "base.html" %}
 {% load training_tags %}
+{% load custom_tags_and_filters %}
 {% block extrahead %}
     {% load static %}
     <script type="text/javascript" src="{% static "datetimepicker/bootstrap-datetimepicker.js" %}"></script>
@@ -13,14 +14,16 @@
 {% block content %}
     <h1 style="margin-top: 0; margin-bottom: 25px">Training dashboard</h1>
     <ul class="nav nav-tabs">
-        {% url 'training_requests' as training_requests_url %}
-        <li class="nav-item{% if training_requests_url in request.path %} active{% endif %}">
-            <a class="nav-link" href="{{ training_requests_url }}">My training requests
-                {% if training_invitation_notification_count %}
-                    <span class="badge badge-tab-top">{{ training_invitation_notification_count }}</span>
-                {% endif %}
-            </a>
-        </li>
+        {% if customizations|get_item:"training_requests_enabled" == "enabled" %}
+            {% url 'training_requests' as training_requests_url %}
+            <li class="nav-item{% if training_requests_url in request.path %} active{% endif %}">
+                <a class="nav-link" href="{{ training_requests_url }}">My training requests
+                    {% if training_invitation_notification_count %}
+                        <span class="badge badge-tab-top">{{ training_invitation_notification_count }}</span>
+                    {% endif %}
+                </a>
+            </li>
+        {% endif %}
         {% url 'training_history' as training_history_url %}
         <li class="nav-item{% if training_history_url in request.path %} active{% endif %}">
             <a class="nav-link" href="{{ training_history_url }}">Training history</a>