View Issue Details

IDProjectCategoryView StatusLast Update
0006749JEDI VCL02 Installationpublic2023-11-11 19:19
Reporteroro06Assigned ToAHUser 
PrioritynormalSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
Product VersionDaily / GIT 
Target VersionFixed in VersionDaily / GIT 
Summary0006749: parameter missing (2 instead of 3) in copy function in JVCLInstall/DelphiData.pas Line 897
DescriptionWin7, Fresh D7 install, with fresh JCL+JVCL install (git repo of today/220506)
running jcl install went fine
then running jvcl install give this error :

>install.bat
[Compiling installer...]
Using Delphi 7

Borland Delphi Version 15.0
Copyright (c) 1983,2002 Borland Software Corporation
DelphiData.pas(897) Error: Not enough actual parameters
Utils.pas(83) Fatal: Could not compile used unit 'DelphiData.pas'


line 897 is
            CandidateEdition := Copy(KeyNames[i], Length(BDS19UpCoreCommonFilesPrefix) + 1);

2 params instead of 3 : index or length is missing
Steps To ReproduceWin7, Fresh D7 install, with fresh JCL+JVCL install (git repo of today/220506)
then install jvcl
TagsNo tags attached.

Activities

oro06

2022-05-10 10:50

reporter   ~0022029

from radstudio documentation
https://docwiki.embarcadero.com/Libraries/Sydney/en/System.Copy:

function Copy(S: <string or dynamic array>; Index: Integer; Count: Integer): string;

there should be 3 parameters

delphichem1

2022-06-01 10:40

reporter   ~0022032

line 897 is
            CandidateEdition := Copy(KeyNames[i], Length(BDS19UpCoreCommonFilesPrefix) + 1);

change it with

            CandidateEdition := Copy(KeyNames[i], Length(BDS19UpCoreCommonFilesPrefix) + 1, MaxInt);

AHUser

2023-11-11 19:19

developer   ~0022051

Fixed in git main branch.

Issue History

Date Modified Username Field Change
2022-05-06 13:15 oro06 New Issue
2022-05-10 10:50 oro06 Note Added: 0022029
2022-06-01 10:40 delphichem1 Note Added: 0022032
2023-11-11 19:19 AHUser Assigned To => AHUser
2023-11-11 19:19 AHUser Status new => resolved
2023-11-11 19:19 AHUser Resolution open => fixed
2023-11-11 19:19 AHUser Fixed in Version => Daily / GIT
2023-11-11 19:19 AHUser Note Added: 0022051