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

Changing ILM to non-ILM status throws 500 error #1267

Closed
saschaben opened this issue Jan 18, 2016 · 1 comment · Fixed by #1269
Closed

Changing ILM to non-ILM status throws 500 error #1267

saschaben opened this issue Jan 18, 2016 · 1 comment · Fixed by #1269
Assignees
Labels

Comments

@saschaben
Copy link
Member

while the call appears to be fully functioning to destroy the ilm_session_facet entry when you try to turn an ILM into a non-ILM, it returns a 500 error to the user:

{code: 500, message: "Internal Server Error"}
code: 500
message: "Internal Server Error" 

Thinking since this is on a session based PUT that it might be due to trying to re-save the session, rather than just deleting the ilm_session_facet...

@saschaben saschaben added the BUG label Jan 18, 2016
@stopfstedt stopfstedt self-assigned this Jan 19, 2016
@stopfstedt
Copy link
Member

seems to be two separate operations:

DELETE /api/v1/ilmsessions/1980 204 186.875 ms - 0
PUT /api/v1/sessions/26561 500 2193.164 ms - 46

from the error logs:

[2016-01-19 18:29:53] request.CRITICAL: Uncaught PHP Exception Symfony\Component\Debug\Exception\ContextErrorException: "Catchable Fatal Error: Argument 1 passed to Ilios\CoreBundle\Entity\Session::setIlmSession() must implement interface Ilios\CoreBundle\Entity\IlmSessionInterface, null given, called in /vagrant/vendor/symfony/symfony/src/Symfony/Component/PropertyAccess/PropertyAccessor.php on line 502 and defined" at /vagrant/src/Ilios/CoreBundle/Entity/Session.php line 390 {"exception":"[object] (Symfony\\Component\\Debug\\Exception\\ContextErrorException(code: 0): Catchable Fatal Error: Argument 1 passed to Ilios\\CoreBundle\\Entity\\Session::setIlmSession() must implement interface Ilios\\CoreBundle\\Entity\\IlmSessionInterface, null given, called in /vagrant/vendor/symfony/symfony/src/Symfony/Component/PropertyAccess/PropertyAccessor.php on line 502 and defined at /vagrant/src/Ilios/CoreBundle/Entity/Session.php:390)"} []

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

Successfully merging a pull request may close this issue.

2 participants