Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

[statistics-imaging stats] Fix: Imaging Statistics fatal type error in stats_mri#10364

Merged
driusan merged 1 commit intoaces:main from
arnav-makkar:fix/10272-imaging-stats
Feb 23, 2026
Merged

[statistics-imaging stats] Fix: Imaging Statistics fatal type error in stats_mri #10364
driusan merged 1 commit intoaces:main from
arnav-makkar:fix/10272-imaging-stats

Conversation

@arnav-makkar
Copy link
Contributor

@arnav-makkar arnav-makkar commented Feb 13, 2026

Brief summary of changes

This PR fixes the error on Reports -> Statistics -> Imaging Statistics.

Stats_MRI::renderStatsTable() requires argument $data to be an array, but a LORIS\Database\Query object was being passed from setup().

Updated modules/statistics/php/stats_mri.class.inc to convert the pselect() result to an array using iterator_to_array(...) before passing it to renderStatsTable().

Testing

Before:
Screenshot 2026年02月13日 at 13 13 41

After:
Screenshot 2026年02月13日 at 13 03 22

Link(s) to related issue(s)

@github-actions github-actions bot added Language: PHP PR or issue that update PHP code Module: statistics PR or issue related to statistics module labels Feb 13, 2026
@kongtiaowang kongtiaowang added the Passed manual tests PR has been successfully tested by at least one peer label Feb 17, 2026
@driusan driusan merged commit 3d02ebd into aces:main Feb 23, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kongtiaowang kongtiaowang kongtiaowang approved these changes

Labels

Language: PHP PR or issue that update PHP code Module: statistics PR or issue related to statistics module Passed manual tests PR has been successfully tested by at least one peer

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[statistics - imaging statistics] 500 when acceding this tab

AltStyle によって変換されたページ (->オリジナル) /