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

MOODLE_17_STABLE missing file moodle/pix/i/approve.gif used by database module

    Details

    • Affected Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      MOODLE_17_STABLE missing file moodle/pix/i/approve.gif used by database module.

      This makes it impossible to approve database entries in safari, which apparently doesn't fall back to the alt text appropriately, and makes firefox display the string "approve" rather than the image approve.gif.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            jballard James Ballard added a comment -

            This is still missing from 1.8 and 1.9 as well.

            It needs a new icon in the right location or I guess pix/t/clear.gif may be suitable.

            In mod/data/lib.php:

            $patterns[]='##approve##';
            if (has_capability('mod/data:approve', $context) && ($data->approval) && (!$record->approved))

            { $replacement[] = '<a href="'.$CFG->wwwroot.'/mod/data/view.php?d='.$data->id.'&approve='.$record->id.'&sesskey='.sesskey().'"><img src="'.$CFG->pixpath.'/i/approve.gif" class="iconsmall" alt="'.get_string('approve').'" /></a>'; }

            else

            { $replacement[] = ''; }

            Could be:

            $patterns[]='##approve##';
            if (has_capability('mod/data:approve', $context) && ($data->approval) && (!$record->approved)){ $replacement[] = '<a href="'.$CFG->wwwroot.'/mod/data/view.php?d='.$data->id.'&approve='.$record->id.'&sesskey='.sesskey().'"><img src="'.$CFG->pixpath.'/t/clear.gif" class="iconsmall" alt="'.get_string('approve').'" /></a>'; } else { $replacement[] = ''; }

            I added a green tick, approve.gif to the folder instead.

            Yours
            James

            Show
            jballard James Ballard added a comment - This is still missing from 1.8 and 1.9 as well. It needs a new icon in the right location or I guess pix/t/clear.gif may be suitable. In mod/data/lib.php: $patterns[]='##approve##'; if (has_capability('mod/data:approve', $context) && ($data->approval) && (!$record->approved)) { $replacement[] = '<a href="'.$CFG->wwwroot.'/mod/data/view.php?d='.$data->id.'&approve='.$record->id.'&sesskey='.sesskey().'"><img src="'.$CFG->pixpath.'/i/approve.gif" class="iconsmall" alt="'.get_string('approve').'" /></a>'; } else { $replacement[] = ''; } Could be: $patterns[]='##approve##'; if (has_capability('mod/data:approve', $context) && ($data->approval) && (!$record->approved)){ $replacement[] = '<a href="'.$CFG->wwwroot.'/mod/data/view.php?d='.$data->id.'&approve='.$record->id.'&sesskey='.sesskey().'"><img src="'.$CFG->pixpath.'/t/clear.gif" class="iconsmall" alt="'.get_string('approve').'" /></a>'; } else { $replacement[] = ''; } I added a green tick, approve.gif to the folder instead. Yours James
            Hide
            jballard James Ballard added a comment -

            A file for those who want it.

            Show
            jballard James Ballard added a comment - A file for those who want it.
            Hide
            sbourget Stephen Bourget added a comment -

            This seems to also affect both Moodle 1.8 and Moodle 1.9

            Show
            sbourget Stephen Bourget added a comment - This seems to also affect both Moodle 1.8 and Moodle 1.9
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Confirmed... assigning to DC:

            1) There is one "check.gif" icon in the glossary module.
            2) There is another alternative here, courtesy of James Ballard. B-)
            3) Please confirm with MD what one to use.

            In any case I would propose to (confirm with MD too).

            1) Move the decided icon to pix/i/approve.gif (17_STABLE to HEAD)
            2) Move glossary usages of mod/glossary/check.gif to pix/i/approve.gif (19_STABLE and HEAD)
            3) Test both modules icon appearance.

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Confirmed... assigning to DC: 1) There is one "check.gif" icon in the glossary module. 2) There is another alternative here, courtesy of James Ballard. B-) 3) Please confirm with MD what one to use. In any case I would propose to (confirm with MD too). 1) Move the decided icon to pix/i/approve.gif (17_STABLE to HEAD) 2) Move glossary usages of mod/glossary/check.gif to pix/i/approve.gif (19_STABLE and HEAD) 3) Test both modules icon appearance. Ciao
            Hide
            dongsheng Dongsheng Cai added a comment -

            For Branch 1.7 and 1.8
            a new approve.gif was added
            For Branch 1.9 and HEAD
            check.gif in mod/glossary was moved to pix/i/approve.gif, the usages of check.gif was changed to pix/i/approve.gif

            Show
            dongsheng Dongsheng Cai added a comment - For Branch 1.7 and 1.8 a new approve.gif was added For Branch 1.9 and HEAD check.gif in mod/glossary was moved to pix/i/approve.gif, the usages of check.gif was changed to pix/i/approve.gif
            Hide
            sbourget Stephen Bourget added a comment -

            The fix seems to be incomplete as it is still broken in the chameleon theme under Moodle 1.9.
            (The chameleon theme has it's own set of icons see http://cvs.moodle.org/moodle/theme/chameleon/pix/i/)

            Show
            sbourget Stephen Bourget added a comment - The fix seems to be incomplete as it is still broken in the chameleon theme under Moodle 1.9. (The chameleon theme has it's own set of icons see http://cvs.moodle.org/moodle/theme/chameleon/pix/i/ )
            Hide
            dongsheng Dongsheng Cai added a comment -

            Add approve.gif file to chameleon them, I dug a little deeper, find this:
            $CFG->pixpath = $CFG->themewww .'/'. $theme .'/pix';
            $CFG->modpixpath = $CFG->themewww .'/'. $theme .'/pix/mod';
            $CFG->pixpath may be different when we use different them.

            Show
            dongsheng Dongsheng Cai added a comment - Add approve.gif file to chameleon them, I dug a little deeper, find this: $CFG->pixpath = $CFG->themewww .'/'. $theme .'/pix'; $CFG->modpixpath = $CFG->themewww .'/'. $theme .'/pix/mod'; $CFG->pixpath may be different when we use different them.
            Hide
            scyrma Mathieu Petit-Clair added a comment -

            QA-Tuesday: made sure no other core theme needed it... Closed.

            Show
            scyrma Mathieu Petit-Clair added a comment - QA-Tuesday: made sure no other core theme needed it... Closed.

              People

              • Votes:
                1 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  15/May/08