Skip to content

Commit

Permalink
Merge branch 'MDL-69805-master' of https://github.com/dravek/moodle
Browse files Browse the repository at this point in the history
  • Loading branch information
abgreeve committed Oct 8, 2020
2 parents 348961d + 84e9360 commit b239ee8
Show file tree
Hide file tree
Showing 2 changed files with 42 additions and 0 deletions.
4 changes: 4 additions & 0 deletions mod/data/mod_form.php
Expand Up @@ -38,6 +38,10 @@ function definition() {
$mform->hideIf('manageapproved', 'approval', 'eq', 0);

$mform->addElement('selectyesno', 'comments', get_string('allowcomments', 'data'));
if (empty($CFG->usecomments)) {
$mform->hardFreeze('comments');
$mform->setConstant('comments', 0);
}

$countoptions = array(0=>get_string('none'))+
(array_combine(range(1, DATA_MAX_ENTRIES), // Keys.
Expand Down
38 changes: 38 additions & 0 deletions mod/data/tests/behat/create_activity.feature
@@ -0,0 +1,38 @@
@mod @mod_data
Feature: Teachers can enable comments only if comments are enabled at site level
In order to enable comments on entries
As an admin
I need to enable comments at site level

@javascript
Scenario: Teacher cannot enable comments if they are disabled at site level
Given the following "users" exist:
| username | firstname | lastname | email |
| teacher1 | Teacher | 1 | teacher1@example.com |
And the following "courses" exist:
| fullname | shortname | category |
| Course 1 | C1 | 0 |
And the following "course enrolments" exist:
| user | course | role |
| teacher1 | C1 | editingteacher |
When I log in as "teacher1"
And I am on "Course 1" course homepage
And I press "Turn editing on"
And I press "Add an activity or resource"
And I click on "Add a new Database" "link" in the "Add an activity or resource" "dialogue"
And I expand all fieldsets
And "Allow comments on entries" "field" should exist
And I set the field "Name" to "Test Database name"
And I set the field "Allow comments on entries" to "Yes"
And I press "Save and return to course"
# Disable comments in site config.
And the following config values are set as admin:
| usecomments | 0 |
And I press "Add an activity or resource"
And I click on "Add a new Database" "link" in the "Add an activity or resource" "dialogue"
And I expand all fieldsets
And I set the field "Name" to "Test Database name 2"
And "Allow comments on entries" "field" should not exist
Then I should see "No" in the "//*[@id=\"fitem_id_comments\"]/*[@data-fieldtype=\"selectyesno\"]" "xpath_element"
And I press "Save and return to course"
And I log out

0 comments on commit b239ee8

Please sign in to comment.