aa-fleetfinder 2.6.1__py3-none-any.whl → 2.7.0__py3-none-any.whl

fleetfinder/templates/fleetfinder/dashboard.html

@@ -19,12 +19,7 @@
                             <th>{% translate "Fleet commander" %}</th>
                             <th>{% translate "Fleet name" %}</th>
                             <th>{% translate "Created at" %}</th>
-                            <th>{% translate "Join" %}</th>
-
-                            {% if perms.fleetfinder.manage_fleets %}
-                                <th>{% translate "Details" %}</th>
-                                <th>{% translate "Edit" %}</th>
-                            {% endif %}
+                            <th class="text-end"><!-- {% translate "Join" %} --></th>
                         </tr>
                     </thead>
 
@@ -43,20 +38,33 @@
 {% block extra_javascript %}
     {% include "bundles/datatables-js-bs5.html" %}
     {% include "bundles/moment-js.html" with locale=True %}
+    {% include "fleetfinder/bundles/js/fleetfinder-js.html" %}
 
     {% get_datatables_language_static LANGUAGE_CODE as DT_LANG_PATH %}
 
     <script>
+        /* global fetchGet, fleetfinderBootstrapTooltip, DataTable */
+        'use strict';
+
         $(document).ready(() => {
             const DATETIME_FORMAT = 'YYYY-MM-DD, HH:mm';
             const table_fleet_overview = $('#table_available-fleets');
+            let dataTable = null;
 
             /**
-             * Initialize the datatable for fleet overview
+             * Initialize or update the datatable.
+             * If the table already exists, it will be updated with new data.
+             * If it does not exist, a new DataTable will be created.
+             *
+             * @param {Object} data - The fleet data to populate the table.
              */
-            fetchGet({url: '{% url "fleetfinder:ajax_dashboard" %}'})
-                .then((data) => {
-                    table_fleet_overview.DataTable({
+            const initializeOrUpdateTable = (data) => {
+                if (dataTable) {
+                    // Update existing table
+                    dataTable.clear().rows.add(data).draw();
+                } else {
+                    // Initialize new table
+                    dataTable = table_fleet_overview.DataTable({
                         language: {
                             url: '{{ DT_LANG_PATH }}'
                         },
@@ -69,15 +77,16 @@
                                     sort: 'sort'
                                 }
                             },
-                            {data: 'fleet_name'},
+                            {
+                                data: 'fleet_name'
+                            },
                             {
                                 data: 'created_at',
                             },
-                            {data: 'join'},
-                            {% if perms.fleetfinder.manage_fleets %}
-                                {data: 'details'},
-                                {data: 'edit'},
-                            {% endif %}
+                            {
+                                data: 'actions',
+                                className: 'text-end',
+                            },
                         ],
                         columnDefs: [
                             {
@@ -88,66 +97,32 @@
                                 orderable: false,
                                 targets: [3]
                             },
-                            {% if perms.fleetfinder.manage_fleets %}
-                                {
-                                    orderable: false,
-                                    targets: [4, 5]
-                                },
-                            {% endif %}
                         ],
                         order: [[0, 'asc']],
                         paging: false
                     });
-                })
-                .catch((error) => {
-                    console.error('Error fetching fleet data:', error);
-                });
+                }
 
-            /**
-             * Refresh the datatable information every 30 seconds
-             */
-            const intervalReloadDatatable = 30000; // ms
-            let expectedReloadDatatable = Date.now() + intervalReloadDatatable;
+                // Initialize Bootstrap tooltips
+                fleetfinderBootstrapTooltip({selector: '#table_available-fleets'});
+            };
 
             /**
-             * reload datatable "fleetfinderFleetsTable"
+             * Fetch and update fleet data
              */
-            const reloadDataTable = () => {
-                const dt = Date.now() - expectedReloadDatatable; // The drift (positive for overshooting)
-
-                if (dt > intervalReloadDatatable) {
-                    // Something awful happened. Maybe the browser (tab) was inactive?
-                    // Possibly special handling to avoid futile "catch up" run
-                    console.log('Something went wrong, reloading page ...');
-
-                    window.location.replace(
-                        window.location.pathname + window.location.search + window.location.hash
-                    );
-                }
-
+            const fetchFleetData = () => {
                 fetchGet({url: '{% url "fleetfinder:ajax_dashboard" %}'})
-                    .then((newData) => {
-                        const dataTable = table_fleet_overview.DataTable();
-
-                        dataTable.clear().rows.add(newData).draw();
-                    })
+                    .then(initializeOrUpdateTable)
                     .catch((error) => {
-                        console.error('Error fetching updated data:', error);
+                        console.error('Error fetching fleet data:', error);
                     });
-
-                expectedReloadDatatable += intervalReloadDatatable;
-
-                // Take drift into account
-                setTimeout(
-                    reloadDataTable,
-                    Math.max(0, intervalReloadDatatable - dt)
-                );
             };
 
-            setTimeout(
-                reloadDataTable,
-                intervalReloadDatatable
-            );
+            // Initial load
+            fetchFleetData();
+
+            // Refresh every 30 seconds
+            setInterval(fetchFleetData, 30000);
         });
     </script>
 {% endblock %}

