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

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

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

            Activity

            Hide
            salvetore 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
            salvetore 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
            iarenaza 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
            skodak Petr Skoda added a comment -

            looks ok, +1, thanks

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

            I suppose this can be pushed now...

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

            Integrated (24 & master), thanks!

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

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

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Tested while reviewing and looking for 23_STABLE potential backport. The PHP notices are now out.
            Hide
            stronk7 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
            stronk7 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:
                  Fix Release Date:
                  18/Mar/13