Portfolio completion page not accessible when coming via LTI

Bug #1965061 reported by Kristina Hoeppner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Robert Lyon
21.04
Fix Released
High
Unassigned
21.10
Fix Released
High
Unassigned
22.04
Fix Released
High
Robert Lyon

Bug Description

Noticed that in Mahara 21.10 but not 20.10.

When you use the Mahara assignment submission plugin web services and have a portfolio with the portfolio completion page, then you can't open that portfolio from the assignment any more.

To replicate:

1. In Mahara, create an institution that allows portfolio completion.
2. Set that institution up with LTI to Moodle.
3. Set up a Mahara assignment submission in Moodle as teacher.
4. As student, log into Moodle and go to the assignment, creating a portfolio with portfolio completion and two pages. The pages don't need to have any blocks.
5. Submit your portfolio into the assignment.
6. As teacher, go to the grading view for the assignment and click the link to the portfolio.

Results:
- Expected: You can click through and see the portfolio completion page.
- Actual: You get an 'Access denied' alert.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "main" branch: https://reviews.mahara.org/12500

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/12500
Committed: https://git.mahara.org/mahara/mahara/commit/4b9dd2fc5508d1fe94fee88b83e73dc933df6e9f
Submitter: Robert Lyon (<email address hidden>)
Branch: main

commit 4b9dd2fc5508d1fe94fee88b83e73dc933df6e9f
Author: Robert Lyon <email address hidden>
Date: Wed Mar 16 10:35:48 2022 +1300

Bug 1965061: Need to include the progress view id in check

Now that progress page has an actual page (view) on it

Also adjusted the lti module to do one of it's checks the same way as the
lit_advantage module

Change-Id: If0083e238618c96c822c36a895bb1b5021b89d1b
Signed-off-by: Robert Lyon <email address hidden>

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "21.10_DEV" branch: https://reviews.mahara.org/12501

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "21.04_DEV" branch: https://reviews.mahara.org/12502

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/12501
Committed: https://git.mahara.org/mahara/mahara/commit/448b530a82d4296ca60584aa4d0209bc38c73a5a
Submitter: Robert Lyon (<email address hidden>)
Branch: 21.10_DEV

commit 448b530a82d4296ca60584aa4d0209bc38c73a5a
Author: Robert Lyon <email address hidden>
Date: Wed Mar 16 10:35:48 2022 +1300

Bug 1965061: Need to include the progress view id in check

Now that progress page has an actual page (view) on it

Also adjusted the lti module to do one of it's checks the same way as the
lit_advantage module

Change-Id: If0083e238618c96c822c36a895bb1b5021b89d1b
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit 4b9dd2fc5508d1fe94fee88b83e73dc933df6e9f)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/12502
Committed: https://git.mahara.org/mahara/mahara/commit/d4491c24c7f9b532a53a000effa01f6751880b49
Submitter: Robert Lyon (<email address hidden>)
Branch: 21.04_DEV

commit d4491c24c7f9b532a53a000effa01f6751880b49
Author: Robert Lyon <email address hidden>
Date: Wed Mar 16 10:35:48 2022 +1300

Bug 1965061: Need to include the progress view id in check

Now that progress page has an actual page (view) on it

Also adjusted the lti module to do one of it's checks the same way as the
lit_advantage module

Change-Id: If0083e238618c96c822c36a895bb1b5021b89d1b
Signed-off-by: Robert Lyon <email address hidden>

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.