Below is the quick patch for the issue:
Yogesh Kambari → created an issue.
Added the Quick Patch. Please verify.
Yogesh Kambari → created an issue.
Hi,
Created a patch for this issue. Please review.
Hello amit.drupal
I was trying to replicate this issue on my local with drupal 10.2.0 and tvi version 2.0.0-alpha2. I am successfully able to run drush cr after enabling the tvi module.
Also noticed that with "tvi version 2.0.0-alpha2" i cant see any redeclaration of "Drupal\tvi\Enhancer\RouteEnhancer::$tviManager" in codebase.
Attaching screenshot.
Hi,
MR looks good to me. Validated on Drupal instance with version 10.0.10. Attaching screenshots for reference.
I am able to reproduce similar kind of scenario.
Like i have created two nodes with URL alias as "/redirect-1" and "/redirect-2". Now i have added redirect rule in which i have source as "/redirect-2" and destination as "/redirect-1".
Now if i hit "/redirect-2" then it doesn't get redirected to "/redirect-1".
It looks like we need to add node id ('node/nid') in redirection rule (for source node which are having some url alias).
If it is the same scenario you are facing then requesting you to try with adding node id "node/nid" in redirect rule. Though i tried below work around on my local and it works for me.
Added below line of code in file : /src/EventSubscriber/RedirectRequestSubscriber.php
@@ -136,6 +136,7 @@ class RedirectRequestSubscriber implements EventSubscriberInterface {
// Do the inbound processing so that for example language prefixes are
// removed.
$path = $this->pathProcessor->processInbound($request->getPathInfo(), $request);
+ $path = \Drupal::service('path_alias.manager')->getAliasByPath($path);
}
$path = trim($path, '/');
Tried to replicate this issue with drupal version "10.1.2-dev" and php version 8.2, I could not able to reproduce the issue. Attaching the screenshots.
Hi Shanu Chouhan,
This issue is addressed in development version "8.x-1.x-dev" of this module.
Hello efil,
Will you please add steps to reproduce the issue.
Hi,
Patch looks good to me. Validated on Drupal instance with version 10.1.2-dev. Attaching screenshots for reference.
Hi,
It has come to my attention that the problem cannot be resolved effectively without considering the dependency on the opigno_statistic module. Themes are primarily responsible for the presentation layer and should not handle module dependencies.
Here is quick fix to address above issue.
Here is the patch to resolve this issue.
Hello ady1503,
I have Install "views_data_export_phpspreadsheet" (Version: 8.x-1.5) module in Drupal 10.0.1 (language-Spanish). I couldn't able to reproduce the issue mentioned above. Though i was able to reproduce issue
https://www.drupal.org/project/views_data_export_phpspreadsheet/issues/3...
🐛
Missing return type error after D10 update
Fixed
with the same steps mentioned above.
Will you please provide more details and steps to reproduce the issue if it still exists.
Attaching a screenshot for reference.
Hi,
Tested the Patch with version "10.1.2-dev" on my local, it's working fine and resolved the issue. Attaching the screenshot before and after applying the patch.
Thanks,
I have reviewed the patch and it's resolved the issue mentioned. I have attached before and after state screenshots for reference.
Hello Fvd,
I have Install token_custom (Version: 8.x-1.0-alpha4) module in Drupal 10.0.9, I couldn't able to reproduce the issue mentioned above.
Please provide more details and steps to reproduce the issue if it still exists.
Attaching a screenshot for reference.
madhukunj,
I have Install pathauto module in Drupal 10.0.9, I couldn't able to reproduce the issue mentioned above.
Please provide more details and steps to reproduce the issue if it still exists.
Attaching a screenshot for reference.