Bug 423001 - Re-opening folders does not work correctly
Summary: Re-opening folders does not work correctly
Status: RESOLVED FIXED
Alias: None
Product: kdiff3
Classification: Applications
Component: application (other bugs)
Version First Reported In: 1.8.x
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: michael
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-06-15 07:30 UTC by Gert Lynge
Modified: 2020-06-19 17:39 UTC (History)
0 users

See Also:
Latest Commit: https://commits.kde.org/kdiff3/commit/bdc5c73927a58a2765d33c49350b50e240d2898d
Version Fixed/Implemented In: 1.8.3
Sentry Crash Report:

Attachments
Zipped png-pictures (30.59 KB, application/x-zip-compressed)
2020-06-15 07:30 UTC, Gert Lynge 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Gert Lynge 2020-06-15 07:30:02 UTC 
Created attachment 129375 [details]
Zipped png-pictures

SUMMARY
When you do a folder merge and want to re-open (i.e. restart the merge), the dialog is pre-filled with illegal values.

STEPS TO REPRODUCE
1. Select folders as shown in the Open.png in attached zip
2. Wait for the file merge to complete
3. Click the open icon again. The directory names in Re-open.png in attached zip will be shown
4. Click OK. The error in Error.ng in attached zip will be shown

OBSERVED RESULT
Wrong directory names shown on re-open. A click on Ok throws an error due to these wrong directory names

EXPECTED RESULT
The selected directories should be shown so you can restart the merge process by just clicking ok without getting error.

SOFTWARE/OS VERSIONS
Windows: 10 Pro 1909. Build 18363.900
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Keep up the good work :-)
Comment 1 michael 2020-06-19 17:34:23 UTC 
Yep fixed in https://invent.kde.org/sdk/kdiff3/-/commit/bdc5c73927a58a2765d33c49350b50e240d2898d. The issue is that for some reason on Windows and only Windows Qt likes to encode file:// urls for display. Not what kdiff3 wants. Will be fixed in 1.8.3.