Skip to content

Commit b01d1e9

Browse files
author
Stewart Miles
committed
Changed obsolete files dialog to display VH package names.
The version handler obsolete files dialog would show all manifest files referencing each obsolete file which is pretty hard to read. This commit changes this to display the package name / display name of each manifest referencing each file to make it easier to see where a file came from. Change-Id: If4d1255ba5327061264f6450fb0feb23f501f113
1 parent ad8bd8d commit b01d1e9

File tree

1 file changed

+13
-10
lines changed

1 file changed

+13
-10
lines changed

source/VersionHandlerImpl/src/VersionHandlerImpl.cs

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1998,12 +1998,12 @@ public class ObsoleteFiles {
19981998
/// Obsolete files that are referenced by manifests. Each item in
19991999
/// the dictionary contains a list of manifests referencing the file.
20002000
/// </summary>
2001-
public Dictionary<string, List<string>> referenced;
2001+
public Dictionary<string, List<ManifestReferences>> referenced;
20022002

20032003
/// <summary>
20042004
/// Same as the "referenced" member excluding manifest files.
20052005
/// </summary>
2006-
public Dictionary<string, List<string>> referencedExcludingManifests;
2006+
public Dictionary<string, List<ManifestReferences>> referencedExcludingManifests;
20072007

20082008
/// <summary>
20092009
/// Get all referenced and unreferenced obsolete files.
@@ -2058,17 +2058,16 @@ public ObsoleteFiles(
20582058
// which contains a list of manifest filenames which reference
20592059
// each file.
20602060
var referencedObsoleteFiles =
2061-
new Dictionary<string, List<string>>();
2062-
var referencedObsoleteFilesExcludingManifests = new Dictionary<string, List<string>>();
2061+
new Dictionary<string, List<ManifestReferences>>();
2062+
var referencedObsoleteFilesExcludingManifests =
2063+
new Dictionary<string, List<ManifestReferences>>();
20632064
var obsoleteFilesToDelete = new HashSet<string>();
20642065
var obsoleteFilesToDeleteExcludingManifests = new HashSet<string>();
20652066
foreach (var obsoleteFile in obsoleteFiles) {
2066-
var manifestsReferencingFile = new List<string>();
2067+
var manifestsReferencingFile = new List<ManifestReferences>();
20672068
foreach (var manifestReferences in manifestReferencesList) {
2068-
if (manifestReferences.currentFiles.Contains(
2069-
obsoleteFile)) {
2070-
manifestsReferencingFile.Add(
2071-
manifestReferences.currentMetadata.filename);
2069+
if (manifestReferences.currentFiles.Contains(obsoleteFile)) {
2070+
manifestsReferencingFile.Add(manifestReferences);
20722071
}
20732072
}
20742073
// If the referenced file doesn't exist, ignore it.
@@ -2777,7 +2776,11 @@ private static void UpdateVersionedAssetsOnMainThread(bool forceUpdate,
27772776
if (!ExecutionEnvironment.InBatchMode) {
27782777
foreach (var item in obsoleteFiles.referenced) {
27792778
var filename = item.Key;
2780-
var references = item.Value;
2779+
var manifestReferencesList = item.Value;
2780+
var references = new List<string>();
2781+
foreach (var manifestReferences in manifestReferencesList) {
2782+
references.Add(manifestReferences.filenameCanonical);
2783+
}
27812784
cleanupFiles.Add(
27822785
new KeyValuePair<string, string>(
27832786
filename,

0 commit comments

Comments
 (0)