Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

issues #81: Resolve database schema issue #82

Open
wants to merge 1 commit into
base: MOODLE_401_STABLE
Choose a base branch
from

Conversation

ScottVerbeek
Copy link

Noticed the current id column has been used a foreign key. This caused a database schema error. Below you can see the before and after upgrade results.

root@2886180ea26f:/var/www/vanilla-401# php admin/cli/check_database_schema.php 
-------------------------------------------------------------------------------
block_grade_me
 * Missing index 'id' (not unique (id)). 
CREATE INDEX mdl_blocgradme_id_ix ON mdl_block_grade_me (id);
-------------------------------------------------------------------------------
root@2886180ea26f:/var/www/vanilla-401# php admin/cli/upgrade.php 
== Upgrading Moodle database from version 4.1.8 (Build: 20231222) (2022112808) to 4.1.8 (Build: 20231222) (2022112808) ==

Your Moodle files have been changed, and you are about to automatically
upgrade your server to this version:

4.1.8 (BUILD: 20231222) (2022112808)

Once you do this you can not go back again. Please note that this process
can take a long time.

Are you sure you want to upgrade this server to this version?

type y (means yes) or n (means no)
: y
-->block_grade_me
++ 2024013100: Success (0.09 seconds) ++
++ Success (0.16 seconds) ++
-->upgrade_noncore()
++ Success (0.39 seconds) ++
== Setting new default values ==
Command line upgrade from 4.1.8 (Build: 20231222) (2022112808) to 4.1.8 (Build: 20231222) (2022112808) completed successfully.
root@2886180ea26f:/var/www/vanilla-401# php admin/cli/check_database_schema.php 
Database structure is ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant