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

PHP warning during CLI install

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.3.2, 2.4
    • Fix Version/s: 2.3.3
    • Component/s: Installation
    • Labels:

      Description

      If I run the CLI installer the first question it asks is about language. Immediately before the list of available languages is the following PHP warning.

      PHP Warning: str_repeat(): Second argument has to be greater than or equal to 0 in /home/andrew/Desktop/code/moodle/dev/master/admin/cli/install.php on line 249

      Warning: str_repeat(): Second argument has to be greater than or equal to 0 in /home/andrew/Desktop/code/moodle/dev/master/admin/cli/install.php on line 249

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              skodak Petr Skoda added a comment -

              Thanks for the report, I have fixed the notice and compatibility with RTL languages at the same time by eliminating multiple columns - admins will have to scroll a bit more, unfortunately there is no easy way to print LTR and RTL strings on one line in terminal.

              Show
              skodak Petr Skoda added a comment - Thanks for the report, I have fixed the notice and compatibility with RTL languages at the same time by eliminating multiple columns - admins will have to scroll a bit more, unfortunately there is no easy way to print LTR and RTL strings on one line in terminal.
              Hide
              nebgor Aparup Banerjee added a comment -

              itnegrated into master and 23. thanks. (works for MOODLE_23_STABLE here)

              Show
              nebgor Aparup Banerjee added a comment - itnegrated into master and 23. thanks. (works for MOODLE_23_STABLE here)
              Hide
              abgreeve Adrian Greeve added a comment -

              Tested on the 2.3 and master integration branches.
              The languages are now displayed in a list and there is no error being displayed.
              Test passed.

              Show
              abgreeve Adrian Greeve added a comment - Tested on the 2.3 and master integration branches. The languages are now displayed in a list and there is no error being displayed. Test passed.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              From somewhere within the clouds...

              Congrats, this has been sent upstream and is now part of Moodle (your favorite LMS platform). Many thanks for your awesome collaboration!

              Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - From somewhere within the clouds... Congrats, this has been sent upstream and is now part of Moodle (your favorite LMS platform). Many thanks for your awesome collaboration! Ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    12/Nov/12