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

ErrorException thrown with message "Class srag\DIC\OpenCast\Database\DatabaseDetector #187

Closed
reiferschris opened this issue Aug 23, 2023 · 4 comments · Fixed by #186
Closed

Comments

@reiferschris
Copy link
Contributor

Hi @mjansenDatabay
in which version of the plugin did this error occur for you? Also which version or better commit of ilias itself did you use?

sorry, maybe I was wrong. A colleague of mine sent me the following error ...

root@octa04-vm01:/srv/www/my.ilias.de/admin/ilias_errors# cat 83f24_1477.log
Whoops\Exception\ErrorException thrown with message "Class srag\DIC\OpenCast\Database\DatabaseDetector contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (ilDBInterface::primaryExistsByFields)"

Stacktrace:
#2 Whoops\Exception\ErrorException in /srv/www/my.ilias.de/ilias/Customizing/global/plugins/Services/Repository/RepositoryObject/OpenCast/vendor/srag/dic/src/Database/DatabaseDetector.php:19
#1 Whoops\Run:handleError in /srv/www/my.ilias.de/ilias/libs/composer/vendor/filp/whoops/src/Whoops/Run.php:514
#0 Whoops\Run:handleShutdown in [internal]:0


-- GET Data --

ref_id                   : 31
admin_mode               : settings
ctype                    : Services
cname                    : Repository
slot_id                  : robj
pname                    : OpenCast
cmd                      : refreshLanguages
cmdClass                 : ilobjcomponentsettingsgui
cmdNode                  : 1d:m0
baseClass                : ilAdministrationGUI

... and pointed me to THIS issue here. Bug maybe this is a different problem. If yes, please ignore my comment.

Originally posted by @mjansenDatabay in #169 (comment)

@tmuellerdatabay
Copy link

I updated ILIAS Core and roll out the plugin yesterday afternoon fresh from repository.
Everything is up to date as well.
It is possible that improvements to ILIAS have led to incompatibility.

To reproduce it, you would have to check out both fresh.
I suspect that pure updates (minor version jumps) are not affected.

@tmuellerdatabay
Copy link

ILIAS:

  • release_7 3618457016 Merge pull request #6256 from Minervis-GmbH/_7/soap/group/container_settings

Plugin:

  • main bc9b2a1 Added Readme for Branching and Versioning

@reiferschris
Copy link
Contributor Author

Thank you for the details. Unfortunately we (Uni Köln) are also affected even with the old version of the plugin (not the community version).
Our test system also has this error when updating it to the current main branch.

@ferishili
Copy link
Contributor

I can confirm that PR #186 (i.e. removing DICTrait) solve this problem, therefore, this issue has to be linked with that PR!

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.

3 participants