Eloy Lafuente (stronk7) [Wed, 12 Jul 2017 23:39:45 +0000 (01:39 +0200)]
MDL-59437 behat: avoid using participants page for tag tests
While participants page is being refactored, there are links (fullname)
going and changing, leading to inestability in tests that were using
those links.
When possible, provide an alternative, stable path, in this case, the
tags page itself can be used to jump to the user profile, keeping the
tag tests better isolated.
Eloy Lafuente (stronk7) [Thu, 13 Jul 2017 00:27:09 +0000 (02:27 +0200)]
MDL-59437 behat: old "edit" cog to go to enrollments page is gone
There use to be a little cot to easily navigate from the
participations page to the enrollments page.
In boost it was already hidden, but present and the test was still
working (non-javascript).
With the conversion, the functionality is 100% gone so, right now,
the only way to navigate from participats to enrollments (that will
dissappear with with unification), is using the course admin cog.
Damyon Wiese [Thu, 13 Jul 2017 02:55:01 +0000 (10:55 +0800)]
MDL-59365 enrol: Behat fixes
Fixes for the new enrol users popup with behat.
David Monllao [Wed, 12 Jul 2017 14:53:55 +0000 (16:53 +0200)]
MDL-59368 users: Clear whitespace
David Monllao [Wed, 12 Jul 2017 14:47:20 +0000 (16:47 +0200)]
MDL-59368 users: Fix incorrect reference
David Monllao [Wed, 12 Jul 2017 14:46:49 +0000 (16:46 +0200)]
Merge branch 'MDL-59368-master' of git://github.com/damyon/moodle
David Monllao [Wed, 12 Jul 2017 12:12:53 +0000 (14:12 +0200)]
Merge branch 'MDL-59384-master-final' of https://github.com/lameze/moodle
Simey Lameze [Wed, 12 Jul 2017 12:03:04 +0000 (20:03 +0800)]
MDL-59384 core_calendar: add delete event logic to modal
Part of MDL-59333.
Simey Lameze [Wed, 12 Jul 2017 12:00:37 +0000 (20:00 +0800)]
MDL-59384 core_calendar: add calendar event amd module
Simey Lameze [Wed, 12 Jul 2017 11:58:57 +0000 (19:58 +0800)]
MDL-59384 core_calendar: add necessary data-* attributes
Part of MDL-59333.
Simey Lameze [Wed, 12 Jul 2017 11:57:22 +0000 (19:57 +0800)]
MDL-59384 core_modal: trigger bodyRendered event
Part of MDL-59333.
Simey Lameze [Wed, 12 Jul 2017 11:56:27 +0000 (19:56 +0800)]
MDL-59384 core_calendar: add deleteEvent function to repository
Part of MDL-59333.
Simey Lameze [Wed, 12 Jul 2017 11:55:24 +0000 (19:55 +0800)]
MDL-59384 core_calendar: make delete_calendar_events ajax visible
Part of MDL-59333.
Dan Poltawski [Wed, 12 Jul 2017 11:22:32 +0000 (12:22 +0100)]
Merge branch 'MDL-56486-master-workshopfiletypes' of git://github.com/mudrd8mz/moodle
Simey Lameze [Wed, 12 Jul 2017 10:28:11 +0000 (18:28 +0800)]
MDL-59383 core_modal: trigger event on modal module
Simey Lameze [Wed, 12 Jul 2017 05:53:16 +0000 (13:53 +0800)]
MDL-59383 core_calendar: change modal body rendering
Simey Lameze [Wed, 12 Jul 2017 03:40:35 +0000 (11:40 +0800)]
MDL-59383 core_calendar: proper handling of description on webservice
Simey Lameze [Wed, 12 Jul 2017 03:39:04 +0000 (11:39 +0800)]
MDL-59383 core_calendar: css fixes to summary modal body
Simey Lameze [Wed, 12 Jul 2017 10:28:11 +0000 (18:28 +0800)]
MDL-59383 core_modal: trigger event on modal module
David Mudrák [Wed, 12 Jul 2017 09:58:08 +0000 (11:58 +0200)]
MDL-56486 workshop: Do not rely on the button implementation in behat
It turned out that behat's "I should see ..." does not work for buttons
rendered as `<input>` elements so this assertion worked only in Boost
where the button is rendered as `<button>` but not in Clean.
But we do not really need this check here. All we need is to make sure
that the form has been saved successfully. For which, checking for the
text on the page is enough.
p.s. Another mistake of mine to not double-check behat on both standard
themes. Sigh ...
David Mudrák [Wed, 12 Jul 2017 09:39:49 +0000 (11:39 +0200)]
MDL-56486 workshop: Fix behat scenario for submission attachments
This Behat scenario was not failing only because it actually relied on
the bug in the wrongly set accepted_types for submission attachments.
Now that the previous commit has fixed that bug, this scenario started
to fail correctly.
We need to do the same trick here as we are doing in the latter
scenario: we first upload a valid file to bypass the upload repository
checks, and rename it before submitting the form. That allows us to see
that the invalid file is caught finally.
p.s. It was a silly mistake of mine. I should had been wondering why
the same procedure worked for one field and not for the other. I would
spot the bug with wrong accepted_types earlier.
Dan Poltawski [Wed, 12 Jul 2017 08:50:36 +0000 (09:50 +0100)]
Merge branch 'MDL-56486-master-workshopfiletypes' of git://github.com/mudrd8mz/moodle
David Mudrák [Wed, 12 Jul 2017 07:57:06 +0000 (09:57 +0200)]
MDL-56486 workshop: Fix accepted_types for submission attachments
It was a copy&paste mistake that the accepted_types option for the
submission attachments used the value configured for overall feedback
attachments.
Credit goes to Marina Glancy for catching this during testing.
Dan Poltawski [Wed, 12 Jul 2017 07:15:38 +0000 (08:15 +0100)]
Merge branch 'MDL-59365-master-fix1' of github.com/damyon/moodle
David Monllao [Wed, 12 Jul 2017 07:07:03 +0000 (09:07 +0200)]
Merge branch 'MDL-59383-fix-1' of https://github.com/lameze/moodle
Simey Lameze [Wed, 12 Jul 2017 05:53:16 +0000 (13:53 +0800)]
MDL-59383 core_calendar: change modal body rendering
Simey Lameze [Wed, 12 Jul 2017 03:40:35 +0000 (11:40 +0800)]
MDL-59383 core_calendar: proper handling of description on webservice
Simey Lameze [Wed, 12 Jul 2017 03:39:04 +0000 (11:39 +0800)]
MDL-59383 core_calendar: css fixes to summary modal body
Damyon Wiese [Wed, 12 Jul 2017 06:47:11 +0000 (14:47 +0800)]
MDL-59365 enrol: Teachers can see other users on the site
In order to enrol new users into a course, you need to see users who are not in the current course.
David Monllao [Wed, 12 Jul 2017 06:34:30 +0000 (08:34 +0200)]
Merge branch 'MDL-59431-master-fix2' of git://github.com/andrewnicols/moodle
Andrew Nicols [Wed, 12 Jul 2017 02:49:59 +0000 (10:49 +0800)]
Merge branch 'wip-MDL-57462-master-fix' of git://github.com/abgreeve/moodle
Damyon Wiese [Wed, 12 Jul 2017 02:41:32 +0000 (10:41 +0800)]
MDL-59368 groups: More phpunit coverage
Cover $withmembers parameter to groups_get_all_groups with a unit test.
Adrian Greeve [Wed, 12 Jul 2017 02:39:20 +0000 (10:39 +0800)]
MDL-57432 installation: Fix to the php version check.
Damyon Wiese [Fri, 7 Jul 2017 02:08:18 +0000 (10:08 +0800)]
MDL-59368 groups: Peer review fixes
* Fix coalesce on postgres.
* The edit icon's alt shows the HTML entities causing
* enrol/tests/behat/add_to_group.feature Contains a '@doit' tag which I assume is there from testing.
* group/classes/output/user_groups_editable.php
** Missing MOODLE_INTERNAL check.
** Unused variables context_system and moodle_exception.
** The PHPDocs for the constructors are wrong.
** export_for_template() returns an array, not stdClass (according to parent docs).
** Change moodle_exception to coding_exception at the beginning.
* group/lib.php
** Missing docs for core_group_inplace_editable().
* user/classes/participants_table.php
** The docs for the class variables $groups, $course and $context need a '\' beforehand as they are in a namespace.
** I would prefer $this->context = $context; and $this->groups = ... to be done at the end of the constructor with the other class variable assignments.
** You could get rid of the class variable courseid if we are setting course and use $this->course->id instead.
** The function col_groups has @param \stdclass $row but it should be $data
* lib/amd/src/form-autocomplete.js and lib/amd/src/inplace_editable.js
** Some issues here CiBot has pointed out.
* lib/classes/output/inplace_editable.php
** @see should be on a new line.
Damyon Wiese [Fri, 7 Jul 2017 01:38:00 +0000 (09:38 +0800)]
MDL-59368 groups: Postgres fix
Postgres is strict about mixing types.
Damyon Wiese [Wed, 5 Jul 2017 02:10:13 +0000 (10:10 +0800)]
MDL-59368 groups: Show no groups instead of ''
This is on the inplace editable for a users course groups.
Damyon Wiese [Mon, 3 Jul 2017 03:46:26 +0000 (11:46 +0800)]
MDL-59368 groups: Add behat for new group edit
Damyon Wiese [Fri, 30 Jun 2017 07:01:49 +0000 (15:01 +0800)]
MDL-59368 enrol: Ensure remove_group_member_allowed works
Wierd rare callback that does not work in core. Still respect it in the UI.
Damyon Wiese [Fri, 30 Jun 2017 06:15:24 +0000 (14:15 +0800)]
MDL-59368 inplace_editable: Focus on auto-complete
Also fix JS coding style and build the amd.
Damyon Wiese [Fri, 30 Jun 2017 05:57:48 +0000 (13:57 +0800)]
MDL-59368 groups: Add an explicit cancel button
Autocomplete already uses Escape keyboard handler - so we can't use it to cancel an edit-inplace thing.
Make an obvious cancel button instead.
Damyon Wiese [Thu, 29 Jun 2017 02:30:04 +0000 (10:30 +0800)]
MDL-59368 groups: Inplace editing for users groups
Andrew Nicols [Wed, 12 Jul 2017 02:05:59 +0000 (10:05 +0800)]
MDL-59431 requirejs: Prioritise errors
Andrew Nicols [Wed, 12 Jul 2017 01:45:20 +0000 (09:45 +0800)]
MDL-59431 requirejs: Correct failed JS check
David Monllao [Tue, 11 Jul 2017 19:49:03 +0000 (21:49 +0200)]
Merge branch 'MDL-59039-master' of https://github.com/sammarshallou/moodle
sam marshall [Thu, 25 May 2017 17:19:06 +0000 (18:19 +0100)]
MDL-59039 Global search: Allow partial indexing (in scheduled task)
Dan Poltawski [Tue, 11 Jul 2017 13:06:11 +0000 (14:06 +0100)]
Merge branch 'MDL-59019-master' of git://github.com/jleyva/moodle
Damyon Wiese [Tue, 4 Jul 2017 08:53:08 +0000 (16:53 +0800)]
MDL-59365 enrol: Use standard modal
Replace the custom footer with a standard modal.
Damyon Wiese [Tue, 4 Jul 2017 08:34:52 +0000 (16:34 +0800)]
MDL-59365 behat: Autocomplete fixes
The suggestion list stays open for multi-selects now. We need to close it
explicitly.
Damyon Wiese [Tue, 4 Jul 2017 05:25:16 +0000 (13:25 +0800)]
MDL-59365 eslint: Noop changes for eslint
Damyon Wiese [Wed, 28 Jun 2017 02:25:45 +0000 (10:25 +0800)]
MDL-59365 enrol: enrol button participants page
Damyon Wiese [Wed, 28 Jun 2017 02:11:06 +0000 (10:11 +0800)]
MDL-59365 enrol: Make select users first
This is the most important element in the form - make it first.
Damyon Wiese [Tue, 27 Jun 2017 08:36:19 +0000 (16:36 +0800)]
MDL-59365 enrol: Switch the user selector
The user selector was calling the wrong API to get the potential list of users to enrol.
Now it works.
Damyon Wiese [Tue, 27 Jun 2017 07:43:41 +0000 (15:43 +0800)]
MDL-59365 autocomplete: Keep the suggestions open
If the list allows multiple selections, don't auto-close the suggestions list
every time something is chosen.
Damyon Wiese [Thu, 20 Oct 2016 09:13:49 +0000 (17:13 +0800)]
MDL-59365 enrol_manual: Rewrite the yui enrolment popup in amd
The add/roles thing was only converted to bootstrap 2/4 markup and the yui left in place.
The modal for adding users to a course was unsavable so I rewrote it with an mform in a popup, still calling
the same (barely) modified ajax script.
The webservice for searching cohorts was taken from admin/tool/lp and moved into /cohort. I added a generic "cohort"
mform element at the same time.
The webservice for searching for users was taken from the original ajax script.
Eloy Lafuente (stronk7) [Tue, 11 Jul 2017 11:52:24 +0000 (13:52 +0200)]
MDL-56629 dataformat_html: Adding UTF-8 charset
Also adding the HEAD tag which was missing.
(credit goes to Frédéric Massart - FMCorz, thanks!)
Eloy Lafuente (stronk7) [Tue, 11 Jul 2017 11:41:28 +0000 (13:41 +0200)]
Merge branch 'MDL-54965-master' of git://github.com/tobiasreischmann/moodle
David Monllao [Tue, 11 Jul 2017 10:36:58 +0000 (12:36 +0200)]
Merge branch 'wip-MDL-59363-master' of git://github.com/marinaglancy/moodle
Dan Poltawski [Tue, 11 Jul 2017 10:29:08 +0000 (11:29 +0100)]
MDL-59287 upgrade.txt: avoid misleading versions
Dan Poltawski [Tue, 11 Jul 2017 10:29:53 +0000 (11:29 +0100)]
Merge branch 'wip-MDL-59287-master-v3' of git://github.com/abgreeve/moodle
David Monllao [Tue, 11 Jul 2017 10:05:30 +0000 (12:05 +0200)]
Merge branch 'MDL-59431' of git://github.com/timhunt/moodle
David Monllao [Tue, 11 Jul 2017 09:41:03 +0000 (11:41 +0200)]
Merge branch 'wip-MDL-59441-master' of git://github.com/marinaglancy/moodle
Dan Poltawski [Tue, 11 Jul 2017 09:18:48 +0000 (10:18 +0100)]
Merge branch 'MDL-56486-master-workshopfiletypes' of git://github.com/mudrd8mz/moodle
Dan Poltawski [Tue, 11 Jul 2017 09:16:40 +0000 (10:16 +0100)]
Merge branch 'MDL-59443-master-filetypesvalidation' of git://github.com/mudrd8mz/moodle
Eloy Lafuente (stronk7) [Tue, 11 Jul 2017 08:44:06 +0000 (10:44 +0200)]
Merge branch 'MDL-57432-master' of git://github.com/danpoltawski/moodle
David Monllao [Tue, 11 Jul 2017 08:05:03 +0000 (10:05 +0200)]
Merge branch 'MDL-59383-master' of git://github.com/lameze/moodle
Simey Lameze [Mon, 10 Jul 2017 02:10:54 +0000 (10:10 +0800)]
MDL-59383 core_calendar: make event summary display on modal
Part of MDL-59333.
Simey Lameze [Mon, 10 Jul 2017 02:10:03 +0000 (10:10 +0800)]
MDL-59383 core_calendar: add get_calendar_event_by_id webservice
Part of MDL-59333.
Simey Lameze [Mon, 10 Jul 2017 02:08:23 +0000 (10:08 +0800)]
MDL-59383 core_calendar: add lang strings for event types
Part of MDL-59333.
Simey Lameze [Mon, 10 Jul 2017 02:07:40 +0000 (10:07 +0800)]
MDL-59383 core_calendar: create summary modal templates
Part of MDL-59333.
Simey Lameze [Mon, 10 Jul 2017 02:06:51 +0000 (10:06 +0800)]
MDL-59383 core_calendar: create summary modal AMD module
Part of MDL-59333.
Simey Lameze [Mon, 10 Jul 2017 02:06:01 +0000 (10:06 +0800)]
MDL-59383 core_calendar: create calendar repository AMD module
Part of MDL-59333.
Simey Lameze [Mon, 10 Jul 2017 02:04:54 +0000 (10:04 +0800)]
MDL-59383 core_calendar: create main calendar AMD module
Part of MDL-59333.
Dan Poltawski [Mon, 10 Jul 2017 13:36:40 +0000 (14:36 +0100)]
Merge branch 'wip-MDL-40408-master' of git://github.com/marinaglancy/moodle
Dan Poltawski [Mon, 10 Jul 2017 13:31:06 +0000 (14:31 +0100)]
Merge branch 'wip-MDL-59262-master-2' of git://github.com/marinaglancy/moodle
David Monllao [Mon, 10 Jul 2017 12:38:27 +0000 (14:38 +0200)]
Merge branch 'MDL-52501' of https://github.com/NeillM/moodle
Dan Poltawski [Mon, 10 Jul 2017 12:28:28 +0000 (13:28 +0100)]
Merge branch 'MDL-59317' of https://github.com/NeillM/moodle
David Monllao [Mon, 10 Jul 2017 12:26:24 +0000 (14:26 +0200)]
MDL-58851 lti: Fix undefined var PHP notice
David Monllao [Mon, 10 Jul 2017 12:25:24 +0000 (14:25 +0200)]
Merge branch 'lti_admin' of https://github.com/baholladay/moodle
Bryan Holladay [Mon, 8 May 2017 20:28:09 +0000 (16:28 -0400)]
MDL-58851 LTI module: Add LTI admin role capability
Adding a new 'mod/lti:admin' capability and passes the LTI admin role if the user has the 'mod/lti:admin' capability.
David Monllao [Mon, 10 Jul 2017 12:19:31 +0000 (14:19 +0200)]
Merge branch 'MDL-58332-master' of git://github.com/ryanwyllie/moodle
Dan Poltawski [Mon, 10 Jul 2017 12:15:21 +0000 (13:15 +0100)]
Merge branch 'wip-MDL-46495-master' of git://github.com/marinaglancy/moodle
Tobias Reischmann [Thu, 8 Jun 2017 09:56:03 +0000 (11:56 +0200)]
MDL-54965 mod_data: removed unused code in display_add_field()
Since the refactoring towards using the filepicker for file and picture
fields there are several variables, which are set but never read. Thus,
I removed them.
Tobias Reischmann [Thu, 8 Jun 2017 08:47:14 +0000 (10:47 +0200)]
MDL-54965 mod_data: Init draft area for empty file/pic fields
When a file or picture field was added with existing records, these
records could not be saved, since the draft area was not created
properly.
Co-authored-by: Jérôme Mouneyrac <mouneyrac>
Eloy Lafuente (stronk7) [Mon, 10 Jul 2017 10:19:09 +0000 (12:19 +0200)]
Merge branch 'MDL-59452-master-filetypesbehat' of git://github.com/mudrd8mz/moodle
Dan Poltawski [Mon, 10 Jul 2017 10:15:00 +0000 (11:15 +0100)]
Merge branch 'MDL-58957-master' of https://github.com/sammarshallou/moodle
David Monllao [Mon, 10 Jul 2017 10:04:14 +0000 (12:04 +0200)]
Merge branch 'wip-MDL-56579-master-2' of git://github.com/marinaglancy/moodle
Eloy Lafuente (stronk7) [Wed, 14 Jun 2017 17:33:22 +0000 (19:33 +0200)]
MDL-57432 upgrade: remove unused functions/settings from upgradelib
All these functions were used only by deleted upgrade steps
so it's safe to proceed with straight deletion, considering
them internal. Deletion has been documented in corresponding
upgrade.txt files:
- upgrade_mimetypes()
- upgrade_fix_missing_root_folders_draft()
- upgrade_minmaxgrade() and upgrade_minmaxgradestepignored setting
- upgrade_course_tags()
- atto_equation_update_librarygroup4_setting()
- mod_lti_upgrade_custom_separator()
These have been kept because continue being used by restore:
- upgrade_group_members_only()
- upgrade_extra_credit_weightoverride()
- upgrade_calculated_grade_items()
Eloy Lafuente (stronk7) [Wed, 14 Jun 2017 15:13:02 +0000 (17:13 +0200)]
MDL-57432 upgrade: clean < 3.0.0 upgrade steps
This just deletes all the upgrade steps previous to 3.0.0. Some
small adjustments, like adding missing MOODLE_INTERNAL or tweaking
globals can also be applied when needed.
Also includes an upgrade step to prevent upgrading from any
version <
2015111600 (v3.0.0) as anti-cheating measure.
Next commit will get rid of/deprecate all the upgradelib functions
not used anymore in codebase.
Dan Poltawski [Fri, 9 Jun 2017 10:49:00 +0000 (11:49 +0100)]
MDL-57432 environment: raise Moodle requirement
Due to Moodle 2.7 and Moodle 3.4 not having a common compatible php version
between them to upgrade from Moodle 3.4 from Moodle 2.7 you'll need to upgrade
to a higher version first or upgrade php at the same time as upgrade,
this encourages the former.
Dan Poltawski [Thu, 8 Jun 2017 10:28:15 +0000 (11:28 +0100)]
MDL-57432 travis: move to PHP 7.x only
* We have to make the upgrade test from 3.0 to
avoid messing messing around with versions
Dan Poltawski [Thu, 8 Jun 2017 10:15:55 +0000 (11:15 +0100)]
MDL-57432 environment: raise Moodle 3.4 requirements
1) PHP minimum version raised to 7.0.0
2) Intl extension raised from optional to required
Dan Poltawski [Thu, 8 Jun 2017 10:12:27 +0000 (11:12 +0100)]
MDL-57432 environment: 3.4 base information
This is a direct copy of the 3.3 branch to
have changes under control easier in next commits.
Dan Poltawski [Thu, 8 Jun 2017 09:51:32 +0000 (10:51 +0100)]
MDL-57432 lib: unify PHP version checks
* Introduce a new php compatible library which can be used
early in execution, without Moodle dependencies
* Use it where we previously had hardcoded checks
This was originally suggested in MDL-39007.
Dan Poltawski [Mon, 10 Jul 2017 07:36:40 +0000 (08:36 +0100)]
Merge branch 'MDL-58544-master' of git://github.com/damyon/moodle
Eloy Lafuente (stronk7) [Mon, 10 Jul 2017 02:25:23 +0000 (04:25 +0200)]
Merge branch 'wip-MDL-59468-master' of https://github.com/marinaglancy/moodle
Ryan Wyllie [Mon, 27 Mar 2017 05:49:27 +0000 (05:49 +0000)]
MDL-58332 message: remove concat in get_popup_notifications sql
Removed the concat to generate the uniqueid field for the popup
notifications data. The concat can't be used directly in the SQL because
the syntax changes between databases. The sql_concat helper can't be
used because it assumes all values are database columns (which they
aren't in this case).
Instead I've just removed the uniqueid field because it isn't required
for the union all to work and the field isn't being used by anything.
This should fixed the compatibility issues between databases.
Adrian Greeve [Mon, 3 Jul 2017 04:04:58 +0000 (12:04 +0800)]
MDL-59287 calendar_events: Update modules to create all events.
All events equals all calendar events including completion events.
Adrian Greeve [Tue, 4 Jul 2017 01:21:17 +0000 (09:21 +0800)]
MDL-59287 course_module: switching to new function.
Instead of using *_refresh_events directly
course_module_update_calendar_events is called so
that all types of calendar events can be updated.
Adrian Greeve [Tue, 4 Jul 2017 06:43:25 +0000 (14:43 +0800)]
MDL-59287 course_module: Create calendar event update function.
Adrian Greeve [Thu, 29 Jun 2017 01:08:34 +0000 (09:08 +0800)]
MDL-59287 core_completion: Bulk completion now updates events.