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

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

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4.1, 2.6.8, 3.2.2, BACKEND
    • Fix Version/s: 3.3
    • Component/s: Files API
    • Labels:
    • Testing Instructions:
      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
    • Affected Branches:
      MOODLE_24_STABLE, MOODLE_26_STABLE, MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_33_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-38278-pptx-et-al-presentation-group

      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

              Assignee:
              lucaboesch Luca Bösch
              Reporter:
              jaywink Jason Robinson
              Peer reviewer:
              Marina Glancy
              Integrator:
              Dan Poltawski
              Tester:
              Damyon Wiese
              Participants:
              Component watchers:
              Matteo Scaramuccia, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              3 Vote for this issue
              Watchers:
              8 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                15/May/17