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

PPTX and other similar filetypes should be listed in 'presentation' group

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.4.1, 2.6.8, 3.2.2, BACKEND
    • 3.3
    • Files API
    • MOODLE_24_STABLE, MOODLE_26_STABLE, MOODLE_32_STABLE
    • MOODLE_33_STABLE
    • MDL-38278-pptx-et-al-presentation-group
    • Hide

      I think, there is no way to test with the current code because this will affect developers and plugin developers.
      However a small piece of code can be written, something like this:

      $mform->addElement('filemanager', 'attachments', get_string('attachment', 'moodle'), null,
                          array('subdirs' => 0, 'maxbytes' => $maxbytes, 'maxfiles' => 50,
                                'accepted_types' => array('presentation'), 'return_types'=> FILE_INTERNAL | FILE_EXTERNAL));
      

      You must add a file and see if it is accepted by the filemanager to be uploaded

      Show
      I think, there is no way to test with the current code because this will affect developers and plugin developers. However a small piece of code can be written, something like this: $mform->addElement('filemanager', 'attachments', get_string('attachment', 'moodle'), null, array('subdirs' => 0, 'maxbytes' => $maxbytes, 'maxfiles' => 50, 'accepted_types' => array('presentation'), 'return_types'=> FILE_INTERNAL | FILE_EXTERNAL)); You must add a file and see if it is accepted by the filemanager to be uploaded

    Description

      Filelib does not define pptx files in the group 'presentation'. Ppt groups are defined in this group.

      Will gladly submit fix for this as done on our own repo.

      Attachments

        Issue Links

          Activity

            People

              lucaboesch Luca Bösch
              jaywink Jason Robinson
              Marina Glancy Marina Glancy
              Dan Poltawski Dan Poltawski
              Damyon Wiese Damyon Wiese
              Matteo Scaramuccia, Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              3 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                15/May/17