S3Panel en WebdavPanel consolideren (~120-180 regels overlap) #2072

New issue

Closed

opened 2026-09-13 08:05:34 +00:00 by brenno · 0 comments

brenno commented

2026-09-13 08:05:34 +00:00

Owner

Probleem

lib/widgets/dialogs/settings/s3_panel.dart en lib/widgets/dialogs/settings/webdav_panel.dart volgen nagenoeg dezelfde flow met veel identieke code.

Identieke / vrijwel identieke delen

Onderdeel	S3Panel	WebdavPanel
Widget constructor/props	regels 20-42	regels 19-40
`_update()` helper	regels 49-52	regels 45-48
`_addressingField`/`_kindField` (DropdownButtonFormField met reset van teststate)	regels 56-95	regels 52-86
`build()`: Column, SettingsSectionTitle, introtekst, text fields, SwitchListTile, `_testSection`, footertekst	regels 97-173	regels 88-172
`_testSection()` (testknop, spinner, succes-row, certificaat-button, error-row)	regels 177-248	regels 244-315
`_testConnection()` (validatie, setState, probe, exception-afhandeling)	regels 250-287	regels 317-354
`_trustCertificate()` (toon certificaat, pin, retest)	regels 290-307	regels 362-381

Verschillen

Het specifieke formulier (S3Form vs WebdavForm), labels, hints, de enum voor foutafhandeling (S3Error vs WebdavError), en enkele veldspecifieke details.

Voorstel

Trek de gemeenschappelijke _testSection, _testConnection en _trustCertificate plus het gedeelde build()-skeleton naar een gedeelde base of helper voor online-opslagpanels.

Schatting

~120-180 regels die in een gedeelde base of helper zouden kunnen zitten. Per panel zouden 50-80 regels wegvallen.

Labels

duplicate, enhancement

## Probleem `lib/widgets/dialogs/settings/s3_panel.dart` en `lib/widgets/dialogs/settings/webdav_panel.dart` volgen nagenoeg dezelfde flow met veel identieke code. ## Identieke / vrijwel identieke delen | Onderdeel | S3Panel | WebdavPanel | |---|---|---| | Widget constructor/props | regels 20-42 | regels 19-40 | | `_update()` helper | regels 49-52 | regels 45-48 | | `_addressingField`/`_kindField` (DropdownButtonFormField met reset van teststate) | regels 56-95 | regels 52-86 | | `build()`: Column, SettingsSectionTitle, introtekst, text fields, SwitchListTile, `_testSection`, footertekst | regels 97-173 | regels 88-172 | | `_testSection()` (testknop, spinner, succes-row, certificaat-button, error-row) | regels 177-248 | regels 244-315 | | `_testConnection()` (validatie, setState, probe, exception-afhandeling) | regels 250-287 | regels 317-354 | | `_trustCertificate()` (toon certificaat, pin, retest) | regels 290-307 | regels 362-381 | ## Verschillen Het specifieke formulier (`S3Form` vs `WebdavForm`), labels, hints, de enum voor foutafhandeling (`S3Error` vs `WebdavError`), en enkele veldspecifieke details. ## Voorstel Trek de gemeenschappelijke `_testSection`, `_testConnection` en `_trustCertificate` plus het gedeelde `build()`-skeleton naar een gedeelde base of helper voor online-opslagpanels. ## Schatting ~120-180 regels die in een gedeelde base of helper zouden kunnen zitten. Per panel zouden 50-80 regels wegvallen. ## Labels duplicate, enhancement

brenno added the

enhancement

duplicate

labels

2026-09-13 08:05:34 +00:00

brenno referenced this issue from a commit

2026-09-13 10:06:00 +00:00

refactor: extraheer ConnectionTestSection uit S3Panel en WebdavPanel (#2072)

brenno referenced this issue

2026-09-13 10:06:07 +00:00

refactor: dupliceer-codes opruimen (13 issues) #2084

brenno referenced this issue from a commit