From 6fb5d2647427e46c055b7bf6bf1c2a5a5a092fcd Mon Sep 17 00:00:00 2001 From: Jordi Boggiano Date: Wed, 18 Jul 2012 18:40:17 +0200 Subject: [PATCH] Revert "Add test and some fixes to allow absolute paths, refs #869" This reverts commit a2a2f4af59724090d875eb4460d7b7c690d30b54. --- src/Composer/Autoload/AutoloadGenerator.php | 24 ++++++------------- .../Test/Autoload/AutoloadGeneratorTest.php | 6 ++--- .../Autoload/Fixtures/autoload_classmap4.php | 1 - 3 files changed, 9 insertions(+), 22 deletions(-) diff --git a/src/Composer/Autoload/AutoloadGenerator.php b/src/Composer/Autoload/AutoloadGenerator.php index 9a52de54c..6bc06aa52 100644 --- a/src/Composer/Autoload/AutoloadGenerator.php +++ b/src/Composer/Autoload/AutoloadGenerator.php @@ -53,7 +53,7 @@ return array( EOF; $packageMap = $this->buildPackageMap($installationManager, $mainPackage, $localRepo->getPackages()); - $autoloads = $this->parseAutoloads($packageMap, $filesystem); + $autoloads = $this->parseAutoloads($packageMap); foreach ($autoloads['psr-0'] as $namespace => $paths) { $exportedPaths = array(); @@ -118,9 +118,10 @@ EOF; $autoloads['classmap'] = new \RecursiveIteratorIterator(new \RecursiveArrayIterator($autoloads['classmap'])); foreach ($autoloads['classmap'] as $dir) { foreach (ClassMapGenerator::createMap($dir) as $class => $path) { + $classmapFile .= ' '.var_export($class, true) . ' => '; - if (0 === strpos($path, getcwd())) { + if (0 === strpos($path, dirname($vendorPath), 0)) { // this path seems to be located within this application/package $path = '/' . $filesystem->findShortestPath(getcwd(), $path, true); $classmapFile .= '$baseDir . '; @@ -169,16 +170,11 @@ EOF; /** * Compiles an ordered list of namespace => path mappings * - * @param array $packageMap array of array(package, installDir-relative-to-composer.json) - * @param Filesystem $filesystem optional filesystem instance to use - * @return array array('psr-0' => array('Ns\\Foo' => array('installDir'))) + * @param array $packageMap array of array(package, installDir-relative-to-composer.json) + * @return array array('psr-0' => array('Ns\\Foo' => array('installDir'))) */ - public function parseAutoloads(array $packageMap, Filesystem $filesystem = null) + public function parseAutoloads(array $packageMap) { - if (!$filesystem) { - $filesystem = new Filesystem; - } - $autoloads = array('classmap' => array(), 'psr-0' => array(), 'files' => array()); foreach ($packageMap as $item) { list($package, $installPath) = $item; @@ -194,13 +190,7 @@ EOF; } foreach ($mapping as $namespace => $paths) { foreach ((array) $paths as $path) { - if ($filesystem->isAbsolutePath($path)) { - // handle absolute paths in raw form - $autoloads[$type][$namespace][] = $path; - } else { - // prepend install path (including target dir) to the relative paths - $autoloads[$type][$namespace][] = empty($installPath) ? $path : $installPath.'/'.$path; - } + $autoloads[$type][$namespace][] = empty($installPath) ? $path : $installPath.'/'.$path; } } } diff --git a/tests/Composer/Test/Autoload/AutoloadGeneratorTest.php b/tests/Composer/Test/Autoload/AutoloadGeneratorTest.php index 17cbcc1ee..e3b91127d 100644 --- a/tests/Composer/Test/Autoload/AutoloadGeneratorTest.php +++ b/tests/Composer/Test/Autoload/AutoloadGeneratorTest.php @@ -190,7 +190,7 @@ class AutoloadGeneratorTest extends TestCase $packages[] = $a = new MemoryPackage('a/a', '1.0', '1.0'); $packages[] = $b = new MemoryPackage('b/b', '1.0', '1.0'); $a->setAutoload(array('classmap' => array('src/'))); - $b->setAutoload(array('classmap' => array('src/', 'lib/', __FILE__))); + $b->setAutoload(array('classmap' => array('src/', 'lib/'))); $this->repository->expects($this->once()) ->method('getPackages') @@ -211,12 +211,10 @@ class AutoloadGeneratorTest extends TestCase 'ClassMapFoo' => $this->workingDir.'/composer-test-autoload/a/a/src/a.php', 'ClassMapBar' => $this->workingDir.'/composer-test-autoload/b/b/src/b.php', 'ClassMapBaz' => $this->workingDir.'/composer-test-autoload/b/b/lib/c.php', - 'Composer\Test\Autoload\AutoloadGeneratorTest' => __FILE__, ), include ($this->vendorDir.'/composer/autoload_classmap.php') ); - $expected = str_replace('%path%', var_export(__FILE__, true), file_get_contents(__DIR__.'/Fixtures/autoload_classmap4.php')); - $this->assertEquals($expected, file_get_contents($this->vendorDir.'/composer/autoload_classmap.php')); + $this->assertAutoloadFiles('classmap4', $this->vendorDir.'/composer', 'classmap'); } public function testClassMapAutoloadingEmptyDirAndExactFile() diff --git a/tests/Composer/Test/Autoload/Fixtures/autoload_classmap4.php b/tests/Composer/Test/Autoload/Fixtures/autoload_classmap4.php index 9e8546050..944a80bed 100644 --- a/tests/Composer/Test/Autoload/Fixtures/autoload_classmap4.php +++ b/tests/Composer/Test/Autoload/Fixtures/autoload_classmap4.php @@ -6,7 +6,6 @@ $vendorDir = dirname(__DIR__); $baseDir = dirname($vendorDir); return array( - 'Composer\\Test\\Autoload\\AutoloadGeneratorTest' => %path%, 'ClassMapBaz' => $baseDir . '/composer-test-autoload/b/b/lib/c.php', 'ClassMapFoo' => $baseDir . '/composer-test-autoload/a/a/src/a.php', 'ClassMapBar' => $baseDir . '/composer-test-autoload/b/b/src/b.php',