Convert hook_archiver_info into the New Shiny(TM)

Looks good to me.

Eclipse

archiver_info_shiny_edition.patch queued for re-testing.

Bit of nitpicking, my apologies. Functionally this is RTBC, the conversion looks awesome.

+++ b/core/includes/common.incundefined
@@ -6551,24 +6525,11 @@ function archiver_get_extensions() {
-  $filepath = drupal_realpath($file);
-  if (!is_file($filepath)) {
+  $filename = drupal_realpath($file);
+  if (!is_file($filename)) {

Here you change the variable name away from $filepath, but both Components refer to it as such. Why not leave it?

+++ b/core/lib/Drupal/Core/Archiver/ArchiverManager.phpundefined
@@ -0,0 +1,58 @@
+<?php
+
+/**
+ * Definition of \Drupal\Core\Archiver\ArchiverManager.
+ */
+
+namespace Drupal\Core\Archiver;
+
+
+use Drupal\Component\Plugin\Factory\DefaultFactory;

Contains \Drupal\...

and there is an extra blank line

+++ b/core/lib/Drupal/Core/Archiver/ArchiverManager.phpundefined
@@ -0,0 +1,58 @@
+use Drupal\Core\Plugin\Discovery\CacheDecorator;
+
+class ArchiverManager extends PluginManagerBase {

Missing a docblock

+++ b/core/lib/Drupal/Core/Archiver/ArchiverManager.phpundefined
@@ -0,0 +1,58 @@
+   *   An array of paths keyed by it's corresponding namespaces.

s/it's/its

+++ b/core/lib/Drupal/Core/Archiver/ArchiverManager.phpundefined
@@ -0,0 +1,58 @@
+   * Overrides \Drupal\Component\Plugin\PluginManagerBase\createInstance().

::createInstance, not \

+++ b/core/lib/Drupal/Core/Archiver/ArchiverManager.phpundefined
@@ -0,0 +1,58 @@
+   * Implements \Drupal\Core\PluginManagerInterface\getInstance.

::getInstance()

+++ b/core/modules/system/lib/Drupal/system/Plugin/Core/Archiver/Tar.phpundefined
@@ -0,0 +1,25 @@
+ * Definition of Drupal\system\Plugin\Core\Archiver\Tar.

+++ b/core/modules/system/lib/Drupal/system/Plugin/Core/Archiver/Zip.phpundefined
@@ -0,0 +1,27 @@
+ * Definition of Drupal\system\Plugin\Core\Archiver\Zip.

Contains \Drupal\

Thanks!

#4: 1950726_4.patch queued for re-testing.

OKie doke!

Committed and pushed to 8.x. Thanks!

This needs a change notice. Also see #1987298: Shorten directory structure and PSR-0 namespacing for plugins for the updated path.

I created the change record. https://drupal.org/node/2003376

Some before and after will certainly help so please add that and name spaces as well. Thanks for the change notice.

Thanks @chx.

Reverting title.