search

themesalmond / moodle-theme_almondb

5 stars 4 forks source link

theme_almondb_frontpageblock07 does not check for course visibility and is nasty on performance #5

Closed danmarsden closed 2 years ago

danmarsden commented 2 years ago

This function seems to display all courses - even courses the user should not be able to see and is extremely in-efficient - you need to rework this function so that it does not make so many db calls. - think about an example where a site might have 10,000 courses.

themesalmond commented 2 years ago

We fixed issues. Thank you for your help.

jrsaw commented 1 year ago

I saw the same thing, I ended up commenting out that (block-7 Available Courses) and used Block-2 Icon Box for the same idea.