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

Vague README-LDAP file in \auth\ldap

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.4, 2.1.1, 2.2
    • Fix Version/s: 2.0.5, 2.1.2
    • Component/s: Authentication
    • Labels:
    • Environment:
      Any

      Description

      The README-LDAP text file in the \auth\ldap directory contains one line. It says, "Please read comments from lib.php".
      This is very vague and quite unusable - which lib.php file is it referring to? There are about 157 lib.php files in the Moodle 2 codebase!

      Please add more details to this readme file to make it clear which lib.php file it is referring to.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Yes, that is a bit annoying.

            Thanks for suggesting this. I've put it on our backlog and we'll try to get to it as soon as we can.

            If you'd like to have a crack at improving that file, please do so.

            Michael;

            Show
            salvetore Michael de Raadt added a comment - Yes, that is a bit annoying. Thanks for suggesting this. I've put it on our backlog and we'll try to get to it as soon as we can. If you'd like to have a crack at improving that file, please do so. Michael;
            Show
            libertymoodle Luis de Vasconcelos added a comment - As per http://moodle.org/mod/forum/post.php?reply=803526
            Hide
            iarenaza Iñaki Arenaza added a comment -

            Note for integrators: Please, backport this to MOODLE_21_STABLE and MOODLE_20_STABLE. Thanks

            Show
            iarenaza Iñaki Arenaza added a comment - Note for integrators: Please, backport this to MOODLE_21_STABLE and MOODLE_20_STABLE. Thanks
            Hide
            nebgor Aparup Banerjee added a comment -

            cherry-picked and integrated to 2.x branches, thanks

            Show
            nebgor Aparup Banerjee added a comment - cherry-picked and integrated to 2.x branches, thanks
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Pass...

            Show
            samhemelryk Sam Hemelryk added a comment - Pass...
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            git & cvs repositories updated with your gorgeous code. Many thanks!

            Closing and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - git & cvs repositories updated with your gorgeous code. Many thanks! Closing and ciao

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Oct/11