fix(prezentacja): podsumowanie bierze wszystkie policzone opcje (stacje, tabele, domy) #46

Merged

gitea merged 1 commits from fix/compile-carry-options into master 2026-07-30 13:44:35 +00:00

Conversation 0 Commits 1 Files Changed 7

+371 -55

gitea commented 2026-07-30 12:02:04 +00:00

Owner

Copy Link

Copy Source

Zgłoszona regresja: gdy horoskop liczony jest z dodatkowymi opcjami
(porównanie domów, stacje, żywioły/tabele), podsumowanie ich nie zawierało —
„Skompiluj" przeliczało horoskop od nowa z okrojonym zestawem (tylko system domów

• zodiak) zamiast pokazać to, co policzono.

Podejście

Zostaje czysty przelicz (bez trzymania dużego wyniku w przeglądarce), ale z tymi
samymi opcjami co przy horoskopie:

• formsync synchronizuje między zakładkami także opcje — checkboxy
  (stacje, tabele) i wielo-checkbox (porównanie domów). Wcześniej umiał tylko
  .value.
• „Skompiluj" dostaje te same pola opcji; compile_build i compile_pdf
  przekazują je do logiki (stations/tables/house_systems), a compile.js
  wysyła je w payloadzie PDF-a.
• Wspólny plik _result_tables.html: podsumowanie renderuje dokładnie te
  same tabele co „Horoskop" (porównanie domów, stacje, aspekty, paralele,
  antyscja, żywioły/faza/godziny, Lots…). Sekcja pokazuje się tylko gdy jej dane
  są w wyniku — opcja niepoliczona → sekcji nie ma (zgodnie z prośbą).

Uczciwie: chart.html na razie NIE korzysta jeszcze z tego include (to była
operacja na ~240 liniach szablonu — odłożona, żeby nie ryzykować widoku
głównego). Jest komentarz-TODO i zgodność pilnowana ręcznie. Domknę osobno.

Weryfikacja

Render wspólnego pliku na bogatym wyniku pokazuje wszystkie sekcje (domy/stacje/
aspekty/paralele/antyscja/żywioły/Lots). Testy: +8, prezentacja 202.

Zgłoszona regresja: gdy horoskop liczony jest z **dodatkowymi opcjami** (porównanie domów, stacje, żywioły/tabele), **podsumowanie ich nie zawierało** — „Skompiluj" przeliczało horoskop od nowa z okrojonym zestawem (tylko system domów + zodiak) zamiast pokazać to, co policzono. ## Podejście Zostaje czysty przelicz (bez trzymania dużego wyniku w przeglądarce), ale **z tymi samymi opcjami** co przy horoskopie: - **`formsync`** synchronizuje między zakładkami także opcje — **checkboxy** (stacje, tabele) i **wielo-checkbox** (porównanie domów). Wcześniej umiał tylko `.value`. - **„Skompiluj"** dostaje te same pola opcji; `compile_build` i `compile_pdf` przekazują je do logiki (`stations`/`tables`/`house_systems`), a `compile.js` wysyła je w payloadzie PDF-a. - Wspólny plik **`_result_tables.html`**: podsumowanie renderuje **dokładnie te same tabele** co „Horoskop" (porównanie domów, stacje, aspekty, paralele, antyscja, żywioły/faza/godziny, Lots…). Sekcja pokazuje się **tylko gdy jej dane są w wyniku** — opcja niepoliczona → sekcji nie ma (zgodnie z prośbą). > Uczciwie: `chart.html` na razie NIE korzysta jeszcze z tego include (to była > operacja na ~240 liniach szablonu — odłożona, żeby nie ryzykować widoku > głównego). Jest komentarz-TODO i zgodność pilnowana ręcznie. Domknę osobno. ## Weryfikacja Render wspólnego pliku na bogatym wyniku pokazuje wszystkie sekcje (domy/stacje/ aspekty/paralele/antyscja/żywioły/Lots). Testy: **+8**, prezentacja **202**.

gitea added 1 commit 2026-07-30 12:02:04 +00:00

fix(prezentacja): podsumowanie bierze WSZYSTKIE policzone opcje (stacje, tabele, domy) ... 52b7c20c2a

Regresja: „Skompiluj" przeliczało horoskop od nowa z OKROJONYM zestawem opcji
(tylko system domów + zodiak), więc jeśli przy horoskopie policzyłeś stacje,
tabele (żywioły, faza Księżyca…) albo porównanie domów — w podsumowaniu ich NIE
było. Zamiast pokazać to, co policzono, liczyło uboższą wersję.

Zostaje czysty przelicz (bez trzymania dużego wyniku w przeglądarce), ale z TYMI
SAMYMI opcjami co przy horoskopie:
- `formsync`: synchronizuje między zakładkami też opcje — checkboxy (stacje,
  tabele) i wielo-checkbox (porównanie domów). Wcześniej umiał tylko `.value`.
- „Skompiluj" (formularz): dostaje te same opcje; `compile_build` i `compile_pdf`
  przekazują je do logiki (stations/tables/house_systems). `compile.js` wysyła je
  w payloadzie PDF-a.
- Wspólny plik `_result_tables.html`: podsumowanie renderuje DOKŁADNIE te same
  tabele co „Horoskop" (porównanie domów, stacje, aspekty, paralele, antyscja,
  żywioły/faza/godziny, Lots…). Każda sekcja pokazuje się tylko, gdy jej dane są
  w wyniku — opcja niepoliczona → sekcji nie ma (zgodnie z prośbą).
  (TODO: przełączyć też chart.html na ten include, by widoki nie mogły się
  rozjechać — na razie zgodność pilnowana ręcznie, jest komentarz w pliku.)

Weryfikacja: render wspólnego pliku na bogatym wyniku pokazuje wszystkie sekcje.
Testy: +8 (opcje niesione w /compile i /compile/pdf, sync checkboxów/multi,
wspólny include). Prezentacja 202.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

gitea merged commit 52b7c20c2a into master 2026-07-30 13:44:35 +00:00

gitea deleted branch fix/compile-carry-options 2026-07-30 13:44:36 +00:00

gitea referenced this issue from a commit 2026-08-03 11:16:33 +00:00

feat: konfigurowalne aspekty i orby (PRE-06)

gitea referenced this pull request 2026-08-03 11:16:50 +00:00

feat: konfigurowalne aspekty i orby (PRE-06) #49

gitea referenced this issue from a commit 2026-08-03 16:09:57 +00:00

refactor(prezentacja): wspólne partiale formularza — koniec dublowania chart/compile

gitea referenced this pull request 2026-08-03 16:10:13 +00:00

refactor(prezentacja): wspólne partiale formularza — koniec dublowania chart/compile #50

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

gitea

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: gitea/astrololo#46