Rechte für Inventar und Ausgabe trennen, Scan-Ziel korrigieren #12

Open

thiel wants to merge 1 commit from fix/rechte-scan into fix/bestandsjournal

thiel commented

2026-10-01 12:22:45 +00:00

Owner

Behebt Befund 7 des externen Checks.

Die gesamte Geräteressource hing an Inventar verwalten. Damit konnte ausgeben, wer nur verwalten sollte — und wer nur ausgeben sollte, kam gar nicht an die Geräteliste; der gedruckte QR-Code führte ihn auf die Gerätebearbeitung und damit ins 403.

Ansehen (Liste + Ansicht): eines der beiden Rechte genügt.
Ausgeben, Zurücknehmen, Reservieren (Einzel- und Sammelaktionen): Ausgabe verwalten.
Anlegen, Bearbeiten, Löschen, Import, Export, Bestandsliste, Etiketten: Inventar verwalten.

Edit- und Delete-Aktion in der Tabelle prüfen ihr Recht jetzt selbst — sie blieben sonst trotz fehlender Berechtigung sichtbar (die Seite dahinter hätte geblockt, die Aktion war aber da).

Das Scan-Ziel öffnet die Geräteansicht statt der Bearbeitung; Ausgabe und Rücknahme sind von dort aus erreichbar. Der bestehende Test wurde entsprechend angepasst.

6 neue Tests. Volle Suite grün (191 Tests), Pint sauber.

Setzt auf #11 auf — Reihenfolge #8, #9, #10, #11, dieser.

🤖 Generated with Claude Code

https://claude.ai/code/session_01QKiq6LV8Lk1vst5CQW15ar

Behebt Befund 7 des externen Checks. Die gesamte Geräteressource hing an `Inventar verwalten`. Damit konnte ausgeben, wer nur verwalten sollte — und wer nur ausgeben sollte, kam gar nicht an die Geräteliste; der gedruckte QR-Code führte ihn auf die Gerätebearbeitung und damit ins 403. - **Ansehen** (Liste + Ansicht): eines der beiden Rechte genügt. - **Ausgeben, Zurücknehmen, Reservieren** (Einzel- und Sammelaktionen): `Ausgabe verwalten`. - **Anlegen, Bearbeiten, Löschen, Import, Export, Bestandsliste, Etiketten**: `Inventar verwalten`. Edit- und Delete-Aktion in der Tabelle prüfen ihr Recht jetzt selbst — sie blieben sonst trotz fehlender Berechtigung sichtbar (die Seite dahinter hätte geblockt, die Aktion war aber da). Das Scan-Ziel öffnet die Geräteansicht statt der Bearbeitung; Ausgabe und Rücknahme sind von dort aus erreichbar. Der bestehende Test wurde entsprechend angepasst. 6 neue Tests. Volle Suite grün (191 Tests), Pint sauber. **Setzt auf #11 auf** — Reihenfolge #8, #9, #10, #11, dieser. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01QKiq6LV8Lk1vst5CQW15ar

thiel added 1 commit

2026-10-01 12:22:45 +00:00

Rechte für Inventar und Ausgabe trennen, Scan-Ziel korrigieren

Tests / phpunit (pull_request) Successful in 1m16s

Details

527065fc16

Die gesamte Geräteressource hing an "Inventar verwalten". Damit konnte
ausgeben, wer nur verwalten sollte, und wer nur ausgeben sollte, kam gar
nicht an die Liste - der gedruckte QR-Code führte ihn auf die
Gerätebearbeitung und damit ins 403.

Ansehen darf jetzt, wer eines der beiden Rechte hat. Ausgeben,
Zurücknehmen und Reservieren hängen an "Ausgabe verwalten", Anlegen,
Bearbeiten, Löschen, Import, Export und Etiketten an "Inventar
verwalten". Edit- und Delete-Aktion in der Liste prüfen ihr Recht jetzt
selbst, sonst blieben sie trotz fehlender Berechtigung sichtbar.

Das Scan-Ziel öffnet die Geräteansicht statt der Bearbeitung; Ausgabe und
Rücknahme sind von dort aus erreichbar.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QKiq6LV8Lk1vst5CQW15ar

thiel referenced this pull request

2026-10-01 12:26:00 +00:00

Ausleihen verlängern und Rückgabe mit Zustandsprüfung #13

Tests / phpunit (pull_request) Successful in 1m16s

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin fix/rechte-scan:fix/rechte-scan

git switch fix/rechte-scan

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.