Access to files extracted as part of the data of the file scopes is handled through the classes ScopeFileAccessFlat and ScopeFileAccessArchive. There are still some places where the code directly accesses the files and doesn't go through the encapsulation layer. We should make use of the abstraction there as well.
The following places need to be adapted:
[ ] Autoyast#apply_unmanaged_files (directly uses path of scope file store)
[ ] Autoyast#write (directly copies the content of all description sub directories)
thardeck
commented
9 years ago
Access to files extracted as part of the data of the file scopes is handled through the classes
ScopeFileAccessFlatandScopeFileAccessArchive. There are still some places where the code directly accesses the files and doesn't go through the encapsulation layer. We should make use of the abstraction there as well.The following places need to be adapted:
Autoyast#apply_unmanaged_files(directly uses path of scope file store)Autoyast#write(directly copies the content of all description sub directories)AnalyzeConfigFileDiffsTask#analyze(directly reads config files from scope directory, directly writes diffs)KiwiConfig#check_existance_of_extracted_files(directly checks existence of scope store directories)ConfigFilesRenderer#do_render(implicitly assumes flat file storage for diffs directory)Html.generate(implicitly assumes flat file storage for diffs directory)UnmanagedFilesInspector#extract_tar_metadata(relies on specific tar layout of stored files)FileValidator#expected_files(directly accesses internal storage format)