Remove incorrect modification of `$pendingPackages` in PackageInstallationNodeBuilder...
authorTim Düsterhus <duesterhus@woltlab.com>
Thu, 6 Apr 2023 09:32:52 +0000 (11:32 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Mon, 17 Apr 2023 14:09:15 +0000 (16:09 +0200)
commit16611b023c45a4fc70518b3a977ac2e7d376285e
tree7bfac66c934e1d61eb3ea71ae2a4dacf690aba55
parentcfc1d51329e877823e4d5d4975e3821f5817ac3c
Remove incorrect modification of `$pendingPackages` in PackageInstallationNodeBuilder::buildRequirementNodes()

Previously the `$pendingPackages` were only used to resolve dependencies, thus
the exact position for registering the new version does not really matter.
However with 5df3e65185d4d3f1be7464b1999b949946a5c765 the version is also used
to select the correct update instructions.

Thus the new package version must only be registered after the nodes for the
package in question have actually been built, similarly to the change in
2406351b19c5fac66b8413ba1f6660a376b7e18f.

In fact the registration in `buildRequirementNodes()` is completely redundant
since commit 7ddf2af9528be4c114d4ad83b804c3529769f24c, as the new version will
be registered in `->buildNodes()` a few lines afterwards. The
`$pendingPackages` variable is not read in-between. Thus this will be our fix:
The incorrect line is simply removed without replacement.
wcfsetup/install/files/lib/system/package/PackageInstallationNodeBuilder.class.php