Moodle
  1. Moodle
  2. MDL-27689

Vague README-LDAP file in \auth\ldap

        Details

        • Type: Improvement Improvement
        • Status: Closed
        • Priority: Minor 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

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              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
              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;
              Hide
              Permalink
              Luis de Vasconcelos added a comment -

              As per http://moodle.org/mod/forum/post.php?reply=803526

              Show
              Luis de Vasconcelos added a comment - As per http://moodle.org/mod/forum/post.php?reply=803526
              Hide
              Permalink
              Iñaki Arenaza added a comment -

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

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

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

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

              Pass...

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

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

              Closing and ciao

              Show
              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: