Moodle
  1. Moodle
  2. MDL-38262

Undefined property: stdClass::$ldap_version in HEAD/auth/ldap/auth.php on line 1739

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Major Major
        • Resolution: Fixed
        • Affects Version/s: 2.4.1, 2.5
        • Fix Version/s: 2.4.3
        • Component/s: Authentication
        • Labels:

          Description

          attached the xdebug message accessing the URL http://localhost/HEAD/admin/auth_config.php?auth=ldap&lang=en
          The page was accessed for the first time. Not any field of the form in this page has been set before.

            Gliffy Diagrams

              Issue Links

              is a regression caused by

              Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-36119 ldap auth sync - add paged support

              • Minor - Minor loss of function where workaround is possible
              • Closed

                Activity

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

                This looks like it may be the product of a change made with MDL-36119. I've added Iñaki as a watcher as he was involved in that change.

                Show
                Michael de Raadt added a comment - This looks like it may be the product of a change made with MDL-36119 . I've added Iñaki as a watcher as he was involved in that change.
                Hide
                Permalink
                Iñaki Arenaza added a comment -

                Hi,

                I've pushed a couple of patches to https://github.com/iarenaza/moodle/compare/master...wip_master_mdl-38262 and https://github.com/iarenaza/moodle/compare/MOODLE_24_STABLE...wip_24_mdl-38262 for peer review.

                Saludos.
                Iñaki.

                Show
                Iñaki Arenaza added a comment - Hi, I've pushed a couple of patches to https://github.com/iarenaza/moodle/compare/master...wip_master_mdl-38262 and https://github.com/iarenaza/moodle/compare/MOODLE_24_STABLE...wip_24_mdl-38262 for peer review. Saludos. Iñaki.
                Hide
                Permalink
                Petr Skoda added a comment -

                looks ok, +1, thanks

                Show
                Petr Skoda added a comment - looks ok, +1, thanks
                Hide
                Permalink
                Petr Skoda added a comment -

                I suppose this can be pushed now...

                Show
                Petr Skoda added a comment - I suppose this can be pushed now...
                Hide
                Permalink
                Eloy Lafuente (stronk7) added a comment -

                Integrated (24 & master), thanks!

                Show
                Eloy Lafuente (stronk7) added a comment - Integrated (24 & master), thanks!
                Hide
                Permalink
                Eloy Lafuente (stronk7) added a comment -

                Tested while reviewing and looking for 23_STABLE potential backport. The PHP notices are now out.

                Show
                Eloy Lafuente (stronk7) added a comment - Tested while reviewing and looking for 23_STABLE potential backport. The PHP notices are now out.
                Hide
                Permalink
                Eloy Lafuente (stronk7) added a comment -

                This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

                Thanks!

                PS: Yay, legacy template messages. Yes, you're ok, we don't have CVS anymore!

                Show
                Eloy Lafuente (stronk7) added a comment - This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads). Thanks! PS: Yay, legacy template messages. Yes, you're ok, we don't have CVS anymore!

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: