[site:status] Configuration directory with unescaped string.

[GuidoRobertone]

Problem/Motivation

The result of site status command returns an unescaped config directory string when the settings.php file was edit.

How to reproduce

1. Set config directory as $config_directories['sync'] = '../config/sync';
   
2. run drupal site:status
3. An unescaped string is shown (Configuration directory: sync)
   

Details to include:

• Drupal version : 8.5.6
• Console version : 1.8.0
• Console Launcher version : 1.8.0

Solution

A correct directory string will be displayed using strip_tags() function.

[LOBsTerr]

@GuidoRobertone First of all I have checked the code

drupal-console/src/Command/Site/StatusCommand.php

Line 158 in 912508f

$systemData['system'][$title] = strip_tags($requirement['value']);

Tags striping is there.
I have tried to reproduce this, but unfortunately I'm not able.
When I run this command I don't see Configuration directory information there.
I even installed lighting profile and still it is not there.
The only difference there I have Drupal 8.6.1

[LOBsTerr]

@GuidoRobertone Ok, After debugging how Drupal displays system requirements. I was able to reproduce it. Actually, the config directory should exist in order to see in system requirements. In my case these folders were there. OK, now I see and I will debug it.

[LOBsTerr]

PR - #3958
ok, I wrapped titles to strip_tags and it will solve the current issue, but we still issue with data now there will be no value for Configuration directory, because of a bug I have just found

https://www.drupal.org/project/drupal/issues/3007260

I think we also should display description as additional information. I need to find to present it nice way.
Anyway, it could be done in a separate ticket

[LOBsTerr]

@GuidoRobertone I also added description information, like this users will see more additional information.

[LOBsTerr]

Merged