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:
    • Rank:
      48111

      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.

        Issue Links

          Activity

          Hide
          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.
          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
          Petr Škoda added a comment -

          looks ok, +1, thanks

          Show
          Petr Škoda added a comment - looks ok, +1, thanks
          Hide
          Petr Škoda added a comment -

          I suppose this can be pushed now...

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

          Integrated (24 & master), thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated (24 & master), thanks!
          Hide
          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
          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: