mahara-contributors team mailing list archive

[Mahara Bot <1946616@xxxxxxxxxxxxxxxxxx>]

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

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: mahara-contributors
https://bugs.launchpad.net/bugs/1946616

Title:
  MySQL8 syntax error - reserved word 'lock'

Status in Mahara:
  Confirmed

Bug description:
  Release testing 21.10:
  master
  MySQL 8
  Firefox
  Maroon theme
  PHP 7.4

  To test:

  1. Create an institution template collection (set to 'Current auto-copied template') with portfolio completion.
  2. Add 1 review block, type a review statement and set 'Lock portfolio' to 'Yes'.
  3. Create a new account and run the cron so that the portfolio is created.
  4. Share the portfolio with a person who can do a review.
  5. Log in as the reviewer and approve the review statement.

  Expected: box should be able to be checked and portfolio locked

  Actual: Does not allow you to check the box. On page reload, sql
  syntax error appears.

  On investigation, the problem was found to be the name of the column
  in the collection table 'lock' - this is a reserved word in MySQL8.
  When I changed the column name to 'locked' this fixed the problem.

  Carrying on through the test scenario:
  6. As portfolio author, check that the portfolio is locked. You can't delete the portfolio.
  7. Wait for 6 months ;-) or rather manipulate the locking date. (By running UPDATE cron SET nextrun = null WHERE callfunction = 'unlock_collections_by_rollover';)
  8. Run the cron again and the portfolio should be unlocked.

  Expected result: portfolio is unlocked
  Actual result: sql error (probably also to do with the 'lock' column name.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1946616/+subscriptions