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

          Attachments

            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