From b4ed331168c72b65d01d81b403507e244f0f84a3 Mon Sep 17 00:00:00 2001
From: Nils Adermann <naderman@naderman.de>
Date: Thu, 30 Apr 2015 17:24:24 +0200
Subject: [PATCH] The Solver Request no longer depends on the Pool

---
 src/Composer/DependencyResolver/Request.php        |  4 +---
 src/Composer/Installer.php                         |  8 ++++----
 .../Test/DependencyResolver/RequestTest.php        | 14 +++-----------
 3 files changed, 8 insertions(+), 18 deletions(-)

diff --git a/src/Composer/DependencyResolver/Request.php b/src/Composer/DependencyResolver/Request.php
index 79904a3a1..0776f040e 100644
--- a/src/Composer/DependencyResolver/Request.php
+++ b/src/Composer/DependencyResolver/Request.php
@@ -20,11 +20,9 @@ use Composer\Package\LinkConstraint\LinkConstraintInterface;
 class Request
 {
     protected $jobs;
-    protected $pool;
 
-    public function __construct(Pool $pool)
+    public function __construct()
     {
-        $this->pool = $pool;
         $this->jobs = array();
     }
 
diff --git a/src/Composer/Installer.php b/src/Composer/Installer.php
index e1b712954..d075f6b46 100644
--- a/src/Composer/Installer.php
+++ b/src/Composer/Installer.php
@@ -287,7 +287,7 @@ class Installer
                     $pool->addRepository($installedRepo, $aliases);
 
                     // creating requirements request
-                    $request = $this->createRequest($pool, $this->package, $platformRepo);
+                    $request = $this->createRequest($this->package, $platformRepo);
                     $request->updateAll();
                     foreach ($this->package->getRequires() as $link) {
                         $request->install($link->getTarget(), $link->getConstraint());
@@ -398,7 +398,7 @@ class Installer
         }
 
         // creating requirements request
-        $request = $this->createRequest($pool, $this->package, $platformRepo);
+        $request = $this->createRequest($this->package, $platformRepo);
 
         if (!$installFromLock) {
             // remove unstable packages from the localRepo if they don't match the current stability settings
@@ -735,9 +735,9 @@ class Installer
         return new DefaultPolicy($preferStable, $preferLowest);
     }
 
-    private function createRequest(Pool $pool, RootPackageInterface $rootPackage, PlatformRepository $platformRepo)
+    private function createRequest(RootPackageInterface $rootPackage, PlatformRepository $platformRepo)
     {
-        $request = new Request($pool);
+        $request = new Request();
 
         $constraint = new VersionConstraint('=', $rootPackage->getVersion());
         $constraint->setPrettyString($rootPackage->getPrettyVersion());
diff --git a/tests/Composer/Test/DependencyResolver/RequestTest.php b/tests/Composer/Test/DependencyResolver/RequestTest.php
index 78a1da962..e16ca6e11 100644
--- a/tests/Composer/Test/DependencyResolver/RequestTest.php
+++ b/tests/Composer/Test/DependencyResolver/RequestTest.php
@@ -13,7 +13,6 @@
 namespace Composer\Test\DependencyResolver;
 
 use Composer\DependencyResolver\Request;
-use Composer\DependencyResolver\Pool;
 use Composer\Repository\ArrayRepository;
 use Composer\TestCase;
 
@@ -21,7 +20,6 @@ class RequestTest extends TestCase
 {
     public function testRequestInstallAndRemove()
     {
-        $pool = new Pool;
         $repo = new ArrayRepository;
         $foo = $this->getPackage('foo', '1');
         $bar = $this->getPackage('bar', '1');
@@ -30,9 +28,8 @@ class RequestTest extends TestCase
         $repo->addPackage($foo);
         $repo->addPackage($bar);
         $repo->addPackage($foobar);
-        $pool->addRepository($repo);
 
-        $request = new Request($pool);
+        $request = new Request();
         $request->install('foo');
         $request->fix('bar');
         $request->remove('foobar');
@@ -48,7 +45,6 @@ class RequestTest extends TestCase
 
     public function testRequestInstallSamePackageFromDifferentRepositories()
     {
-        $pool = new Pool;
         $repo1 = new ArrayRepository;
         $repo2 = new ArrayRepository;
 
@@ -58,10 +54,7 @@ class RequestTest extends TestCase
         $repo1->addPackage($foo1);
         $repo2->addPackage($foo2);
 
-        $pool->addRepository($repo1);
-        $pool->addRepository($repo2);
-
-        $request = new Request($pool);
+        $request = new Request();
         $request->install('foo', $constraint = $this->getVersionConstraint('=', '1'));
 
         $this->assertEquals(
@@ -74,8 +67,7 @@ class RequestTest extends TestCase
 
     public function testUpdateAll()
     {
-        $pool = new Pool;
-        $request = new Request($pool);
+        $request = new Request();
 
         $request->updateAll();