Updating $pluginPath loops to use App::pluginPath().

This commit is contained in:
mark_story 2009-10-14 09:47:07 -04:00
parent de8f251758
commit 2f81c29def
4 changed files with 11 additions and 29 deletions

View file

@ -222,20 +222,17 @@ class ShellDispatcher {
*/ */
function __buildPaths() { function __buildPaths() {
$paths = array(); $paths = array();
$pluginPaths = App::path('plugins');
if (!class_exists('Folder')) { if (!class_exists('Folder')) {
require LIBS . 'folder.php'; require LIBS . 'folder.php';
} }
foreach ($pluginPaths as $pluginPath) { $plugins = App::objects('plugin', null, false);
$Folder = new Folder($pluginPath);
list($plugins,) = $Folder->read(false, true);
foreach ((array)$plugins as $plugin) { foreach ((array)$plugins as $plugin) {
$path = $pluginPath . Inflector::underscore($plugin) . DS . 'vendors' . DS . 'shells' . DS; $pluginPath = App::pluginPath($plugin);
$path = $pluginPath . 'vendors' . DS . 'shells' . DS;
if (file_exists($path)) { if (file_exists($path)) {
$paths[] = $path; $paths[] = $path;
} }
} }
}
$vendorPaths = array_values(App::path('vendors')); $vendorPaths = array_values(App::path('vendors'));
foreach ($vendorPaths as $vendorPath) { foreach ($vendorPaths as $vendorPath) {

View file

@ -649,15 +649,7 @@ class Shell extends Object {
* @return string $path path to the correct plugin. * @return string $path path to the correct plugin.
**/ **/
function _pluginPath($pluginName) { function _pluginPath($pluginName) {
$pluginPaths = App::path('plugins'); return App::pluginPath($pluginName);
$pluginDirName = Inflector::underscore($pluginName);
foreach ($pluginPaths as $path) {
if (is_dir($path . $pluginDirName)) {
return $path . $pluginDirName . DS ;
}
}
return $pluginPaths[0] . $pluginDirName . DS;
} }
} }
?> ?>

View file

@ -648,11 +648,7 @@ class Dispatcher extends Object {
if ($pos > 0) { if ($pos > 0) {
$plugin = substr($url, 0, $pos - 1); $plugin = substr($url, 0, $pos - 1);
$url = preg_replace('/^' . preg_quote($plugin, '/') . '\//i', '', $url); $url = preg_replace('/^' . preg_quote($plugin, '/') . '\//i', '', $url);
$pluginPaths = App::path('plugins'); $paths[] = App::pluginPath($plugin) . 'vendors' . DS;
$count = count($pluginPaths);
for ($i = 0; $i < $count; $i++) {
$paths[] = $pluginPaths[$i] . $plugin . DS . 'vendors' . DS;
}
} }
$paths = array_merge($paths, App::path('vendors')); $paths = array_merge($paths, App::path('vendors'));

View file

@ -357,12 +357,9 @@ class TestManager {
} }
} else if (!empty($this->pluginTest)) { } else if (!empty($this->pluginTest)) {
$_pluginBasePath = APP . 'plugins' . DS . $this->pluginTest . DS . 'tests'; $_pluginBasePath = APP . 'plugins' . DS . $this->pluginTest . DS . 'tests';
$pluginPaths = App::path('plugins'); $pluginPath = App::pluginPath($this->pluginTest);
foreach ($pluginPaths as $path) { if (file_exists($pluginPath . DS . 'tests')) {
if (file_exists($path . $this->pluginTest . DS . 'tests')) { $_pluginBasePath = $pluginPath . DS . 'tests';
$_pluginBasePath = $path . $this->pluginTest . DS . 'tests';
break;
}
} }
$result = $_pluginBasePath . DS . $type; $result = $_pluginBasePath . DS . $type;
} else { } else {