- Issue created by @beedaddy
- π©πͺGermany Anybody Porta Westfalica
Thanks, I think we should remove the Drupal 10 compatibility from 8.x. Please use 2.x instead.
- Assigned to Grevil
- Status changed to Needs work
over 1 year ago 8:44am 31 March 2023 - π©πͺGermany Grevil
Yes definitely! I am not even sure why we kept it.
- @grevil opened merge request.
- Status changed to Needs review
over 1 year ago 8:50am 31 March 2023 - π©πͺGermany beedaddy
Thanks! Sorry for the stupid question:
Should/can I just runcomposer require 'drupal/captcha:^2.0@beta'
in order to upgrade? - π©πͺGermany Anybody Porta Westfalica
You can :)
But please note, that especially the template struture changed. So have a loot at the output. And make a backup before! - Status changed to RTBC
over 1 year ago 8:59am 31 March 2023 - Status changed to Needs review
over 1 year ago 9:12am 31 March 2023 - π©πͺGermany beedaddy
mhh, now, with 2.0beta, if I try to open /admin/config/people/captcha/image_captcha or if I want to clear cache I get an uncaught PHP error:
[error] 143292#143292: *101564 FastCGI sent in stderr: "PHP message: Uncaught PHP Exception ReflectionException: "Class "\Drupal\image_captcha\Controller\CaptchaFontPreviewController" does not exist" at /var/www/mysite/web/core/lib/Drupal/Core/Entity/EntityResolverManager.php line 135" while reading response header from upstream
- Issue was unassigned.
- Status changed to Fixed
over 1 year ago 9:20am 31 March 2023 - π©πͺGermany Anybody Porta Westfalica
@beedaddy could you please create a separate issue for that one? @Grevil will have a look asap.
Let's close this one fixed.
-
Anybody β
committed c196b38c on 8.x-1.x authored by
Grevil β
Issue #3351472 by Grevil, Anybody: Image not shown, PHP error
-
Anybody β
committed c196b38c on 8.x-1.x authored by
Grevil β
- π©πͺGermany Anybody Porta Westfalica
Release 8.x-1.10 without D10 compatibility.
- π©πͺGermany Grevil
@beedaddy, try clearing all caches first using "drush cr" if the issue still persists, I can have a look.
May I suggest not to remove captcha 8.x-1.10 completely from drupal 10 because it works fine combined with Captcha Riddler p. e.?
- π©πͺGermany Anybody Porta Westfalica
@w0lf, no that won't happen. Feel free to hack your compatibility locally, but we won't do. Captcha Riddler should be compatible. If not, please create an issue and describe what's wrong there. Thanks
- Status changed to Needs work
over 1 year ago 5:01pm 17 July 2023 - πΊπΈUnited States japerry KVUO
Marking needs work and bumping to the latest version of 8.x-1.x. You should -never- remove upper composer constraints from a module.
- π©πͺGermany Anybody Porta Westfalica
@japerry you're totally right. The point is, that ^10 *should never* have been added for 8.x-1.x due to the incompatibility above.
Instead, 2.x should be used for Drupal 10.I'd strongly suggest closing this again and instead continue in β¨ Difficult migration from Drupal 9 to 10 Fixed - if anyone finds a good solution for 8.x-1.x I'm happy to review the solution. I don't have one.
- Status changed to Closed: duplicate
over 1 year ago 11:04pm 17 July 2023 - πΊπΈUnited States japerry KVUO
I agree, after working on β¨ Difficult migration from Drupal 9 to 10 Fixed I think this can be re-closed. But wanted to comment here so there is some history on the d10 readiness thread.