paweld/lazarus
1
0
Fork 0
mirror of https://gitlab.com/freepascal.org/lazarus/lazarus.git synced 2025-04-06 15:37:51 +02:00
Code Issues Packages Projects Releases Wiki Activity

LCL: TOpenDialog: set ofExtensionDifferent if (DefaultExt <> '') and (Extension <> DefExt) for all WS.

Browse Source 
Our docs about ofExtensionDifferent wrongfully states: "Allows a file name with an extension which does not match the filters or default extension in a file dialog."
This is not how Delphi defines it, and also not how it's implemented in pre-Vista dialogs (using OPENFILE structure, the OS sets the corresponding flag).
(Since there have been no bugreports about this, it's doubtfull anybody cares, but nevertheless...)
This commit is contained in:
Bart 2023-08-30 23:23:17 +02:00
parent 33484eab0b
commit 7d4a36a290
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lcl/include/filedialog.inc
View File

@ -352,6 +352,9 @@ begin
else
AFilename:=AFilename+DefaultExt;
end;
if (not {already} (ofExtensionDifferent in FOptions)) and (DefaultExt <> '')
and (CompareFileNames(DefaultExt,ExtractFileExt(AFilename)) <> 0) then
Include(FOptions, ofExtensionDifferent);
//ofOverwritePrompt -> is done in the interface
if (ofPathMustExist in Options)
and (not DirPathExists(ExtractFileDir(AFilename))) then begin
@ -430,6 +433,7 @@ end;
------------------------------------------------------------------------------}
function TOpenDialog.DoExecute: boolean;
begin
Options := Options - [ofExtensionDifferent]; //clear, it'll be set by WS or in CheckFile
Result:=inherited DoExecute;
if not (ofNoResolveLinks in Options) then
ResolveLinks;