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

Add "test database setup" to the "External Database" enrollment and authentication plugins

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8, 1.8.1, 1.8.2, 1.8.3, 1.8.4, 1.8.5, 1.8.6, 1.9, 1.9.1, 1.9.2, 2.5
    • Fix Version/s: 2.6
    • Component/s: Authentication, Enrolments
    • Labels:
    • Database:
      Any
    • Testing Instructions:
      Hide
      1. go to admin -> plugins -> auth plugins -> overview
      2. Enable the Database plugin
      3. try the new "Test settings" link next to db auth - you should see errors
      4. fix the settings and execute test again
      5. repeat for external DB enrolments
      Show
      go to admin -> plugins -> auth plugins -> overview Enable the Database plugin try the new "Test settings" link next to db auth - you should see errors fix the settings and execute test again repeat for external DB enrolments
    • Affected Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w41_MDL-16073_m26_extdbtest
    • Sprint:
      BACKEND Sprint 5
    • Story Points (Obsolete):
      4
    • Sprint:
      BACKEND Sprint 5

      Description

      It is hard to test whether the administrative panel for External Database Authentication and for External Database Enrollment is configured correctly. Currently, you have to create fake user accounts or similar, and try to authenticate them or enroll them and see if they receive email, etc. (At least, that's how I tested it! Does someone have a better way?)

      It woule be nice to have a "Test Connection" button on that page which goes through and makes sure all the db info (username, password, connection, etc.) and data mapping is correct. AND prints out all this stuff to the screen in an easy-to-read report.

      Thank you!

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              skodak Petr Skoda added a comment -

              Thanks for the report, submitting for review. I guess we could test more settings there and even add support for other plugins such as LDAP...

              Show
              skodak Petr Skoda added a comment - Thanks for the report, submitting for review. I guess we could test more settings there and even add support for other plugins such as LDAP...
              Hide
              skodak Petr Skoda added a comment -

              No objections so far, submitting for integration. I think this could be useful and we can improve it more in the future.

              Show
              skodak Petr Skoda added a comment - No objections so far, submitting for integration. I think this could be useful and we can improve it more in the future.
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Hi Petr, this has conflicted in a couple of places with your plugins improvement I believe.
              Any chance you'd be able to rebase this on top of integration and fix those conflicts?
              If not no probs I'll have a go at it later today.

              Thanks
              Sam

              Show
              samhemelryk Sam Hemelryk added a comment - Hi Petr, this has conflicted in a couple of places with your plugins improvement I believe. Any chance you'd be able to rebase this on top of integration and fix those conflicts? If not no probs I'll have a go at it later today. Thanks Sam
              Show
              skodak Petr Skoda added a comment - rebased in https://github.com/skodak/moodle/commits/w41_MDL-16073_m26_extdbtest_i
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Awesome thanks Petr this has been integrated now. It looks like a really good base to build upon.

              Show
              samhemelryk Sam Hemelryk added a comment - Awesome thanks Petr this has been integrated now. It looks like a really good base to build upon.
              Hide
              dobedobedoh Andrew Nicols added a comment -

              Tested against another moodle database as I didn't have an appropriate external auth source handy.

              Show
              dobedobedoh Andrew Nicols added a comment - Tested against another moodle database as I didn't have an appropriate external auth source handy.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Yes, it's happening (somewhere in the French Polynesia, right now). And you did it, raising Moodle to new excellency levels.

              Or, if you prefer, yes, you fixed that boring issue.

              Thanks anyway! Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Yes, it's happening (somewhere in the French Polynesia, right now). And you did it, raising Moodle to new excellency levels. Or, if you prefer, yes, you fixed that boring issue. Thanks anyway! Ciao

                People

                • Votes:
                  4 Vote for this issue
                  Watchers:
                  8 Start watching this issue

                  Dates

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