Moodle
  1. Moodle
  2. MDL-29955

f4v extension not recognised as web_video filetype

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.1.2, 2.2
        • Fix Version/s: 2.0.6, 2.1.3
        • Component/s: Files API, Repositories
        • Labels:
          None
        • Testing Instructions:
          Hide

          You'll need a repository plugin which only accepts web video. The one I'm testing with is a custom plugin for a client's specific requirements.

          1. Use your repository plugin to navigate to a folder which should contain an f4v file
          2. Videos with a .f4v extension are not listed.

          After applying the supplied patches, you'll need to refresh the page. .f4v files are now listed

          Show
          You'll need a repository plugin which only accepts web video. The one I'm testing with is a custom plugin for a client's specific requirements. Use your repository plugin to navigate to a folder which should contain an f4v file Videos with a .f4v extension are not listed. After applying the supplied patches, you'll need to refresh the page. .f4v files are now listed
        • Affected Branches:
          MOODLE_21_STABLE, MOODLE_22_STABLE
        • Fixed Branches:
          MOODLE_20_STABLE, MOODLE_21_STABLE
        • Pull from Repository:
        • Pull Master Branch:
        • Pull Master Diff URL:
        • Rank:
          19499

          Description

          lib/filestorage/file_types.mm contains various file types under the category 'web_video', but does not include the f4v extension.

          Other extension are probably also missing

            Issue Links

            has been marked as being related by

            Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-30165 Repository Filter performs case sensitive search on file extensions

            • Critical - Crashes server, loss of data, severe memory leak
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Rossiani Wijaya added a comment -

              Thanks Andrew for the patch.

              I updated the patch to add more extensions and approved by DS.

              Submitting for integration.

              Show
              Rossiani Wijaya added a comment - Thanks Andrew for the patch. I updated the patch to add more extensions and approved by DS. Submitting for integration.
              Hide
              Permalink
              Eloy Lafuente (stronk7) added a comment -

              The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

              TIA and ciao

              Show
              Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
              Hide
              Permalink
              Sam Hemelryk added a comment -

              Thanks Rosie, this has been integrated now.
              Could you please update the testing instructions on this issue.

              Thanks
              Sam

              Show
              Sam Hemelryk added a comment - Thanks Rosie, this has been integrated now. Could you please update the testing instructions on this issue. Thanks Sam
              Hide
              Permalink
              Rajesh Taneja added a comment -

              Thanks for fixing this Andrew and Rossie
              Works Great.

              Show
              Rajesh Taneja added a comment - Thanks for fixing this Andrew and Rossie Works Great.
              Hide
              Permalink
              Eloy Lafuente (stronk7) added a comment -

              Yes, you got this finally upstream, just in time for Moodle 2.2beta. Congrats and thanks!

              Ciao

              Show
              Eloy Lafuente (stronk7) added a comment - Yes, you got this finally upstream, just in time for Moodle 2.2beta. Congrats and thanks! Ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: