Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-19077 Implement the theme engines proposal
  3. MDL-19690

Replace $CFG->pixpath with an $OUTPUT->icon_url_old($icon, $module);

XMLWordPrintable

    • Icon: Sub-task Sub-task
    • Resolution: Fixed
    • Icon: Minor Minor
    • 2.0
    • 2.0
    • Libraries
    • None
    • MOODLE_20_STABLE
    • MOODLE_20_STABLE

      We are having problems with $CFG->pixpath in HEAD>

      To start with, it is a nasty global variables hack.

      But the problem is that you cannot set up $CFG->pixpath until the theme is known. But the theme is only know after a variety of other things are done and it is hard to predict when that is.

      However, we do currently have clever code to initialise $OUTPUT the first time it is used.

      Old code that used the string "$CFG->modpixpath/$cm->modname/icon.gif" should be updated to $OUTPUT->mod_icon_url('icon', $cm->modname);

      Old code that used the string "$CFG->pixpath/i/settings.gif" should be updated to $OUTPUT->old_icon_url('i/settings');

      The reason for the slightly ugly function names is what we are planning (http://moodle.org/mod/forum/discuss.php?d=126543) to change the default icon-set in Moodle. When that happens we will probably adopt new names for all the icons. At that point, I propose that we move to a function $OUTPUT->icon_url($icon, $module); where $module is like the same argument to get_string. That is, we let all types of plugins supply their own icons in future, not just modules.

      Note that the API does not include the file extension. We should not hard-code this. Instead themes should be able to choose png or gif.

      Also, eventually, icon_url should do a search of theme, parent theme, standard theme, like smartpix.php does.

            timhunt Tim Hunt
            timhunt Tim Hunt
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

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