search

catalyst / moodle-local_smartmedia

Moodle LMS Smartmedia local plugin
Other
7 stars 5 forks source link

issue220: Instead of fetching all area files, filter the results by the pathname #221

Closed andrewmadden closed 1 year ago

andrewmadden commented 1 year ago

The number of area files can number more than 100,000 which is impractical to load in memory before being filtered. They should be filtered directly in the SQL. This was causing a performance issue with page loads taking excess of 30 seconds.

andrewmadden commented 1 year ago

Making a note that unit tests already cover this method. Confirmed they are proficient to make sure the behaviour hasn't changed.