Problem/Motivation

drupal 9 is eol in 2 days. Need to upgrade this module

Proposed resolution

This patch satisfies drupal upgrade status

Remaining tasks

1. Test in 10

CommentFileSizeAuthor
#4 10x-lti_tool-upgrade-status-3397514-4.patch30.26 KBel1_1el
#3 10x-lti_tool-upgrade-status-3397514-3.patch19.6 KBel1_1el
10x-lti_tool-upgrade-status.patch19.6 KBel1_1el

Comments

el1_1el created an issue. See original summary.

el1_1el’s picture

el1_1el commented
Issue summary: View changes
el1_1el’s picture

el1_1el commented
Issue summary: View changes
StatusFileSize
new 10x-lti_tool-upgrade-status-3397514-3.patch19.6 KB

Looks like i should get the query before adding the access check. doh! been a long month. new patch attached

el1_1el’s picture

el1_1el commented
StatusFileSize
new 10x-lti_tool-upgrade-status-3397514-4.patch30.26 KB

Symfony\Component\EventDispatcher\EventDispatcherInterface swapped args on the dispatch() method (object, string) instead of (string, object). That causes a fatal error in 10.

Attached patch fixes that and seems to work for me in canvas

kenianbei’s picture

kenianbei commented

I will have time to get a release out with this patch sometime in the next month. I no longer actively work on this project so if someone wants to step up as maintainer let me know.

kenianbei’s picture

kenianbei commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.