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

SCSS theme setting validation different when using $CFG->pathtosassc

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Deferred
    • Icon: Minor Minor
    • None
    • 3.6
    • HTML and CSS, Themes
    • MOODLE_36_STABLE

      The Boost theme has an admin setting class that allow raw SCSS to be entered and validated.

      The validate function in lib/adminlib.php for this class checks if the syntax complies, but ignores warnings about missing variables, because the variable may be defined somewhere. outside the text entry box within the theme files e.g.

      $link-color: $sucess;

      Would set one variable to another variable already set by Boost.

      If you then set $CFG->pathtosassc to use a c library to do the SCSS compliing the error handling changes. Now any error gets thrown as a "coding error" rather than either a validation error in the form, or for missing variables being ignored.

      This different kind of error doesn't get caught by the validation routine, and it doesn't suppress missing variable errors in the same way. Both errors seem to return the same error code value of 1 from the command line version so telling them apart might involve reading the text of the error message.

       

            Unassigned Unassigned
            bawjaws David Scotson
            Votes:
            1 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.