Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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

          Issue Links

            Activity

            Hide
            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
            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
            Aparup Banerjee added a comment -

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

            Show
            Aparup Banerjee added a comment - itnegrated into master and 23. thanks. (works for MOODLE_23_STABLE here)
            Hide
            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
            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
            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
            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: