Skip to content

Commit 039b88d

Browse files
committed
Removed PackageContentLoaderFactory
1 parent bf466db commit 039b88d

4 files changed

Lines changed: 17 additions & 42 deletions

File tree

src/PackScan.PackagesProvider.Generator/PackageContents/Core/IPackageContentLoaderFactory.cs

Lines changed: 0 additions & 7 deletions
This file was deleted.

src/PackScan.PackagesProvider.Generator/PackageContents/Core/Loader/PackageContentLoaderFactory.cs

Lines changed: 0 additions & 20 deletions
This file was deleted.

src/PackScan.PackagesProvider.Generator/PackageContents/Core/PackageContentManager.cs

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,18 @@ namespace PackScan.PackagesProvider.Generator.PackageContents.Core;
66

77
internal sealed class PackageContentManager : IPackageContentManager
88
{
9-
private readonly IPackageContentLoader<byte[], ImageType> _imageContentFileProvider;
10-
private readonly IPackageContentLoader<string, TextType> _textContentFileProvider;
119
private IReadOnlyDictionary<string, PackageContents>? _contentsByPackageId;
1210

1311
public required ContentLoadMode IconLoadMode { get; init; }
1412
public required ContentLoadMode LicenseLoadMode { get; init; }
1513
public required ContentLoadMode ReadMeLoadMode { get; init; }
1614
public required ContentLoadMode ReleaseNotesLoadMode { get; init; }
1715

16+
public required IPackageContentLoader<byte[], ImageType> IconContentLoader { get; init; }
17+
public required IPackageContentLoader<string, TextType> LicenseContentLoader { get; init; }
18+
public required IPackageContentLoader<string, TextType> ReadMeContentLoader { get; init; }
19+
public required IPackageContentLoader<string, TextType> ReleaseNotesContentLoader { get; init; }
20+
1821
public bool HasImageFiles
1922
{
2023
get
@@ -61,12 +64,6 @@ public IEnumerable<IPackageContent> AllContents
6164
}
6265
}
6366

64-
public PackageContentManager(IPackageContentLoaderFactory contentLoaderFactory)
65-
{
66-
_imageContentFileProvider = contentLoaderFactory.CreateImageLoader();
67-
_textContentFileProvider = contentLoaderFactory.CreateTextLoader();
68-
}
69-
7067
public void LoadAll(IReadOnlyCollection<IPackageData> packages, bool parallel, CancellationToken cancellationToken)
7168
{
7269
_contentsByPackageId = parallel
@@ -112,10 +109,10 @@ private PackageContents LoadPackageContents(IPackageData package, CancellationTo
112109
{
113110
PackageContents contents = new()
114111
{
115-
Icon = _imageContentFileProvider.TryLoad(IconLoadMode, package.Icon, cancellationToken),
116-
License = _textContentFileProvider.TryLoad(LicenseLoadMode, package.License, cancellationToken),
117-
ReadMe = _textContentFileProvider.TryLoad(ReadMeLoadMode, package.ReadMe, cancellationToken),
118-
ReleaseNotes = _textContentFileProvider.TryLoad(ReleaseNotesLoadMode, package.ReleaseNotes, cancellationToken),
112+
Icon = IconContentLoader.TryLoad(IconLoadMode, package.Icon, cancellationToken),
113+
License = LicenseContentLoader.TryLoad(LicenseLoadMode, package.License, cancellationToken),
114+
ReadMe = ReadMeContentLoader.TryLoad(ReadMeLoadMode, package.ReadMe, cancellationToken),
115+
ReleaseNotes = ReleaseNotesContentLoader.TryLoad(ReleaseNotesLoadMode, package.ReleaseNotes, cancellationToken),
119116
};
120117

121118
RenameFileToPackageId(FileExtensionMappings.ExtensionByImageType, package.Id, contents.Icon, "Icon", ".img");

src/PackScan.PackagesProvider.Generator/PackagesProviderGenerator.cs

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,14 +65,19 @@ private IPackageContentManager CreateAndLoadContentManager(HttpClientFactory htt
6565
? Path.Combine(Path.GetTempPath(), "PackScan.PackagesProvider.Writer", "DownloadCache")
6666
: Environment.ExpandEnvironmentVariables(DownloadCacheFolder);
6767

68-
PackageContentLoaderFactory contentLoaderFactory = new(filesManager, httpClientFactory, downloadCacheFolderPath);
69-
70-
PackageContentManager contentManager = new(contentLoaderFactory)
68+
PackageContentManager contentManager = new()
7169
{
7270
IconLoadMode = IconContentLoadMode,
71+
IconContentLoader = new PackageImageContentLoader(filesManager, httpClientFactory, downloadCacheFolderPath),
72+
7373
LicenseLoadMode = LicenseContentLoadMode,
74+
LicenseContentLoader = new PackageTextContentLoader(filesManager, httpClientFactory, downloadCacheFolderPath),
75+
7476
ReadMeLoadMode = ReadMeContentLoadMode,
77+
ReleaseNotesContentLoader = new PackageTextContentLoader(filesManager, httpClientFactory, downloadCacheFolderPath),
78+
7579
ReleaseNotesLoadMode = ReleaseNotesContentLoadMode,
80+
ReadMeContentLoader = new PackageTextContentLoader(filesManager, httpClientFactory, downloadCacheFolderPath),
7681
};
7782

7883
contentManager.LoadAll(packagesData, LoadContentParallel, cancellationToken);

0 commit comments

Comments
 (0)