Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor needed post-removal of Gemini #2

Closed
noahwsmith opened this issue May 4, 2021 · 8 comments · Fixed by #6
Closed

Refactor needed post-removal of Gemini #2

noahwsmith opened this issue May 4, 2021 · 8 comments · Fixed by #6

Comments

@noahwsmith
Copy link

Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException: You have requested a non-existent service "islandora.gemini.lookup". in Drupal\Component\DependencyInjection\Container->get() (line 153 of /var/www/drupal/web/core/lib/Drupal/Component/DependencyInjection/Container.php).

We're running https://github.com/Islandora/islandora/tree/8.x-1.x after acceptance of Islandora/islandora#822 so my suspicion is that islandora_premis needs to have references to Gemini removed. I saw @dannylamb and @ruebot discussing a similar issue yesterday in the #islandora8 slack channel.

@mjordan
Copy link
Owner

mjordan commented May 4, 2021

I will fix. Thanks for reporting.

mjordan added a commit that referenced this issue May 5, 2021
@mjordan mjordan closed this as completed in 66bcc73 May 5, 2021
@noahwsmith
Copy link
Author

Notes on testing: I updated our module and now are getting a RDF/Graph error:

Error: Class 'EasyRdf\Graph' not found in islandora_premis_islandora_premis_turtle_alter() (line 32 of /var/www/drupal/assets/sites/all/modules/custom/islandora_premis/islandora_premis.module)
#0 [internal function]: islandora_premis_islandora_premis_turtle_alter('53', NULL)
#1 /var/www/drupal/web/core/lib/Drupal/Core/Extension/ModuleHandler.php(403): call_user_func_array('islandora_premi...', Array)
#2 /var/www/drupal/assets/sites/all/modules/custom/islandora_premis/src/Controller/IslandoraPremisPremisController.php(26): Drupal\Core\Extension\ModuleHandler->invokeAll('islandora_premi...', Array)
#3 [internal function]: Drupal\islandora_premis\Controller\IslandoraPremisPremisController->main()
#4 /var/www/drupal/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php(123): call_user_func_array(Array, Array)
#5 /var/www/drupal/web/core/lib/Drupal/Core/Render/Renderer.php(573): Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}()
#6 /var/www/drupal/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php(124): Drupal\Core\Render\Renderer->executeInRenderContext(Object(Drupal\Core\Render\RenderContext), Object(Closure))
#7 /var/www/drupal/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php(97): Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->wrapControllerExecutionInRenderContext(Array, Array)
#8 /var/www/drupal/vendor/symfony/http-kernel/HttpKernel.php(151): Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}()
#9 /var/www/drupal/vendor/symfony/http-kernel/HttpKernel.php(68): Symfony\Component\HttpKernel\HttpKernel->handleRaw(Object(Symfony\Component\HttpFoundation\Request), 1)
#10 /var/www/drupal/web/core/lib/Drupal/Core/StackMiddleware/Session.php(57): Symfony\Component\HttpKernel\HttpKernel->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)

@mjordan
Copy link
Owner

mjordan commented May 6, 2021

Can you clear your cache to see if that helps?

@noahwsmith
Copy link
Author

noahwsmith commented May 6, 2021 via email

@mjordan
Copy link
Owner

mjordan commented May 6, 2021

Yes, that would be a good idea. Can you open a PR or do you want me to take care of it? I'm not getting that error so my environment is presumably different from yours.

@mjordan mjordan reopened this May 7, 2021
@mjordan
Copy link
Owner

mjordan commented May 7, 2021

I've added the class aliases to the "add_easyrdf_alias" branch. Can you test?

@noahwsmith
Copy link
Author

Works GREAT- thank you!

@mjordan
Copy link
Owner

mjordan commented May 8, 2021

Thanks for testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants