_unescapeHtml en decodeNamedHtmlEntities samenvoegen (identieke logica) #2078

New issue

Closed

opened 2026-09-13 08:06:16 +00:00 by brenno · 0 comments

brenno commented

2026-09-13 08:06:16 +00:00

Owner

Probleem

Twee functies in verschillende lagen implementeren exact dezelfde HTML-entity decode-logica:

lib/utils/inline_markdown.dart:149 -- decodeNamedHtmlEntities(String text) (publiek)
lib/services/markdown_service_helpers.dart:249 -- _unescapeHtml(String value) (privaat, part of markdown_service.dart)

Beide decoderen de named HTML-entities (lt, gt, quot, amp) in dezelfde volgorde (amp gaat als laatste om double-decode te voorkomen). De commentaar bij decodeNamedHtmlEntities verwijst expliciet naar _unescapeHtml als de spiegel.

Callers

decodeNamedHtmlEntities wordt aangeroepen vanuit:

widgets/slides/previews/timeline_preview.dart (3x)
widgets/slides/previews/chart_preview.dart (4x)
utils/inline_markdown.dart zelf (1x)

_unescapeHtml wordt aangeroepen vanuit:

services/markdown_parse/markdown_service_parse_columns.dart (2x)

Voorstel

Vervang _unescapeHtml door een aanroep naar decodeNamedHtmlEntities. De functie is al publiek in utils/inline_markdown.dart.

Schatting

~6 regels verwijderbaar.

Labels

duplicate, enhancement

## Probleem Twee functies in verschillende lagen implementeren exact dezelfde HTML-entity decode-logica: 1. `lib/utils/inline_markdown.dart:149` -- `decodeNamedHtmlEntities(String text)` (publiek) 2. `lib/services/markdown_service_helpers.dart:249` -- `_unescapeHtml(String value)` (privaat, part of markdown_service.dart) Beide decoderen de named HTML-entities (lt, gt, quot, amp) in dezelfde volgorde (amp gaat als laatste om double-decode te voorkomen). De commentaar bij `decodeNamedHtmlEntities` verwijst expliciet naar `_unescapeHtml` als de spiegel. ## Callers `decodeNamedHtmlEntities` wordt aangeroepen vanuit: - `widgets/slides/previews/timeline_preview.dart` (3x) - `widgets/slides/previews/chart_preview.dart` (4x) - `utils/inline_markdown.dart` zelf (1x) `_unescapeHtml` wordt aangeroepen vanuit: - `services/markdown_parse/markdown_service_parse_columns.dart` (2x) ## Voorstel Vervang `_unescapeHtml` door een aanroep naar `decodeNamedHtmlEntities`. De functie is al publiek in `utils/inline_markdown.dart`. ## Schatting ~6 regels verwijderbaar. ## Labels duplicate, enhancement

brenno added the

enhancement

duplicate

labels

2026-09-13 08:06:16 +00:00

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

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

refactor: vervang _unescapeHtml door publieke decodeNamedHtmlEntities (#2078)

brenno closed this issue