fleetfinder/templates/fleetfinder/fleet-details.html

@@ -50,6 +50,7 @@
                                     <th>{% translate "Name" %}</th>
                                     <th>{% translate "Ship class" %}</th>
                                     <th>{% translate "System" %}</th>
+                                    <th><!-- {% translate "Action" %} --></th>
                                 </tr>
                             </thead>
 
@@ -60,6 +61,8 @@
             </div>
         </div>
     </div>
+
+    {% include "fleetfinder/modals/kick-fleet-member.html" %}
 {% endblock %}
 
 {% block extra_css %}
@@ -69,39 +72,83 @@
 
 {% block extra_javascript %}
     {% include "bundles/datatables-js-bs5.html" %}
+    {% include "fleetfinder/bundles/js/fleetfinder-js.html" %}
 
     {% get_datatables_language_static LANGUAGE_CODE as DT_LANG_PATH %}
 
+    {% translate "Fleet boss" as l10nFleetBoss %}
+    {% translate "Kick member from fleet" as l10nKickMemberFromFleet %}
+    {% translate "An unknown error occurred." as l10nUnknownError %}
+
     <script>
         $(document).ready(() => {
+            /* DataTables
+            ------------------------------------------------------------------------- */
             const table_fleet_members = $('#table_fleet_members');
-            const tabe_fleet_composition = $('#table_fleet_composition');
+            const table_fleet_composition = $('#table_fleet_composition');
+            const url = {
+                kickFleetMember: '{% url "fleetfinder:ajax_fleet_kick_member" fleet_id %}',
+                fleetDetails: '{% url "fleetfinder:ajax_fleet_details" fleet_id %}'
+            };
+            const dataTableConfig = {
+                language: {
+                    url: '{{ DT_LANG_PATH }}'
+                },
+                paging: false,
+                destroy: true
+            };
 
             const populateDatatables = () => {
                 fetchGet({
-                    url: '{% url "fleetfinder:ajax_fleet_details" fleet_id %}'
+                    url: url.fleetDetails,
                 })
                 .then((data) => {
                     table_fleet_members.DataTable({
-                        language: {
-                            url: '{{ DT_LANG_PATH }}'
-                        },
-                        destroy: true,
-                        paging: false,
+                        ...dataTableConfig,
                         data: data.fleet_member,
                         columns: [
-                            {data: 'character_name'},
+                            {
+                                render: (data, type, row) => {
+                                    const fwIcon = '<i class="fa-solid fa-star"></i>';
+
+                                    return row.is_fleet_boss
+                                        ? `${row.character_name} <sup data-bs-tooltip="aa-fleetfinder" title="{{ l10nFleetBoss|escapejs }}"><small>${fwIcon}</small></sup>`
+                                        : row.character_name;
+                                }
+                            },
                             {data: 'ship_type_name'},
-                            {data: 'solar_system_name'}
-                        ]
+                            {data: 'solar_system_name'},
+                            {
+                                render: (data, type, row) => {
+                                    const fwIcon = '<i class="fa-solid fa-user-minus"></i>';
+                                    const dataAttributes = Object.entries({
+                                        'data-character-id': row.character_id,
+                                        'data-character-name': row.character_name,
+                                        'data-bs-toggle': 'modal',
+                                        'data-bs-target': '#kick-fleet-member',
+                                        'data-bs-tooltip': 'aa-fleetfinder'
+                                    }).map(([key, value]) => {
+                                        return `${key}="${value}"`;
+                                    }).join(' ');
+
+                                    return row.is_fleet_boss
+                                        ? ''
+                                        : `<button type="button" class="btn btn-sm btn-danger" ${dataAttributes} title="{{ l10nKickMemberFromFleet|escapejs }}">${fwIcon}</button>`;
+                                },
+                                orderable: false,
+                                searchable: false,
+                                width: '50px',
+                                className: 'text-end'
+                            }
+                        ],
+                        createdRow: (row, data, rowIndex) => {
+                            $(row).attr('data-row-id', rowIndex);
+                            $(row).attr('data-character-id', data.character_id);
+                        }
                     });
 
-                    tabe_fleet_composition.DataTable({
-                        language: {
-                            url: '{{ DT_LANG_PATH }}'
-                        },
-                        destroy: true,
-                        paging: false,
+                    table_fleet_composition.DataTable({
+                        ...dataTableConfig,
                         data: data.fleet_composition,
                         columns: [
                             {data: 'ship_type_name'},
@@ -110,30 +157,74 @@
                         order: [[1, 'desc']]
                     });
                 })
+                .then(() => {
+                    // Initialize tooltips for the kick buttons
+                    fleetfinderBootstrapTooltip({selector: '#table_fleet_members'});
+                })
                 .catch((error) => {
                     console.error('Error fetching fleet details:', error);
                 });
             };
 
-            table_fleet_members.DataTable({
-                language: {
-                    url: '{{ DT_LANG_PATH }}'
-                },
-                paging: false,
-            });
-
-            tabe_fleet_composition.DataTable({
-                language: {
-                    url: '{{ DT_LANG_PATH }}'
-                },
-                paging: false,
-            });
-
             populateDatatables();
 
-            setInterval(function () {
-                populateDatatables();
-            }, 30000);
+            setInterval(populateDatatables, 30000);
+
+            /* Modals
+            ------------------------------------------------------------------------- */
+            // Handle the kick fleet member modal
+            $('#kick-fleet-member')
+                .on('show.bs.modal', (event) => {
+                    const button = $(event.relatedTarget);
+                    const characterId = button.data('character-id');
+                    const characterName = button.data('character-name');
+                    const link = url.kickFleetMember;
+                    const csrfToken = '{{ csrf_token }}';
+
+                    // Populate the modal content
+                    $('#kick-fleet-member-character-name').text(characterName);
+
+                    $('#modal-button-confirm-kick-fleet-member')
+                        // Remove any previous click handlers to avoid multiple bindings
+                        .off('click.kickMember')
+                        // Bind the click event for the confirmation button
+                        .on('click.kickMember', () => {
+                        fetchPost({
+                            url: link,
+                            csrfToken: csrfToken,
+                            payload: {
+                                memberId: characterId
+                            },
+                            responseIsJson: true
+                        })
+                        .then((data) => {
+                            if (data.success) {
+                                populateDatatables();
+
+                                $('#kick-fleet-member').modal('hide');
+                            } else {
+                                $('#kick-fleet-member .modal-kick-fleet-member-error')
+                                    .removeClass('d-none')
+                                    .find('.modal-kick-fleet-member-error-message')
+                                    .text(data.error || '{{ l10nUnknownError|escapejs }}');
+                            }
+                        })
+                        .catch((error) => {
+                            console.error('Error kicking fleet member:', error);
+                        });
+                    });
+                })
+                .on('hide.bs.modal', () => {
+                    // Reset modal content
+                    $('#kick-fleet-member-character-name').empty();
+                    $('#kick-fleet-member .modal-kick-fleet-member-error')
+                        .addClass('d-none')
+                        .find('.modal-kick-fleet-member-error-message')
+                        .empty();
+
+                    // Clean up event handler
+                    $('#modal-button-confirm-kick-fleet-member').off('click.kickMember');
+                });
         });
     </script>
 {% endblock %}

fleetfinder/templates/fleetfinder/join-fleet.html

@@ -18,11 +18,21 @@
         <div class="card-body container">
             <div class="row">
                 <div class="align-self-center">
+                    <p>
+                        {% translate "Please ensure you don't have any CSPA charges set on your characters, as this will prevent fleet invites from being sent." %}
+                        <br>
+                        {% translate "You can select multiple characters to invite them all at once." %}
+                    </p>
+
+                    <p>
+                        {% translate "The selected characters will receive a fleet invite in-game, which they can accept to join the fleet." %}
+                    </p>
+
                     <form class="form-signin" role="form" action="" method="POST">
                         {% csrf_token %}
 
                         <div class="mb-3">
-                            <label for="character_ids">{% translate "Select the characters to invite" %}</label>
+                            <label for="character_ids" class="mb-1">{% translate "Please select the characters you want to invite:" %}</label>
 
                             <select class="multiselect-ui" multiple="multiple" name="character_ids" id="character_ids">
                                 {% for character in characters %}

fleetfinder/templates/fleetfinder/modals/kick-fleet-member.html

@@ -0,0 +1,46 @@
+{% load bootstrap %}
+{% load i18n %}
+
+<!-- kick fleet member modal -->
+<div class="modal fade" id="kick-fleet-member" tabindex="-1" role="dialog" aria-hidden="true">
+    <div class="modal-dialog modal-dialog-centered" role="document">
+        <div class="modal-content">
+            <div class="modal-header">
+                <div class="modal-title fs-5">
+                    {% translate "Kick member from fleet" %}
+                </div>
+
+                <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="{% translate 'Close' %}"></button>
+            </div>
+
+            <div class="modal-body">
+                <p>
+                    {% translate "Are you sure you want to kick this member from the fleet?" %}
+                </p>
+
+                <p>
+                    <strong id="kick-fleet-member-character-name"></strong>
+                </p>
+
+                <p class="aa-callout aa-callout-danger modal-kick-fleet-member-error d-none">
+                    <i class="fa-solid fa-triangle-exclamation"></i>
+                    {% translate "Error" %}
+                    <br>
+                    <span class="modal-kick-fleet-member-error-message"></span>
+                </p>
+            </div>
+
+            <div class="modal-footer">
+                <button id="modal-button-cancel-kick-fleet-member" type="button" class="btn btn-secondary btn-sm" data-bs-dismiss="modal">
+                    <i class="fa-regular fa-hand"></i>
+                    {% translate "Cancel" %}
+                </button>
+
+                <button id="modal-button-confirm-kick-fleet-member" type="button" class="btn btn-success btn-sm">
+                    <i class="fa-solid fa-check"></i>
+                    {% translate "Confirm" %}
+                </button>
+            </div>
+        </div>
+    </div>
+</div>

fleetfinder/tests/test_views.py

@@ -101,14 +101,14 @@ class TestAjaxDashboardView(FleetfinderTestViews):
     It should also filter fleets based on the user's groups.
     """
 
-    # python
     @patch("fleetfinder.views.get_all_characters_from_user")
-    def test_renders_dashboard_with_fleet_data(self, mock_get_characters):
+    def test_renders_dashboard_with_fleet_data_with_basic_access(
+        self, mock_get_characters
+    ):
         """
-        Test that the ajax_dashboard view renders the dashboard with fleet data.
+        Test that the ajax_dashboard view renders the dashboard with fleet data
+        when the user has basic access permissions.
 
-        :param mock_portrait_url:
-        :type mock_portrait_url:
         :param mock_get_characters:
         :type mock_get_characters:
         :return:
@@ -124,9 +124,55 @@ class TestAjaxDashboardView(FleetfinderTestViews):
 
         self.client.force_login(self.user_with_basic_acces_perms)
         url = reverse("fleetfinder:ajax_dashboard")
+        join_url = reverse("fleetfinder:join_fleet", args=[self.fleet_id])
+        response = self.client.get(url)
+
+        expected_response = [
+            {
+                "fleet_commander": {
+                    "html": '<img class="rounded eve-character-portrait" src="https://images.evetech.net/characters/1000/portrait?size=32" alt="Jean Luc Picard" loading="lazy">Jean Luc Picard',
+                    "sort": "Jean Luc Picard",
+                },
+                "fleet_name": "Starfleet",
+                "created_at": dt_to_iso(self.fleet_created_at),
+                "actions": f'<a href="{join_url}" class="btn btn-sm btn-success ms-1" data-bs-tooltip="aa-fleetfinder" title="Join fleet"><i class="fa-solid fa-right-to-bracket"></i></a>',
+            }
+        ]
+
+        self.assertEqual(response.status_code, HTTPStatus.OK)
+        self.assertIn("Starfleet", response.json()[0]["fleet_name"])
+        self.assertIn("Jean Luc Picard", response.json()[0]["fleet_commander"]["html"])
+        self.assertEqual(response.json(), expected_response)
+
+    @patch("fleetfinder.views.get_all_characters_from_user")
+    def test_renders_dashboard_with_fleet_data_with_manage_access(
+        self, mock_get_characters
+    ):
+        """
+        Test that the ajax_dashboard view renders the dashboard with fleet data
+        when the user has manage access permissions.
+
+        :param mock_get_characters:
+        :type mock_get_characters:
+        :return:
+        :rtype:
+        """
+
+        mock_get_characters.return_value = [
+            self.user_with_manage_perms.profile.main_character
+        ]
+
+        fleet = self.fleet
+        fleet.groups.set([])
+
+        self.client.force_login(self.user_with_manage_perms)
+        url = reverse("fleetfinder:ajax_dashboard")
+        join_url = reverse("fleetfinder:join_fleet", args=[self.fleet_id])
+        details_url = reverse("fleetfinder:fleet_details", args=[self.fleet_id])
+        edit_url = reverse("fleetfinder:edit_fleet", args=[self.fleet_id])
         response = self.client.get(url)
 
-        expected_resonse = [
+        expected_response = [
             {
                 "fleet_commander": {
                     "html": '<img class="rounded eve-character-portrait" src="https://images.evetech.net/characters/1000/portrait?size=32" alt="Jean Luc Picard" loading="lazy">Jean Luc Picard',
@@ -134,16 +180,18 @@ class TestAjaxDashboardView(FleetfinderTestViews):
                 },
                 "fleet_name": "Starfleet",
                 "created_at": dt_to_iso(self.fleet_created_at),
-                "join": '<a href="/fleetfinder/fleet/12345/join/" class="btn btn-sm btn-primary">Join fleet</a>',
-                "details": "",
-                "edit": "",
+                "actions": (
+                    f'<a href="{join_url}" class="btn btn-sm btn-success ms-1" data-bs-tooltip="aa-fleetfinder" title="Join fleet"><i class="fa-solid fa-right-to-bracket"></i></a>'
+                    f'<a href="{details_url}" class="btn btn-sm btn-info ms-1" data-bs-tooltip="aa-fleetfinder" title="View fleet details"><i class="fa-solid fa-eye"></i></a>'
+                    f'<a href="{edit_url}" class="btn btn-sm btn-warning ms-1" data-bs-tooltip="aa-fleetfinder" title="Edit fleet advert"><i class="fa-solid fa-pen-to-square"></i></a>'
+                ),
             }
         ]
 
         self.assertEqual(response.status_code, HTTPStatus.OK)
         self.assertIn("Starfleet", response.json()[0]["fleet_name"])
         self.assertIn("Jean Luc Picard", response.json()[0]["fleet_commander"]["html"])
-        self.assertEqual(response.json(), expected_resonse)
+        self.assertEqual(response.json(), expected_response)
 
     @patch("fleetfinder.views.get_all_characters_from_user")
     def test_returns_empty_data_when_no_fleets_available(self, mock_get_characters):
@@ -171,6 +219,15 @@ class TestAjaxDashboardView(FleetfinderTestViews):
 
     @patch("fleetfinder.views.get_all_characters_from_user")
     def test_filters_fleets_by_user_groups(self, mock_get_characters):
+        """
+        Test that the ajax_dashboard view filters fleets based on the user's groups.
+
+        :param mock_get_characters:
+        :type mock_get_characters:
+        :return:
+        :rtype:
+        """
+
         mock_get_characters.return_value = [
             self.user_with_manage_perms.profile.main_character
         ]

fleetfinder/urls.py

@@ -34,6 +34,11 @@ urlpatterns = [
                     view=views.ajax_fleet_details,
                     name="ajax_fleet_details",
                 ),
+                path(
+                    route="fleet/<int:fleet_id>/member/kick/",
+                    view=views.ajax_fleet_kick_member,
+                    name="ajax_fleet_kick_member",
+                ),
             ]
         ),
     ),