Remove PackageArchive::getAllExistingRequirements()
authorTim Düsterhus <duesterhus@woltlab.com>
Tue, 15 Nov 2022 10:13:32 +0000 (11:13 +0100)
committerTim Düsterhus <duesterhus@woltlab.com>
Tue, 15 Nov 2022 10:13:32 +0000 (11:13 +0100)
commit01d85c250cdf5b18c5a46cbbb31d8a774f37b90e
treed6c3226635c63bbbe9028b747cf4140f9296297b
parent187abea3a3c393e2bc747d1ee5a064c6e0dcd97a
Remove PackageArchive::getAllExistingRequirements()

The last remaining user of this method was removed in the previous commit. The
logic of this method was overly complex:

- It handled the possibility that a single package is installed multiple times,
  which is not possible since forever.
- It contained special (and semi-broken) logic for requirement links that are
  already stored in the database. Since a93b160e72731d72e58a13052c1b6b83b089552a
  all requirements in the database were deleted before this method was called
  in PackageInstallationDispatcher. It also failed to account for a requirement
  link existing, but the target package being of an insufficient version.
wcfsetup/install/files/lib/system/package/PackageArchive.class.php