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:

      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

            Activity

            Hide
            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
            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
            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
            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
            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: