- Issue created by @pookmish
- Status changed to Needs review
over 1 year ago 10:18pm 6 July 2023 - last update
over 1 year ago 3 pass - 🇳🇱Netherlands Ewout Goosmann
I just ran into the same issue and can confirm that this patch is working.
- Status changed to RTBC
over 1 year ago 2:18pm 7 July 2023 - 🇺🇸United States pdcarto
I can also confirm that the patch fixes the issue.
We also encountered this error on a client project. This patch worked for us. RTBC +1
- 🇬🇧United Kingdom alexharries
Just posting that this has happened on our D9 sites too.
Surely there's a more-elegant way for Migrate Tools to have made this breaking change?
- 🇬🇧United Kingdom Alina Basarabeanu
The patch works and also fix https://www.drupal.org/project/path_redirect_import/issues/3313386 🐛 Drush updb fails when upgrading from 8.x-1.0 to 2.0.5 Fixed
- 🇺🇸United States pianomansam
Patch is also working for me. A release should be created.
- First commit to issue fork.
- Merge request !21Issue #3373025: Update to support path redirect import migrate_tools 6.0.2 → (Closed) created by codebymikey
- last update
over 1 year ago 2 fail Added an issue fork implementing the proposed fix as well as enforcing the
migrate_tools >= 6.0.2
requirements, as well as minimal Drush 12 support.- Status changed to Needs review
over 1 year ago 9:25am 8 August 2023 - last update
over 1 year ago 3 pass - last update
over 1 year ago 3 pass - 🇫🇷France ericdsd France
Patch #2 solved the issue over 2.0.7 as i couldn't apply patch from #15 from composer
- 🇨🇴Colombia carma03
Confirming patch #2 🐛 Update to support path redirect import migrate_tools 6.0.2 RTBC worked on D9.5.10 and PHP 8.1.18.
Thanks @pookmish - 🇺🇸United States patrickfgoddard
Patch applied fine for me with with Drupal 9.5.10 and php 8.1.18.
- 🇺🇸United States bburg Washington D.C.
MR patch in #15 does not apply to version 2.0.7 of the module. Patch file in #2 does.
- 🇺🇸United States andyg5000 North Carolina, USA
Hi @pankaj1390. The patch is on the HEAD of the 2.0.x branch (as it should be) and doesn't apply directly to the current release. If you want to use the merge request you'll need to do `composer require drupal/path_redirect_import:dev-2.0.x`
The merge fixes my issue and includes updates to ensure migrate_tools version is pinned. The patch just changes the `use` statement. Merge looks good to me.
- Status changed to RTBC
about 1 year ago 1:11am 3 October 2023 - 🇬🇧United Kingdom Alina Basarabeanu
The pull request works as expected on Drupal Version 10.1.5 and path_redirect_import 2.0.x-dev.
Please create a new release. - 🇳🇱Netherlands ahmetburkan Utrecht
Works here as well. Is it possible to create a Drupal 10 compatible release?
- last update
about 1 year ago Composer require failure - 🇺🇸United States ddavisboxleitner
The pull request patch also worked for us with Drupal 10.1.5 and path_redirect_import 2.0.x-dev.
Having applied the patch I'm now getting this error:
Error: Class "Drupal\migrate_tools\Drush\MigrateToolsCommands" not found in include() (line 27 of *****\web\modules\contrib\path_redirect_import\src\Commands\PathRedirectImportCommands.php).
- 🇺🇸United States calebtr
@Ali_W - can you check that you have Migrate Tools v6 or greater?
- Status changed to Needs work
about 1 year ago 3:42pm 26 October 2023 - 🇺🇸United States rishi kulshreshtha
Using 2.0.x-dev, applied https://git.drupalcode.org/project/path_redirect_import/-/merge_requests... as a patch and using drupal/migrate_tools v6 but facing the exact same issue as reported at #29
- 🇩🇪Germany sfuchsbe
@Ali_W & Rishi Kulshreshtha
I was facing the similar issue and found out that it was related on how Drush 12 is implementing the discovery of classes. After downgrading to Drush 11 the issue was mitigated. Maybe I was just lucky, but maybe for you it is the same. Give it a try ;) - 🇧🇷Brazil carolpettirossi Campinas - SP
Uploading file to be used in composer as .diff is not recommended.
- 🇨🇿Czech Republic Petr Illek
It seems that this issue is solved in the latest DEV version.
For details have a look at related issue https://www.drupal.org/project/path_redirect_import/issues/3396112 📌 Drush 12 compatibility RTBC .
Please test it, so we can get the new release soon. - 🇺🇸United States ethomas08 SF Bay Area
Applied the patch from migrate_tools issue 3372643: https://www.drupal.org/files/issues/2023-11-21/3372643-13.patch →
Now need to re-roll the patch from #2 to resolve the problem. - last update
about 1 year ago Patch Failed to Apply - last update
about 1 year ago Patch Failed to Apply -
codebymikey →
authored a6756833 on 2.0.x
Issue #3373025 by codebymikey, GuillaumePacilly: Update to support path...
-
codebymikey →
authored a6756833 on 2.0.x
- Status changed to RTBC
about 1 year ago 8:52am 6 December 2023 - last update
about 1 year ago Patch Failed to Apply - 🇪🇸Spain plopesc Valladolid
Thank you for your patches and MR in this issue.
Added the necessary bits after 📌 Drush 12 compatibility RTBC so both should be ready to go.
- Status changed to Fixed
about 1 year ago 8:56am 6 December 2023 - 🇧🇪Belgium britt.vanderlinden
The above patches won't work for me.
I am working on a Drupal 10 site with Drush 11. - 🇪🇸Spain plopesc Valladolid
@britt.vanderlinden which Migrate Tools version are you using?
According to the module's repository, the commands path in 6.0.2 version is
Drupal\migrate_tools\Drush
.Thank you!
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
11 months ago 1:34pm 29 January 2024 - 🇹🇷Turkey emircan erkul Turkey
I do not have a downgrade drush 12->11 option due to Drupal >10.2.0 forces me to use Drush 12.
I'm using https://www.drupal.org/files/issues/2023-11-23/3403792-2.patch → migrate_tools patch and path_redirect_import dev version, checked everything and the paths look correct but couldn't find any reason why I still getting the same error.
```
Error: Class "Drupal\migrate_tools\Drush\Commands\MigrateToolsCommands" not found in include() (line 30 of /app/docroot/modules/contrib/path_redirect_import/src/Drush/Commands/PathRedirectImportCommands.php).
```