Skip to content

Comments

KAAV-3557 Raahaus + lausunnot viimeistään (toteutus)#678

Open
henrihaapalasiili wants to merge 3 commits intodevelopmentfrom
KAAV-3557
Open

KAAV-3557 Raahaus + lausunnot viimeistään (toteutus)#678
henrihaapalasiili wants to merge 3 commits intodevelopmentfrom
KAAV-3557

Conversation

@henrihaapalasiili
Copy link
Contributor

@henrihaapalasiili henrihaapalasiili commented Feb 23, 2026

Lausunnot viimeistään syncs with nähtävilläolo päättyy and allows admin override

  • compareAndUpdateDates accepts optional previousPaattyyValues snapshot
    to detect when päättyy actually changed vs unchanged
  • When päättyy changes (any direction): force lausunnot = new päättyy
  • When päättyy unchanged: preserve user-set lausunnot >= päättyy (floor constraint)
  • Reducer captures pre-cascade päättyy snapshot before date adjustments
  • Remove Lausunnot viimeistään exclusion from extractAttributes
  • Add parameterized rule-based tests with mutation-verified coverage

- Fix NaN crash in objectUtil date cascade when initial_distance is null
  (alkaaGap, paattyyGap, lautakuntaGap fallback chains with ??-defaults)
- Remove isLargeProject branch; all sizes use milloin_ehdotuksen_nahtavilla_paattyy
- Add periaatteet/luonnos fallback to esillaolo_paattyy when lautakunta disabled
- Fix field name kaynnistysvaihe_paattyy_pvm → kaynnistys_paattyy_pvm (K2 spec)
- Enable 5 skipped tests, fix test typos and assertion order
…allows admin override

- compareAndUpdateDates accepts optional previousPaattyyValues snapshot
  to detect when päättyy actually changed vs unchanged
- When päättyy changes (any direction): force lausunnot = new päättyy
- When päättyy unchanged: preserve user-set lausunnot >= päättyy (floor constraint)
- Reducer captures pre-cascade päättyy snapshot before date adjustments
- Remove Lausunnot viimeistään exclusion from extractAttributes
- Add parameterized rule-based tests with mutation-verified coverage
@sonarqubecloud
Copy link

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.

1 participant