Loading...

XML

Word

Printable

Affected Branches:
MOODLE_32_STABLE
Fixed Branches:
MOODLE_32_STABLE, MOODLE_33_STABLE
Pull from Repository:
git://github.com/mudrd8mz/moodle.git
Pull Main Branch:
~~MDL-57313~~-master-clibootswatch
Pull Main Diff URL:
https://github.com/mudrd8mz/moodle/compare/590292d48e...MDL-57313-master-clibootswatch
Testing Instructions:
Hide

Run the utility without any arguments and make sure the help is printed. Additionally, if the variables and/or bootswatch files are not found in their default locations, the info about the file not found should be displayed, too.

Run the utility with paths explicitly declared and make sure the output file is created. For example:

php import-bootswatch.php --variables=/tmp/_variables.scss --preset=/tmp/out.scss --bootswatch=/tmp/_bootswatch.scss
Show
Run the utility without any arguments and make sure the help is printed. Additionally, if the variables and/or bootswatch files are not found in their default locations, the info about the file not found should be displayed, too. Run the utility with paths explicitly declared and make sure the output file is created. For example: php import-bootswatch.php --variables=/tmp/_variables.scss --preset=/tmp/out.scss --bootswatch=/tmp/_bootswatch.scss

Attempting to play with the import-bootswatch.php utility, I realised the supported command line arguments are not correctly declared there. The long options are not declared in cli_get_params() so they are not supported.
Running the script without arguments should display help and exit like most Moodle CLI scripts to.
Help is badly formatted.
MIT License and Thomas Park copyright added to all generated files. This looks unusual. Even if Thomas Park was the original author of this script, the license does not automatically apply to generated files. What was the reasoning behind this?

has a non-specific relationship to

MDL-57875 SCSS PHP compiler does not support the '@import url' at-rule.

Participants:: CiBoT, David Mudrák (@mudrd8mz), Jun Pataleta, Mark Hayes, Ryan Wyllie, Simey Lameze