- ๐ฎ๐ณIndia sourav_paul Kolkata
I also getting the same issue in Drupal 10.0.8.
- Assigned to PrabuEla
- ๐ฎ๐ณIndia PrabuEla chennai
Hi @All,
I am not sure this patch will resolve the issue.
The issue is because of passing empty string in preg_quote() - Issue was unassigned.
- Status changed to Postponed: needs info
over 1 year ago 11:24am 25 May 2023 - last update
about 1 year ago Custom Commands Failed - last update
about 1 year ago Custom Commands Failed - last update
about 1 year ago Custom Commands Failed - ๐จ๐ญSwitzerland sir_squall
Hi,
i just tested the patch for 10.2.0 and 10.2.1 and it's working weel.
Thanks
- ๐ณ๐ฑNetherlands diddism
Patch doesn't work if system.site page.front is null. I don't know what lead to the circumstance it was null in my case, i suspect config_filter/config_ignore, but it was. Solved it by doing
drush cset system.site page.front /node
- ๐ณ๐ฟNew Zealand quietone
This issue is filed against a component that has been removed from Drupal core a long time ago and thus is not seen by the community. I am changing the component.
Steps to reproduce were asked for 2 years ago and they have not been supplied. What is needed is how to recreated the problem from a fresh install of Drupal 10 or 11.
Since we need more information to move forward with this issue, I am keeping the status at Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
- ๐จ๐ญSwitzerland sir_squall
I still got this error, when we are using the :
$node = \Drupal::routeMatch()->getParameter('node');in this hook:
_preprocess_html(&$variables) {we have this preg_quote error
- ๐ฒ๐ฆMorocco b.khouy ๐ฒ๐ฆ Morocco
This issue stems from the use of
$this->getFrontPagePath()
as the first parameter of thepreg_quote()
function inweb/core/lib/Drupal/Core/Path/PathMatcher::matchPath
method. We need to ensure that the first argument is always a string. In the attached patch, Iโve added a check to validate the returned value accordingly - ๐บ๐ธUnited States smustgrave
Fixes should be in MRs
Issue summary should be updated with reliable steps to reproduce (moving to PNMI for that)
Also rest of sections should be filled in
Possibly will need test coverage.
- ๐จ๐ญSwitzerland sir_squall
I just tried the patch in #21 and it's working well
@sir_squall: This issue needs to be updated as specified in comment 23. Anything you can do on that end will help this to be solved.
- ๐จ๐ญSwitzerland sir_squall
I found another thread with a Similar issue:
https://www.drupal.org/project/domain/issues/3295160 โI don't know how to do the MRs..