🇮🇳India @gurbakshish

#5 works fine for me, Thanks

#2 Patch worked for me Thanks

#2 Perfect, This patch worked for me Thanks!

I have verified and this MR is working fine. Thanks

#4 patch is not working for the anonymous user, but it works well for authenticated user.

I tested the MR, and space issue has been fixed. Thanks

I have reviewed the issue and MR is working fine at my end. Thanks

Patch worked for me. Thanks

#9 Patch works fine for me, Thanks

Tested this patch on 8.4.x. Getting this error ->

Could not apply patch! Skipping. The error was: Cannot apply patch https://www.drupal.org/files/issues/2023-05-16/3360635-2.patch →

I tried to change it directly and it works in header.css

#2 patch looks good, thanks

#11 patch works for me. Thanks

I changed the Route Interface and it worked for me.

Thanks

gurbakshish → created an issue.

Upgrading the Module to 4.0.2 will fix this issue.

#11 looks good now and works well. Thanks

#2 patch works fine for me. Thanks

#14 patch works fine for me. Thanks

#2 patch works fine for me, Thanks

I checked the #32 patch and it works well.

I have tried to configure the PHP info by this way under modules/system/system.admin.inc:

function system_php() {
$phpinfo_flags = variable_get('sa_core_2023_004_phpinfo_flags');
$phpinfo_flags = empty($phpinfo_flags) ? INFO_ALL : $phpinfo_flags;
phpinfo($phpinfo_flags);
drupal_exit();
}

In our default.settings.php we can add some comments like:

/**
* Configuration for phpinfo() admin status report.
*
* Drupal's admin UI includes a report at admin/reports/status/php which shows
* the output of phpinfo(). The full output can contain sensitive information
* so by default Drupal removes some sections.
*
* This behaviour can be configured by setting this variable to a different
* value corresponding to the flags parameter of phpinfo().
*
* If you need to expose more information in the report - for example to debug a
* problem - consider doing so temporarily.
*
* INFO_GENERAL The configuration line, php.ini location, build date, Web Server, System and more.
* INFO_CREDITS PHP Credits. See also phpcredits().
* INFO_CONFIGURATION Current Local and Master values for PHP directives. See also ini_get().
* INFO_MODULES Loaded modules and their respective settings. See also get_loaded_extensions().
* INFO_ENVIRONMENT Environment Variable information that's also available in $_ENV.
* INFO_VARIABLES Shows all predefined variables from EGPCS (Environment, GET, POST, Cookie, Server).
* INFO_LICENSE PHP License information. See also the » license FAQ.
* INFO_ALL Shows all of the above.
*
* By default INFO_ALL is enabled.
* You can use above parameters to configure the phpinfo for example:
* (INFO_ENVIRONMENT | INFO_MODULES)
* For More information check the link below.
* @see https://www.php.net/manual/function.phpinfo.php
*/
#$conf['sa_core_2023_004_phpinfo_flags'] = (INFO_ALL);

gurbakshish → created an issue.

This is not bug, you need to create comment type for each entity where you want to add comments

This patch can remove the files permanently, when we click on remove button when file is already uploaded.