Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-41528

LDAP Enrolment course creation default settings

          Details

          • Type: Bug
          • Status: Closed
          • Priority: Minor
          • Resolution: Fixed
          • Affects Version/s: 2.4.5, 2.5.1, 2.6
          • Fix Version/s: 2.4.7, 2.5.3
          • Component/s: Enrolments
          • Labels:
          • Testing Instructions:
            Hide

            1/ configure ldap sync with an LDAP server
            2/ configure mapping for shortname, fullname and idnumber of courses
            3/ enable course auto creation
            4/ set enablecompletion=yes on course default settings
            5/ create a course in the ldap server
            6/ run cli script enrol/ldap/cli/sync.php
            7/ make sure the Enable completion tracking is set on the new course's settings page (You might have to enable the feature on the site level first to be able to see the setting in the course)

            Show
            1/ configure ldap sync with an LDAP server 2/ configure mapping for shortname, fullname and idnumber of courses 3/ enable course auto creation 4/ set enablecompletion=yes on course default settings 5/ create a course in the ldap server 6/ run cli script enrol/ldap/cli/sync.php 7/ make sure the Enable completion tracking is set on the new course's settings page (You might have to enable the feature on the site level first to be able to see the setting in the course)
          • Affected Branches:
            MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_26_STABLE
          • Fixed Branches:
            MOODLE_24_STABLE, MOODLE_25_STABLE
          • Pull from Repository:
          • Pull Master Branch:
            MDL-41528_master
          • Pull Master Diff URL:

            Description

            When not using a course as a template for new LDAP courses enable completion tracking is missing

            In addition groupmodeforce is set twice.

              Gliffy Diagrams

                Attachments

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  salvetore Michael de Raadt added a comment -

                  Thanks for working on that, Nitzan.

                  When you think it is ready, feel free to send this to peer review. Don't forget the testing instructions.

                  Show
                  salvetore Michael de Raadt added a comment - Thanks for working on that, Nitzan. When you think it is ready, feel free to send this to peer review. Don't forget the testing instructions.
                  Hide
                  Permalink
                  nitzan Nitzan Bar added a comment -

                  Michael de Raadt, I am submitting this to you since I do not know who should peer review this.

                  Thanks!

                  Show
                  nitzan Nitzan Bar added a comment - Michael de Raadt , I am submitting this to you since I do not know who should peer review this. Thanks!
                  Hide
                  Permalink
                  salvetore Michael de Raadt added a comment -

                  Hi, Nitzan.

                  Thanks for working on this.

                  It's not necessary to assign a peer reviewer when you push an issue for peer review and I'm certainly not the best person to peer review it.

                  I've added Iñaki Arenaza as a watcher on this issue and he may be willing to do a peer review.

                  Iñaki: you may have an ally in your fight for (against) LDAP.

                  Show
                  salvetore Michael de Raadt added a comment - Hi, Nitzan. Thanks for working on this. It's not necessary to assign a peer reviewer when you push an issue for peer review and I'm certainly not the best person to peer review it. I've added Iñaki Arenaza as a watcher on this issue and he may be willing to do a peer review. Iñaki: you may have an ally in your fight for (against) LDAP.
                  Hide
                  Permalink
                  poltawski Dan Poltawski added a comment -

                  Hi Nitzan,

                  Thanks for the perfect patch, i'm sending this for integration.

                  It would be good to write in the testing instructions what is expected to happen to verify the change is correct.

                  thanks!

                  Dan

                  Show
                  poltawski Dan Poltawski added a comment - Hi Nitzan, Thanks for the perfect patch, i'm sending this for integration. It would be good to write in the testing instructions what is expected to happen to verify the change is correct. thanks! Dan
                  Hide
                  Permalink
                  samhemelryk Sam Hemelryk added a comment -

                  Thanks Nitzan this has been integrated now.

                  Show
                  samhemelryk Sam Hemelryk added a comment - Thanks Nitzan this has been integrated now.
                  Hide
                  Permalink
                  ankit_frenz Ankit Agarwal added a comment -

                  works as described.

                  Show
                  ankit_frenz Ankit Agarwal added a comment - works as described.
                  Hide
                  Permalink
                  damyon Damyon Wiese added a comment -

                  This issue along with 77 of it's friends has been sent upstream and released to the world.

                  Thankyou for your contribution.

                  Show
                  damyon Damyon Wiese added a comment - This issue along with 77 of it's friends has been sent upstream and released to the world. Thankyou for your contribution.

                    People

                    • Votes:
                      0 Vote for this issue
                      Watchers:
                      3 Start watching this issue

                      Dates

                      • Created:
                        Updated:
                        Resolved:
                        Fix Release Date:
                        11/Nov/13