Code Tidy: Remove unused obsoleted InstalledPackage mapping

[Migaroez]

Description

Removed the obsoleted InstalledPackage → PackageMigrationStatusResponseModel mapper registration and Map method from PackageViewModelMapDefinition. The current code already uses its replacement IPackagePresentationFactory.CreatePackageMigrationStatusResponseModel(PagedModel<InstalledPackage> installedPackages)

[claude]

Claude finished @Migaroez's task in 1m 48s —— View job

---

PR Review

Target: origin/v18/dev · Based on commit: a3b100b · Classified as: Refactor

Removes the obsoleted InstalledPackage → PackageMigrationStatusResponseModel mapper registration and its private Map method, both of which were marked [Obsolete("Scheduled for removal in Umbraco 18.")] and have been superseded by IPackagePresentationFactory.CreatePackageMigrationStatusResponseModel.

• Breaking changes: None — the removed Map method was private static (not public API), and the mapper registration is an internal implementation detail. Removal is on schedule (obsoleted for v18).

---

Suggestions

• src/Umbraco.Cms.Api.Management/Mapping/Package/PackageViewModelMapDefinition.cs:16: The DefineMaps XML doc still references <see cref="InstalledPackage"/> and <see cref="PackageMigrationStatusResponseModel"/> even though those mappings no longer exist in this method. The description is now stale. Consider trimming it to only mention the two mappings that remain (PackageModelBase → PackageDefinition and PackageDefinition → PackageDefinitionResponseModel). Fix this →

---

Approved with Suggestions for improvement

Good to go — clean removal of a properly scheduled obsoleted mapping. The only thing worth addressing is the stale XML doc comment.

[AndyButland]

Looks good. I just updated the XML docs on the class to better reflect what the methods now do (there were also some other out-dated references to mapped classes).