Bug 477779 - Lokalize cannot open ts files edited in Lokalize
Summary: Lokalize cannot open ts files edited in Lokalize
Status: RESOLVED FIXED
Alias: None
Product: lokalize
Classification: Applications
Component: general (show other bugs)
Version: 23.08.3
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Simon Depiets
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-11-30 16:59 UTC by wojnilowicz
Modified: 2023-12-14 22:12 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
The .ts file that I'm having problems with (388.51 KB, video/mpeg)
2023-12-07 15:23 UTC, wojnilowicz
Details

Note You need to log in before you can comment on or make changes to this bug.
Description wojnilowicz 2023-11-30 16:59:20 UTC
SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1. open ts file
2. translate an entry
3. save
4. close ts file
5. try to reopen ts file

OBSERVED RESULT
Error.

EXPECTED RESULT
No error.

SOFTWARE/OS VERSIONS
Operating System: Fedora Linux 39
KDE Plasma Version: 5.27.9
KDE Frameworks Version: 5.111.0
Qt Version: 5.15.11
Graphics Platform: X11

ADDITIONAL INFORMATION
After opening a ts file, its translations counts isn't displayed correctly.
Comment 1 Albert Astals Cid 2023-12-07 15:14:48 UTC
I've just tried this and I can open it after editing it and saving it in lokalize just fine.

Please attach the .ts file you're having problems with (before editing it)
Comment 2 wojnilowicz 2023-12-07 15:23:38 UTC
Created attachment 163982 [details]
The .ts file that I'm having problems with

Opening after editing gives me an error like this
"Error opening the file /home/$USER/Desktop/sqlb_pl.ts, line: 7 699"
Comment 3 Bug Janitor Service 2023-12-07 16:02:40 UTC
A possibly relevant merge request was started @ https://invent.kde.org/sdk/lokalize/-/merge_requests/81
Comment 4 Albert Astals Cid 2023-12-13 23:57:38 UTC
Git commit 28955c1609200435ba94d31fd3289167e02602e6 by Albert Astals Cid.
Committed on 14/12/2023 at 00:52.
Pushed by aacid into branch 'master'.

Don't break ts files with obsolete fields on saving them

M  +1    -0    src/catalog/ts/tsstorage.cpp

https://invent.kde.org/sdk/lokalize/-/commit/28955c1609200435ba94d31fd3289167e02602e6
Comment 5 Albert Astals Cid 2023-12-14 22:12:47 UTC
Git commit a8e24987c2f3c5f922b905a7271ffd1e361f9e71 by Albert Astals Cid.
Committed on 14/12/2023 at 23:12.
Pushed by aacid into branch 'release/23.08'.

Don't break ts files with obsolete fields on saving them
(cherry picked from commit 28955c1609200435ba94d31fd3289167e02602e6)

M  +1    -0    src/catalog/ts/tsstorage.cpp

https://invent.kde.org/sdk/lokalize/-/commit/a8e24987c2f3c5f922b905a7271ffd1e361f9e71