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

                  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
                    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.
                    Hide
                    Permalink
                    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.

                    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
                    Permalink
                    skodak Petr Skoda added a comment -

                    looks ok, +1, thanks

                    Show
                    skodak Petr Skoda added a comment - looks ok, +1, thanks
                    Hide
                    Permalink
                    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
                    Permalink
                    stronk7 Eloy Lafuente (stronk7) added a comment -

                    Integrated (24 & master), thanks!

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