Skip to content

Commit

Permalink
fix(developer): suppress emission of new empty fields in package editor
Browse files Browse the repository at this point in the history
Fixes #11002.

The fields `<RelatedPackages/>` and `<Examples/>` should not be emitted
if they have no content, to improve file backward-compatibility.
  • Loading branch information
mcdurdin committed Mar 15, 2024
1 parent 7120160 commit 374955d
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion common/windows/delphi/packages/PackageInfo.pas
Original file line number Diff line number Diff line change
Expand Up @@ -1857,7 +1857,8 @@ procedure TPackage.DoSaveXML(ARoot: IXMLNode);
Keyboards.SaveXML(ARoot);
if LexicalModels.Count > 0 then
LexicalModels.SaveXML(ARoot);
RelatedPackages.SaveXML(ARoot);
if RelatedPackages.Count > 0 then
RelatedPackages.SaveXML(ARoot);
end;

procedure TPackage.SaveIni;
Expand Down Expand Up @@ -2742,6 +2743,9 @@ procedure TPackageKeyboardExampleList.SaveXML(ARoot: IXMLNode);
j: Integer;
AExample: IXMLNode;
begin
if Count = 0 then
Exit;

AExamples := ARoot.AddChild(SXML_PackageKeyboard_Examples);
for j := 0 to Count - 1 do
begin
Expand Down

0 comments on commit 374955d

Please sign in to comment.