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

          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: