Skip to content

Conversation

@ibtyog
Copy link
Member

@ibtyog ibtyog commented Dec 24, 2025

  • dodałem walidacje cross-event w metodzie update w formsach, która sprawdza czy atrybut należy do danego eventu.
  • zdecydowałem się zamienić kolejność działania w metodzie, atrybuty są sprawdzane i aktualizowane przed nadpisaniem samego eventu, żeby nie było sytuacji gdzie sam event się zaktualizuje a zwróci błąd związany z atrybutami

Important

Adds cross-event validation and reorders operations in update() in forms_controller.ts to ensure attributes belong to the event before updating forms.

  • Validation:
    • Adds cross-event validation in update() in forms_controller.ts to ensure attributes belong to the event before updating.
  • Operation Order:
    • Reorders update() in forms_controller.ts to validate and update attributes before saving the form to prevent partial updates.

This description was created by Ellipsis for 1e3012b. You can customize this summary. It will automatically update as commits are pushed.

@ibtyog ibtyog requested a review from loudsheep December 24, 2025 12:22
@ibtyog ibtyog self-assigned this Dec 24, 2025
@ibtyog ibtyog linked an issue Dec 24, 2025 that may be closed by this pull request
Copy link
Member

@loudsheep loudsheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super, wygląda git. Przy okazji czy mógłbyś może jeszcze tylko zmienić też tą kolejność zapisu w metodzie store? Widzę że tam też formularz jest najpierw tworzony a potem są sprawdzane błędy atrybutów

@ibtyog ibtyog requested a review from loudsheep December 29, 2025 15:44
Copy link
Member

@loudsheep loudsheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mega, wszystko fajnie. Wgl mega fire że zobaczyłeś że trzeba zmienić tą kolejność zapisu

@loudsheep loudsheep merged commit f2b9145 into main Jan 11, 2026
5 checks passed
@loudsheep loudsheep deleted the fix/forms_controller branch January 11, 2026 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Naprawa FormsController

3 participants