Moodle

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

Details

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

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 - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

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

looks ok, +1, thanks

Show
Petr Škoda added a comment - looks ok, +1, thanks
Hide
Permalink
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
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

Vote (0)
Watch (4)

Dates

  • Created:
    Updated:
    Resolved: