1
0
Fork 0

Remove match types from Pool as these are no longer used

pull/8567/head
Nils Adermann 2020-01-31 01:11:13 +01:00
parent 80a5fdf398
commit 415b36a1a1
1 changed files with 11 additions and 26 deletions

View File

@ -27,11 +27,6 @@ use Composer\Package\PackageInterface;
*/ */
class Pool implements \Countable class Pool implements \Countable
{ {
const MATCH_NONE = 0;
const MATCH = 1;
const MATCH_PROVIDE = 2;
const MATCH_REPLACE = 3;
protected $packages = array(); protected $packages = array();
protected $packageByName = array(); protected $packageByName = array();
protected $versionParser; protected $versionParser;
@ -109,18 +104,8 @@ class Pool implements \Countable
$matches = array(); $matches = array();
foreach ($this->packageByName[$name] as $candidate) { foreach ($this->packageByName[$name] as $candidate) {
switch ($this->match($candidate, $name, $constraint)) { if ($this->match($candidate, $name, $constraint)) {
case self::MATCH_NONE:
break;
case self::MATCH:
case self::MATCH_REPLACE:
case self::MATCH_PROVIDE:
$matches[] = $candidate; $matches[] = $candidate;
break;
default:
throw new \UnexpectedValueException('Unexpected match type');
} }
} }
@ -154,7 +139,7 @@ class Pool implements \Countable
* @param PackageInterface $candidate * @param PackageInterface $candidate
* @param string $name Name of the package to be matched * @param string $name Name of the package to be matched
* @param ConstraintInterface $constraint The constraint to verify * @param ConstraintInterface $constraint The constraint to verify
* @return int One of the MATCH* constants of this class or 0 if there is no match * @return bool
*/ */
public function match($candidate, $name, ConstraintInterface $constraint = null) public function match($candidate, $name, ConstraintInterface $constraint = null)
{ {
@ -165,10 +150,10 @@ class Pool implements \Countable
$pkgConstraint = new Constraint('==', $candidateVersion); $pkgConstraint = new Constraint('==', $candidateVersion);
if ($constraint === null || $constraint->matches($pkgConstraint)) { if ($constraint === null || $constraint->matches($pkgConstraint)) {
return self::MATCH; return true;
} }
return self::MATCH_NONE; return false;
} }
$provides = $candidate->getProvides(); $provides = $candidate->getProvides();
@ -178,28 +163,28 @@ class Pool implements \Countable
if (isset($replaces[0]) || isset($provides[0])) { if (isset($replaces[0]) || isset($provides[0])) {
foreach ($provides as $link) { foreach ($provides as $link) {
if ($link->getTarget() === $name && ($constraint === null || $constraint->matches($link->getConstraint()))) { if ($link->getTarget() === $name && ($constraint === null || $constraint->matches($link->getConstraint()))) {
return self::MATCH_PROVIDE; return true;
} }
} }
foreach ($replaces as $link) { foreach ($replaces as $link) {
if ($link->getTarget() === $name && ($constraint === null || $constraint->matches($link->getConstraint()))) { if ($link->getTarget() === $name && ($constraint === null || $constraint->matches($link->getConstraint()))) {
return self::MATCH_REPLACE; return true;
} }
} }
return self::MATCH_NONE; return false;
} }
if (isset($provides[$name]) && ($constraint === null || $constraint->matches($provides[$name]->getConstraint()))) { if (isset($provides[$name]) && ($constraint === null || $constraint->matches($provides[$name]->getConstraint()))) {
return self::MATCH_PROVIDE; return true;
} }
if (isset($replaces[$name]) && ($constraint === null || $constraint->matches($replaces[$name]->getConstraint()))) { if (isset($replaces[$name]) && ($constraint === null || $constraint->matches($replaces[$name]->getConstraint()))) {
return self::MATCH_REPLACE; return true;
} }
return self::MATCH_NONE; return false;
} }
public function isUnacceptableFixedPackage(PackageInterface $package) public function isUnacceptableFixedPackage(PackageInterface $package)