Moodle

f4v extension not recognised as web_video filetype

Details

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

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 - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

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

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: