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

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

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

                  Show
                  libertymoodle Luis de Vasconcelos added a comment - As per http://moodle.org/mod/forum/post.php?reply=803526
                  Hide
                  Permalink
                  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
                  Permalink
                  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
                  Permalink
                  samhemelryk Sam Hemelryk added a comment -

                  Pass...

                  Show
                  samhemelryk Sam Hemelryk added a comment - Pass...
                  Hide
                  Permalink
                  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