🇮🇳India @Shifali Baghel

Gurgaon
Account created on 17 May 2018, about 6 years ago
#

Recent comments

🇮🇳India Shifali Baghel Gurgaon

I got the solution.

I changed my BLT version from 13.4.0 to 13.7.2 and it worked fine.

"acquia/blt": "^13.7.2"

🇮🇳India Shifali Baghel Gurgaon

Hi all,

I'm getting the same issue after updating my Drupal Core 9.5.10 and Acquia CMS ^2.1.

Fatal error: Declaration of Acquia\Blt\Robo\Common\EnvironmentDetector::getSiteName($site_path) must be compatible with Acquia\DrupalEnvironmentDetector\AcquiaDrupalEnvironmentDetector::getSiteName(string $site_path): ?string in /var/www/html/vendor/acquia/blt/src/Robo/Common/EnvironmentDetector.php on line 236

The attached patch doesn't work because we haven't been using this "acsf" module.
If I apply the below code in my root composer.json file, that doesn't work at all. Quick help would be appreciated.

"drush": {
"services": {
"drush.services.yml": "^9 || ^10"
}
},

🇮🇳India Shifali Baghel Gurgaon

can anyone please help to elaborate how we can reproduce this deprecated issues / warning on local?

🇮🇳India Shifali Baghel Gurgaon

After the PHPCS bugs and the spelling error were fixed, the patch is added once again for the time being.

🇮🇳India Shifali Baghel Gurgaon

Hi All,

In order to make this change in Drupal 11's development version, I added this patch.
Here, the aggregation ccs/Js fields have just been transferred from the System Performance form to the Development settings form. Because many core files have been utilising this same system performance config, we have saved the configuration in system.performance itself to make things simpler.

Here is the patch.

🇮🇳India Shifali Baghel Gurgaon

Hi @erdm

Thank you so much for taking the time to check this out.

If you could let us know the version you tried this patch on, that would be useful. Despite I had experienced these problems with Drupal 9.5 as well, I yet provided a patch to address them. Please let us know if it doesn't work in Drupal 9.5.9.

🇮🇳India Shifali Baghel Gurgaon

I checked this module in the Drupal 9.5.9 and add config dependencies to each dynamic permission. Here is a patch.

🇮🇳India Shifali Baghel Gurgaon

Hi All,

Thanks for the propose solutions.

The issues was when n/a getting Translated in the ConfigTranslationFormBase.php, the object was getting passed in the __construct, due to that, is_string function was not getting true and throwing error.

Here we have added the String keyword to convert TranslatableMarkup object to string in ConfigTranslationFormBase.php Here is the patch.

Tested on Drupal version: 9.5.9

Thanks & Regards
Shifali Baghel

🇮🇳India Shifali Baghel Gurgaon

Hi,

To remove unneeded variables from the file.module, I attempted this patch.

🇮🇳India Shifali Baghel Gurgaon

Hi Josh Waihi @fiasco, hope you are doing well.

I saw a few things, and the problem was with the two YML files for a single form. In my view, each configuration form should have its own configuration file. Could this separate module's configuration (http_cache_control.settings.yml) be moved to this "system.performance.yml" file?
Please share your opinions with me on this.

Thank you

Production build 0.69.0 2024