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.

Comments

lussoluca created an issue. See original summary.

lussoluca’s picture

Status: Active » Needs review
StatusFileSize
new3.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

Status: Needs work » Needs review
StatusFileSize
new3.4 KB

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
StatusFileSize
new3.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
StatusFileSize
new7.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
StatusFileSize
new7.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
StatusFileSize
new7.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
StatusFileSize
new6.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
StatusFileSize
new6.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
StatusFileSize
new6.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.