Skip to content

Commit a951f61

Browse files
committed
Revert #129 to fix regression
1 parent 84ce71b commit a951f61

File tree

2 files changed

+4
-16
lines changed

2 files changed

+4
-16
lines changed

src/Constraint/MultiConstraint.php

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -228,18 +228,6 @@ public static function create(array $constraints, $conjunctive = true)
228228
return $constraints[0];
229229
}
230230

231-
foreach ($constraints as $k => $constraint) {
232-
if ($constraint instanceof MatchAllConstraint) {
233-
if (!$conjunctive) {
234-
return new MatchAllConstraint();
235-
}
236-
unset($constraints[$k]);
237-
if (1 === \count($constraints)) {
238-
return $constraints[0];
239-
}
240-
}
241-
}
242-
243231
$optimized = self::optimizeConstraints($constraints, $conjunctive);
244232
if ($optimized !== null) {
245233
list($constraints, $conjunctive) = $optimized;

tests/Constraint/MultiConstraintTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -290,16 +290,16 @@ public function testCreatesMatchAllConstraintIfNoneGiven()
290290
$this->assertInstanceOf('Composer\Semver\Constraint\MatchAllConstraint', MultiConstraint::create(array()));
291291
}
292292

293-
public function testRemovesMatchAllConstraintIfConjunctiveAndCombinedWithOtherConstraints()
293+
public function testMatchAllConstraintWithinConjunctiveMultiConstraint()
294294
{
295-
$this->assertSame('[>= 2.5.0.0-dev <= 3.0.0.0-dev]', (string) MultiConstraint::create(
295+
$this->assertSame('[>= 2.5.0.0-dev <= 3.0.0.0-dev *]', (string) MultiConstraint::create(
296296
array(new Constraint('>=', '2.5.0.0-dev'), new Constraint('<=', '3.0.0.0-dev'), new MatchAllConstraint())
297297
));
298298
}
299299

300-
public function testCreatesMatchAllConstraintIfDisjunctiveAndCombinedWithAnotherOne()
300+
public function testMatchAllConstraintWithinDisjunctiveMultiConstraint()
301301
{
302-
$this->assertInstanceOf('Composer\Semver\Constraint\MatchAllConstraint', MultiConstraint::create(
302+
$this->assertSame('[>= 2.5.0.0-dev || *]', (string) MultiConstraint::create(
303303
array(new Constraint('>=', '2.5.0.0-dev'), new MatchAllConstraint()), false
304304
));
305305
}

0 commit comments

Comments
 (0)