[MDL-66667] Cache course image in the course_summary_exporter - Moodle Tracker

XML

Word

Printable

Details

Type: Improvement
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: Future Dev
Fix Version/s: 3.11
Component/s: Course
Labels:
- CatalystIT
- ci
- partner
- release_notes
- triaged

Fixed Branches:
MOODLE_311_STABLE
Pull from Repository:
https://github.com/dmitriim/moodle
Pull 3.11 Branch:
~~MDL-66667~~-cache-course-image-MOODLE_311_STABLE
Pull 3.11 Diff URL:
https://github.com/moodle/moodle/compare/MOODLE_311_STABLE...dmitriim:MDL-66667-cache-course-image-MOODLE_311_STABLE
Pull Master Branch:
~~MDL-66667~~-cache-course-image-master
Pull Master Diff URL:
https://github.com/moodle/moodle/compare/master...dmitriim:MDL-66667-cache-course-image-master
Testing Instructions:
Hide

Create course 1 and add an image to course summary files.

Create course 2 without any summary files.

Enrol yourself on the courses with any role

Make sure that course images correctly displayed in the My overview block.

Change the course image for course 1 and course 2

Go to the dashboard.

*Confirm *that both courses show the correct course images.

Remove the course image for course 1.

Go back to the dashboard.

*Confirm *that course 1's image now displays a pattern image.
Show
Create course 1 and add an image to course summary files. Create course 2 without any summary files. Enrol yourself on the courses with any role Make sure that course images correctly displayed in the My overview block. Change the course image for course 1 and course 2 Go to the dashboard. *Confirm *that both courses show the correct course images. Remove the course image for course 1. Go back to the dashboard. *Confirm *that course 1's image now displays a pattern image.

Description

On big moodle instances could be very expensive to build a course image every time you'd need it. E.g. on a dashboard of every user when you are using my_overview block.

Would be good to have this image cached at some point.

Things to think of

Logic of cache invalidation

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

MDL-66667.jpg
35 kB
17/Mar/21 1:08 PM

Issue Links

will be (partly) resolved by

MDL-70368 Improve performance of get_imageinfo

Closed

will help resolve

MDL-63819 Reduce the performance impact of checking if a course has overview files

Closed

Activity

People

Assignee:: Dmitrii Metelkin

Reporter:: Dmitrii Metelkin

Peer reviewer:: Brendan Heywood

Integrator:: Jun Pataleta

Tester:: Anna Carissa Sadia

Participants:: Anna Carissa Sadia, Brendan Heywood, CiBoT, Dan Marsden, Dmitrii Metelkin, Eloy Lafuente (stronk7), Jun Pataleta, Matt Porritt

Component watchers:: Amaia Anabitarte, Carlos Escobedo, Laurent David, Mikel Martín Corrales, Sabina Abellan, Sara Arjona (@sarjona)

Votes:: 0 Vote for this issue

Watchers:: 11 Start watching this issue

Dates

Created:: 13/Sep/19 9:28 AM

Updated:: 07/May/21 6:30 PM

Resolved:: 19/Mar/21 9:31 PM

Fix Release Date:: 17/May/21

Time Tracking

Estimated:

Remaining:

Logged:

6h 30m