- Issue created by @leducdubleuet
- First commit to issue fork.
- Merge request !6Issue #3455511 by LeDucDuBleuet: LogicException: Service... β (Merged) created by mariacha1
- Status changed to Needs review
5 months ago 7:57pm 20 June 2024 - πΊπΈUnited States mariacha1
It looks like during coding standards cleanup at https://git.drupalcode.org/project/rest_api_authentication/-/commit/4121... the name of the service was changed from `restAPI` to `RestAPI` without renaming the file to `RestAPI.php`. Trying to fix, I ran into issues where my git was ignoring capitalization changes in file names.
I've opened a MR for this with the diff provided here:
https://git.drupalcode.org/project/rest_api_authentication/-/merge_reque...The diff makes it look like the new file `src/Authentication/Provider/RestAPI.php` is being created without deleting `src/Authentication/Provider/restAPI.php` so let's see if that works.
- Status changed to RTBC
5 months ago 6:06pm 21 June 2024 - π¨π¦Canada leducdubleuet Chicoutimi QC
I tested the proposed patch in the merge request !6 and I can confirm it is working well.
Version 2.0.14 works with this patch but it would be great to have a new release please.
Thank you.
-
shashank_thigale β
committed 7639886d on 2.0.x authored by
mariacha1 β
Issue #3455511 by LeDucDuBleuet: LogicException: Service...
-
shashank_thigale β
committed 7639886d on 2.0.x authored by
mariacha1 β
- Status changed to Fixed
5 months ago 6:31am 24 June 2024 I have merged the PR and fixed other filename issues as well in the latest release (2.0.16)
- π¨π¦Canada leducdubleuet Chicoutimi QC
@shashank_thigale
Looks like you missed one when visiting the config page, we now get :
InvalidArgumentException: Class "\Drupal\rest_api_authentication\Form\MiniOrangeAPIAuth" does not exist. in Drupal\Core\DependencyInjection\ClassResolver->getInstanceFromDefinition() (line 37 of core/lib/Drupal/Core/DependencyInjection/ClassResolver.php).
Sorry for asking but are you testing anything at all before doing a release?
Automatically closed - issue fixed for 2 weeks with no activity.