Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

POST API Fehler mit bislang ungenutzten Externen Einsatzmitteln #372

Open
TutePlays opened this issue Dec 31, 2024 · 2 comments
Open

POST API Fehler mit bislang ungenutzten Externen Einsatzmitteln #372

TutePlays opened this issue Dec 31, 2024 · 2 comments

Comments

@TutePlays
Copy link

Hallo,
zuerst einmal möchte ich mich herzlich für dieses tolle Plugin bedanken, welches bei unserer Feuerwehr seit 2018 auf der Homepage im Einsatz ist.

Die POST API zum Erstellen eines neuen Einsatzberichtes nimmt ja auch die resources entgegen. Allerdings werden externe Einsatzmittel nur übernommen, wenn diese bereits zu mindestens einem Einsatz zugeordnet sind. Wenn sich der einzige Einsatz mit einer Zuordnung im Papierkorb befindet, funktioniert die Neuzuweisung per API auch noch. Lösche ich den Einsatz aus dem Papierkorb und habe somit ein externes Einsatzmittel ohne Zuordnung, kann ich es nicht über die POST API zuwesisen.

Ob der Fehler auch mit eigenen Einsatzmitteln auftritt, habe ich noch nicht getestet.

Da wir die Anzeige der externen Einsatzmittel etwas umgebaut haben, haben wir nun sehr viele von diesen ohne Zuordnung, die sich dann über die API leider nicht mehr nutzen lassen. Als Workaround habe ich jetzt einen Dummy Einsatz mit allen externen Einsatzmitteln im Papierkorb, damit die Funktion sichergestellt ist.

Ich freue mich über eine Rückmeldung/einen Fix und wünsche einen guten Rutsch!

@abrain
Copy link
Owner

abrain commented Feb 5, 2025

Hallo, vielen Dank für den Fehlerbericht.

Ich habe versucht, das Problem nachzustellen, war aber auf Anhieb nicht erfolgreich.

Auch im Code konnte ich erst mal kein Problem sehen. Nicht verwendete Fahrzeuge und Externe Einsatzmittel werden explizit in die Suche eingebunden.

Werden die Einsatzmittel über den Namen referenziert oder über einen alternativen Bezeichner?

Wie viele Externe Einsatzmittel habt ihr ungefähr hinterlegt?

Wenn bei euch der Fehler auftritt, schlägt der API-Aufruf irgendwie fehl oder ist sonst irgendwie eine Fehlermeldung / Warnung zu beobachten? Oder werden einfach die betreffenden Einsatzmittel kommentarlos ignoriert?

@TutePlays
Copy link
Author

Hi, danke für deine Rückmeldung.

Also ich referenziere über den Namen, dabei spielt es auch keine Rolle, ob ein Leerzeichen vorhanden ist oder nicht.
Wir haben fast 200 externe Einsatzmittel. Das ist gewachsen, da wir früher jedes externe Fahrzeug explizit hinterlegt haben. Mittlerweile nutzen wir nur noch die externen Organisationen wie "Feuerwehr XY". Falls die Anzahl die Ursache sein könnte, kann ich ich alle löschen, die bisher bei keinen Einsätzen genutzt wurden.

Anbei die Payload:

{
"reason":"Test",
"date_start":"2025-02-06T00:00:00+00:00",
"keyword":"Sicherheitswachdienst",
"resources":"Gesamtwehr,NA 10,Test Leerzeichen,Test",
"publish":true
}

Test & Test Leerzeichen sind frisch angelegt als externe Einsatzmittel. Gesamtwehr & NA 10 sind eigene Einheiten bzw. Fahrzeuge die bereits genutzt wurden.

Das hier kommt an:
Image

Die API gibt einen "201 Created" zurück, scheint also keinen Fehler zu erzeugen. Die betroffenen Einsatzmittel werden kommentarlos ignoriert.
Wenn ich "Test" und "Test Leerzeichen" jetzt über das Backend einem Einsatz zuweise und die API erneut ausführe, dann werden sie korrekt zugewiesen.

Selbe Payload führt dann zu:
Image

Wir nutzen dein Plugin in der neusten Version 1.12.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants