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

Hotfix/history #302

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Hotfix/history #302

wants to merge 13 commits into from

Conversation

melegiul
Copy link
Collaborator

fix #300
fix #301

Form was included twice resulting in a rendering error.
@melegiul melegiul requested review from holema and holzi1005 January 17, 2025 14:06
@holzi1005
Copy link
Collaborator

@melegiul
Ich habe gerade die Datenschutzelemente durchgearbeitet und gesehen, dass z.b. bei Datenweitergaben jetzt alle Historien in der Übersicht angezeigt werden. Es sollte aber nur die aktuelle Version in der Übersicht angezeigt werden.

image

@melegiul
Copy link
Collaborator Author

@melegiul Ich habe gerade die Datenschutzelemente durchgearbeitet und gesehen, dass z.b. bei Datenweitergaben jetzt alle Historien in der Übersicht angezeigt werden. Es sollte aber nur die aktuelle Version in der Übersicht angezeigt werden.

image

Ja, das Problem gab es häufiger nach dem Fix. Hier habe ich es übersehen. Danke. Ist gefixt in eb9ef3e

@holzi1005
Copy link
Collaborator

Dies hat funktioniert.
Aber ich habe erst jetzt gemerkt, dass dies uach bei den Auftragsverarbeitung der Fall ist:
https://demo.open-datenschutzcenter.de/auftragsverarbeitung. Kannst du da auch nochmal schauen.

@holzi1005
Copy link
Collaborator

Ich habe gerade gemerkt, dass ich in den Vorfällen auch die Historie bearbeiten kann: https://demo.open-datenschutzcenter.de/vorfall

@melegiul
Copy link
Collaborator Author

Auftragsverarbeitung ist gefixt. Vorfall-Historie war tatsächlich allerdings nicht bearbeitbar:

if ($form->isSubmitted() && $form->isValid() && $vorgang->getActiv() === true) {

Früher gab es gar keine deaktivierten Formulare, sondern es wurde beim Speichern lediglich auf aktiv geprüft. Heißt, man konnte das Formular zwar absenden, aber die Änderungen wurden nicht persistiert. Was bei Vorfall immer noch so implementiert war.

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

Successfully merging this pull request may close these issues.

Historien können nicht abgezeigt werden Historien lassen sich bearbeiten
2 participants