ILIAS icon indicating copy to clipboard operation
ILIAS copied to clipboard

fix for 31090: slow SessionOverviewTableGUI

Open okaufman opened this issue 3 years ago • 5 comments

Hello This change should fix this issue: https://mantis.ilias.de/view.php?id=31090 If you have a course with many sessions and you click on the tab "members" and then the subtab "sessions" you are supposed to get a view that shows which course-member participated in which session. If there are many users and many sessions in a course, loading the page takes too long and we run into a gateway timeout.

Instead of creating a new ilEventsParticipants-object, I use the static method _getParticipated. This is faster because the constructor of ilEventsParticipants looks up a lot of information, that is not needed in this context.

I added a key to the array $user_ids in ilEventParticipants so that I can use array_key_exists instead of in_array. This makes it about 7 times faster.

I tested this change in a course with 700 sessions and 400 participants. Without the change loading the site took more than 10 minutes. With this change it only took 7 seconds.

Thank you and goodbye

okaufman avatar Jul 20 '21 13:07 okaufman

Hi @smeyer-ilias

this came up again in our last TB meeting on the 19-10-2021. Is there anything new regarding this PR? Is there anything we could do to speed up the process?

Best regards!

chfsx avatar Oct 19 '21 10:10 chfsx

Hi @smeyer-ilias

we stumbled upon this PR in our TB meeting. Any updates on this?

Best regards!

mjansenDatabay avatar Feb 08 '22 12:02 mjansenDatabay

Hi @smeyer-ilias

we stumbled upon this PR in our TB meeting. Any updates on this?

Best regards!

kergomard avatar Jun 28 '22 10:06 kergomard

Hi @smeyer-ilias

we stumbled upon this PR in our TB meeting. Any updates on this?

Best regards!

Amstutz avatar Aug 09 '22 10:08 Amstutz

Sorry, for the delay. I will check and test this PR after my vacation.

smeyer-ilias avatar Aug 12 '22 11:08 smeyer-ilias