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

Imaging browser - View session broken on major #4068

Closed
cmadjar opened this issue Nov 6, 2018 · 1 comment
Closed

Imaging browser - View session broken on major #4068

cmadjar opened this issue Nov 6, 2018 · 1 comment
Assignees
Labels
Category: Bug PR or issue that aims to report or fix a bug

Comments

@cmadjar
Copy link
Collaborator

cmadjar commented Nov 6, 2018

Get a 500 error when loading the viewSession page on the major branch with the following error message in loris-error.log (everything OK in minor):

[Tue Nov 06 10:00:59.868959 2018] [php7:error] [pid 19970] [client 142.157.131.89:56401] PHP 
Fatal error:  Uncaught DomainException: Attempt to use pselectRow on a query that returns 
multiple rows in /var/www/loris/php/libraries/Database.class.inc:764\nStack trace:\n#0 
/var/www/loris/modules/imaging_browser/php/imaging_session_controlpanel.class.inc(83): 
Database->pselectRow('SELECT f.Commen...', Array)\n#1 
/var/www/loris/modules/imaging_browser/php/imaging_session_controlpanel.class.inc(183): 
LORIS\\imaging_browser\\Imaging_Session_ControlPanel->getData()\n#2 
/var/www/loris/modules/imaging_browser/php/viewsession.class.inc(741): 
LORIS\\imaging_browser\\Imaging_Session_ControlPanel->display()\n#3 
/var/www/loris/src/Middleware/UserPageDecorationMiddleware.php(178): 
LORIS\\imaging_browser\\ViewSession->getControlPanel()\n#4 
/var/www/loris/src/Middleware/PageDecorationMiddleware.php(49): 
LORIS\\Middleware\\UserPageDecorationMiddleware->process(Object(Zend\\Diactoros\\ServerRequest), 
Object(LORIS\\imaging_browser\\ViewSession))\n#5 
/var/www/loris/php/libraries/NDB_Page.class.inc(650): 
LORIS\\Middleware\\PageDecorationMiddleware->process(O in 
/var/www/loris/php/libraries/Database.class.inc on line 764, referer: https://cecile-dev.loris.ca/imaging_browser/

@davidblader I assigned it to you as I think you are or have been working on the View Session page of the imaging browser lately. Feel free to change the assignee if you can't fix this.

@cmadjar cmadjar added the Category: Bug PR or issue that aims to report or fix a bug label Nov 6, 2018
driusan pushed a commit that referenced this issue Nov 7, 2018
…ments query (#4071)

A query in the imaging browser was throwing exceptions due to there being entries for both initial and double data entry using pselectRow. This fixes it by excluding DDE entries.

Fixes #4068.
@cmadjar
Copy link
Collaborator Author

cmadjar commented Nov 7, 2018

PR was merged into major so closing the issue. Thanks @davidblader for fixing it so quickly!! :D

@cmadjar cmadjar closed this as completed Nov 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Bug PR or issue that aims to report or fix a bug
Projects
None yet
Development

No branches or pull requests

2 participants