https://bugs.kde.org/show_bug.cgi?id=443845

--- Comment #9 from Jim Sanderson <jssalesm...@gmail.com> ---
Robby,

Thank you!  Look forward to the new version.

Jim

On Wed, Oct 20, 2021 at 7:58 PM Robby Stephenson <bugzilla_nore...@kde.org>
wrote:

> https://bugs.kde.org/show_bug.cgi?id=443845
>
> Robby Stephenson <ro...@periapsis.org> changed:
>
>            What    |Removed                     |Added
>
> ----------------------------------------------------------------------------
>    Version Fixed In|                            |3.4.2
>       Latest Commit|                            |
> https://invent.kde.org/offi
>                    |
> |ce/tellico/commit/3824db1ba
>                    |
> |a75a2b6fdb1806198bdf7582ea4
>                    |                            |bf2c
>          Resolution|---                         |FIXED
>              Status|REPORTED                    |RESOLVED
>
> --- Comment #8 from Robby Stephenson <ro...@periapsis.org> ---
> Git commit 3824db1baa75a2b6fdb1806198bdf7582ea4bf2c by Robby Stephenson.
> Committed on 21/10/2021 at 02:54.
> Pushed by rstephenson into branch '3.4'.
>
> Avoid saving invalid XML characters while stripping them during loading
>
> Using QXmlDefaultHandler in Tellico 3.3 silently allowed loading of
> invalid XML characters, while the switch to QXmlStreamReader in 3.4
> resulted in a data file failing to load. In the event of a parsing
> error, strip invalid characters and attempt to reload.
>
> In addition, change the DOM implementation policy to NOT save the
> invalid characters in the fil to begin with.
> FIXED-IN: 3.4.2
>
> A  +14   -0    src/tests/data/bug443845.xml
> M  +26   -0    src/tests/tellicoreadtest.cpp
> M  +1    -0    src/tests/tellicoreadtest.h
> M  +15   -0    src/translators/tellico_xml.cpp
> M  +1    -0    src/translators/tellico_xml.h
> M  +5    -1    src/translators/tellicoimporter.cpp
> M  +5    -0    src/translators/tellicoxmlexporter.cpp
>
>
> https://invent.kde.org/office/tellico/commit/3824db1baa75a2b6fdb1806198bdf7582ea4bf2c
>
> --
> You are receiving this mail because:
> You reported the bug.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to