Project

General

Profile

Fehler #495

Rechteprüfung Brief kaputt

Added by Bernd Bleßmann over 2 years ago.

Status:
Neu
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
04/04/2022
Due date:
% Done:

0%

Estimated time:

Description

Wenn man für die Briefbearbeitung nur die Rechte im Verkauf hat, kann man keine Briefe öffnen.

Die Prüfung hängt von der Form-Variablen "is_sales" ab, die aber nicht (immer) übergeben wird. So wird dann auf das Recht auf der Einkaufseite geprüft.

Ich hatte das mit commit 65604fea85234a5ae3e787f7cafd81ece6b8621d versucht zu reparieren, aber das war nicht weitreichend genug. Deshalb habe ich den commit reverted und dort was dazu in der commit message geschrieben:

This reverts commit 65604fea85234a5ae3e787f7cafd81ece6b8621d.

Der Fix ist zwar richtig, aber nicht weitgehend. Es fehlen an einigen
Stellen weiter Übergaben von "is_sales" (z.B. bei allen redirects).

Zudem gibt es noch ein anderes Problem. Die Rechteprüfung hängt von
der form-Variablen "is_sales", ab, die zwar auch beim Export durch
den Reportgenerator übergeben wird. Diese wird aber erst nach dem
run_before-hook von dfen Report-Generator-spezifischen Hiddens
in der form gesetzt.

Associated revisions

Revision a27846ef (diff)
Added by Bernd Bleßmann over 2 years ago

Revert "Briefe: Rechteprüfung gefixed"

This reverts commit 65604fea85234a5ae3e787f7cafd81ece6b8621d.

Der Fix ist zwar richtig, aber nicht weitgehend. Es fehlen an einigen
Stellen weiter Übergaben von "is_sales" (z.B. bei allen redirects).

Zudem gibt es noch ein anderes Problem. Die Rechteprüfung hängt von
der form-Variablen "is_sales", ab, die zwar auch beim Export durch
den Reportgenerator übergeben wird. Diese wird aber erst nach dem
run_before-hook von dfen Report-Generator-spezifischen Hiddens
in der form gesetzt.

Ich mache dazu ein Ticket auf: Refs #495 (redmine)

Also available in: Atom PDF