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:

          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

            Gliffy Diagrams

              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: