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

Iomad 35 stable #1131

Open
wants to merge 2,824 commits into
base: IOMAD_36_STABLE
Choose a base branch
from
Open

Iomad 35 stable #1131

wants to merge 2,824 commits into from

Conversation

lukebond68
Copy link

*** PLEASE DO NOT OPEN PULL REQUESTS VIA GITHUB ***

The moodle.git repository at Github is just a mirror of the official repository. We do not accept pull requests at Github.

See CONTRIBUTING.txt guidelines for how to contribute patches for Moodle. Thank you.

--

andrewnicols and others added 30 commits July 16, 2020 08:30
This is a huge performance improvement for behat.

The current set of steps loads the page
It then looks for the Classic/Clean version of the settings menu
If it fails to find it looks for the Boost Cog
Then it clicks the "Turn editing on" button

This can take a substantial period.

We do not actually need to run these steps as we are able to jump
straight to the URL. We already have access to the sesskey value
required to do this.

There is not loss in testing functionality because the actual testing of
the Turn editing functionality is tested in other places sufficiently.
Both ldap or the DB can return information in a non-consistent
ordering leading to events to be generated in different order.

And current tests are, right now, assuming a given order.

Note this is a rare random, but it's happening, so better
fix it, see the issue for some more details.

So we just do the tests ordering immune, verifying that all the
expected events have been triggered and done. Irrespectively of their order.
This is a direct copy of the 3.9 branch to have
changes under control easier for next commits.
- mysql requirement up to 5.7
- postgres requirement up to 9.6
The result of the final command `script` phase determines the build
result.

The cleanup belongs in the `after_script` phase.
AMOS bot and others added 30 commits April 2, 2021 00:07
Ensure we are using the ID of the given user rather than falling
back to that of the current user during privacy export.
This improves the code and avoids the risk of SQL injection through the
malicious XML-RPC request from the MNet peer.
Change the yes/no dropdowns to a checkbox. Defaults to unchecked.
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