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

          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