Medienordner: Fehler beim Löschen von Bildern

Zitat von Stef

Noch einmal versucht, jetzt habe ich die Fehlermeldung auch erhalten. Mit der SHIFT-Taste.
Mit 5 Bildern, und es kamen aber auch grüne meldungen.

Aber die Bilder werden ja gelöscht, ist die Hauptsache.

Ja klar werden die gelöscht aber ich finde es trotzdem nicht in Ordnung.

Zitat von -OJ-

Rolf Dautrich bei Github bin ich raus. Ich finde es einfach nur unübersichtlich.

Ich kann es aktuell nicht machen. Gestern hatte ich noch keine abschließende Bewertung. Bei Joomla /Github muss ich alles einzeln aufbröseln statt ein Textfeld mit den abgeteilten Infos zu kopieren. Das ist mir gerade zu aufwendig und ich Poste es mal hier in einen Thread damit ich es die Tage noch bei Github als Issue einstelle.

Zitat

Fix duplicate file selection when using SHIFT in Media Manager

Summary

When selecting multiple files in the Joomla Media Manager using SHIFT, the first selected item is added twice to the internal selectedItems array.

For example:

Code

First selected item: A

SHIFT selection: A, B, C, D, E

Internal selectedItems: A, A, B, C, D, E

When deleting the selection, Joomla therefore sends two DELETE requests for the first file.

• The first request successfully deletes the file and returns HTTP 200.
• The second request tries to delete the already deleted file and returns HTTP 404.

This results in misleading error messages such as:

• "Error deleting file"
• "Delete not possible!"
• "File or directory not found"

Although all selected files are successfully deleted.

Reproduction

1. Open the Joomla Media Manager.
2. Select one file.
3. Hold SHIFT and select another file further down.
4. Delete the selected files.

The issue does not occur when selecting files individually using CTRL.

Cause

The issue is located in:

administrator/components/com_media/resources/scripts/components/browser/utils/utils.es6.js

The SHIFT range selection currently uses:

Code

slice(currentIndex, endIndex + 1)

This includes the already selected starting item again.

The range should exclude the already selected starting item:

Code

slice(currentIndex + 1, endIndex + 1)

Additional safeguard

As an additional safeguard, selectedItems should prevent the same file path from being added more than once:

Code

[types.SELECT_BROWSER_ITEM]: (state, payload) => {
    const alreadySelected = state.selectedItems.some(
        (item) => item.path === payload.path
    );

    if (!alreadySelected) {
        state.selectedItems.push(payload);
    }
},

The deletion routine could also deduplicate selected items by file path before sending DELETE requests.

Tested with

• Joomla 5.4.8
• Joomla 6.1.3
• Firefox

The issue is reproducible in both Joomla versions.

Expected behaviour

Each selected file should occur only once in selectedItems and exactly one DELETE request should be sent for each file.

Alles anzeigen

Hi, wie schon geschrieben, habe ich einen PR noch nie erstellt. Ich melde meist einen Issue bei Github. Wenn Rolf Dautrich das übernehmen würde (hallo Rolf) wäre es prima. Ansonsten melde ich das bei Github als Issue wenn ich wieder von meiner Reise zurück bin.