Bug 477779

Summary: Lokalize cannot open ts files edited in Lokalize
Product: [Applications] lokalize Reporter: wojnilowicz <lukasz.wojnilowicz>
Component: generalAssignee: Simon Depiets <sdepiets>
Status: RESOLVED FIXED    
Severity: normal CC: aacid, shafff
Priority: NOR    
Version First Reported In: 23.08.3   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: https://invent.kde.org/sdk/lokalize/-/commit/a8e24987c2f3c5f922b905a7271ffd1e361f9e71 Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: The .ts file that I'm having problems with

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