Config inspector is super useful for finding configuration files. But when you find the configuration you need, you have to go to the Core download page and waste a lot of time. It would be better to directly download the configuration from the inspect page.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lussoluca created an issue. See original summary.

lussoluca’s picture

Status: Active » Needs review
FileSize
3.47 KB
itamair’s picture

Subscribe!

sarci’s picture

Great!

MXT’s picture

Status: Needs review » Reviewed & tested by the community

It works for me: very useful, thanks!

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Needs work
+++ b/src/Controller/ConfigInspectorController.php
@@ -234,6 +239,27 @@ class ConfigInspectorController extends ControllerBase {
+    $data = $this->configInspectorManager->getConfigData($name);
+    unset($data['uuid']);
+    unset($data['_core']);
+    $data = Yaml::encode($data);

This is not what the core config export page does is it? If we are not downloading what we are inspecting then people will get confused.

lolandese’s picture

Reroll against latest dev, removing unset() as suggested.

vuil’s picture

Status: Needs review » Needs work

The patch can not be applied.

vuil’s picture

Issue tags: +Needs reroll
rpayanm’s picture

Status: Needs work » Needs review
FileSize
3.53 KB

Status: Needs review » Needs work

The last submitted patch, 10: 2684383-10.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

AjitS’s picture

Issue tags: -Needs reroll
vuil’s picture

Status: Needs work » Active
FileSize
7.11 KB
vuil’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 13: 2684383-13.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

vuil’s picture

Status: Needs work » Needs review
FileSize
7.42 KB

Add all required parent::__construct() containers into __construct() method of the ConfigInspectorManager class.

Status: Needs review » Needs work

The last submitted patch, 16: 2684383-16.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

vuil’s picture

Status: Needs work » Needs review
FileSize
7.85 KB

Fix the tests issue.

Status: Needs review » Needs work

The last submitted patch, 18: 2684383-18.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

vuil’s picture

Status: Needs work » Needs review
FileSize
6.41 KB

Status: Needs review » Needs work

The last submitted patch, 20: 2684383-20.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

vuil’s picture

Status: Needs work » Needs review
FileSize
6.58 KB

Status: Needs review » Needs work

The last submitted patch, 22: 2684383-22.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

vuil’s picture

Status: Needs work » Active
vuil’s picture

Status: Active » Needs review
FileSize
6.04 KB
vuil’s picture

Status: Needs review » Patch (to be ported)

  • ilchovuchkov committed f018b0b on 8.x-1.x
    Issue #2684383 by ilchovuchkov, gábor-hojtsy, lussoluca, lolandese,...
vuil’s picture

Status: Patch (to be ported) » Fixed
idebr’s picture

+++ b/src/Controller/ConfigInspectorController.php
@@ -148,7 +152,11 @@ class ConfigInspectorController extends ControllerBase {
+          'download' => [
+            '#markup' => Link::createFromRoute($this->t('Download'), new Url('config_inspector.download', ['name' => $name]))
+          ],

This results in a Fatal error: Link could not be converted to string. A patch to correct this renderable array is available at #3084666: Recoverable fatal error: Object of class Drupal\Core\Link could not be converted to string

vuil’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.