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

f4v extension not recognised as web_video filetype

          Details

          • Type: Bug
          • Status: Closed
          • Priority: 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

                Attachments

                  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
                    rwijaya 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
                    rwijaya 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
                    stronk7 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
                    stronk7 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
                    samhemelryk 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
                    samhemelryk 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
                    rajeshtaneja Rajesh Taneja added a comment -

                    Thanks for fixing this Andrew and Rossie
                    Works Great.

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

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

                    Ciao

                    Show
                    stronk7 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:
                          Fix Release Date:
                          28/Nov/11