From 40e903f89794e1cf99af881c8292ad1937c77379 Mon Sep 17 00:00:00 2001 From: webchick Date: Wed, 20 Nov 2013 12:52:58 -0800 Subject: [PATCH] =?UTF-8?q?Rollback=20of=20Issue=20#2138239=20by=20damiank?= =?UTF-8?q?loip,=20tim.plunkett,=20amateescu:=20Use=20GlobIterator=20inste?= =?UTF-8?q?ad=20of=20glob.=20=E2=80=94=20breaks=20testbot.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- core/lib/Drupal/Core/Config/FileStorage.php | 13 +++++-------- core/lib/Drupal/Core/Config/InstallStorage.php | 7 ++++--- .../lib/Drupal/ckeditor/Plugin/Editor/CKEditor.php | 6 +++--- 3 files changed, 12 insertions(+), 14 deletions(-) diff --git a/core/lib/Drupal/Core/Config/FileStorage.php b/core/lib/Drupal/Core/Config/FileStorage.php index 52c9aa520db..bf178c56194 100644 --- a/core/lib/Drupal/Core/Config/FileStorage.php +++ b/core/lib/Drupal/Core/Config/FileStorage.php @@ -204,14 +204,11 @@ class FileStorage implements StorageInterface { throw new StorageException($this->directory . '/ not found.'); } $extension = '.' . static::getFileExtension(); - $files = new \GlobIterator(DRUPAL_ROOT . '/' . $this->directory . '/' . $prefix . '*' . $extension); - - $names = array(); - foreach ($files as $file) { - $names[] = $file->getBasename($extension); - } - - return $names; + $files = glob($this->directory . '/' . $prefix . '*' . $extension); + $clean_name = function ($value) use ($extension) { + return basename($value, $extension); + }; + return array_map($clean_name, $files); } /** diff --git a/core/lib/Drupal/Core/Config/InstallStorage.php b/core/lib/Drupal/Core/Config/InstallStorage.php index 16bcc1c3523..e8442ea7a77 100644 --- a/core/lib/Drupal/Core/Config/InstallStorage.php +++ b/core/lib/Drupal/Core/Config/InstallStorage.php @@ -134,9 +134,10 @@ class InstallStorage extends FileStorage { foreach ($list as $name) { $directory = $this->getComponentFolder($type, $name); if (file_exists($directory)) { - $files = new \GlobIterator(DRUPAL_ROOT . '/' . $directory . '/*' . $extension); - foreach ($files as $file) { - $folders[$file->getBasename($extension)] = $directory; + $files = glob($directory . '/*' . $extension); + foreach ($files as $filename) { + $name = basename($filename, $extension); + $folders[$name] = $directory; } } } diff --git a/core/modules/ckeditor/lib/Drupal/ckeditor/Plugin/Editor/CKEditor.php b/core/modules/ckeditor/lib/Drupal/ckeditor/Plugin/Editor/CKEditor.php index 3eefd376b7c..bc0dc61ac5b 100644 --- a/core/modules/ckeditor/lib/Drupal/ckeditor/Plugin/Editor/CKEditor.php +++ b/core/modules/ckeditor/lib/Drupal/ckeditor/Plugin/Editor/CKEditor.php @@ -320,9 +320,9 @@ class CKEditor extends EditorBase implements ContainerFactoryPluginInterface { if (empty($langcodes)) { $langcodes = array(); // Collect languages included with CKEditor based on file listing. - $ckeditor_languages = new \GlobIterator(DRUPAL_ROOT . '/core/assets/vendor/ckeditor/lang/*.js'); - foreach ($ckeditor_languages as $language_file) { - $langcode = $language_file->getBasename('.js'); + $ckeditor_languages = glob(DRUPAL_ROOT . '/core/assets/vendor/ckeditor/lang/*.js'); + foreach ($ckeditor_languages as $language_filename) { + $langcode = basename($language_filename, '.js'); $langcodes[$langcode] = $langcode; } cache('ckeditor.languages')->set('langcodes', $langcodes);