Defect #32153

Repository browser does not render previews for audio/video files

Added by Go MAEDA 4 months ago. Updated 19 days ago.

Status:NewStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:SCM
Target version:4.1.1
Resolution: Affected version:

Description

#27336 added preview for audio/video files to attachments and repositories, however, audio/video preview for repositories is not working. It always shows "No preview available" message for all kind of audio/video files.

no-preview-available@2x.png (40.2 KB) Go MAEDA, 2019-09-26 17:24

32153-workaround.patch Magnifier (2.34 KB) Go MAEDA, 2019-09-26 17:26

0002-Add-test.patch Magnifier (166 KB) Mizuki ISHIKAWA, 2019-12-23 09:29


Related issues

Related to Redmine - Patch #27336: Render previews for audio and video files Closed

History

#1 Updated by Go MAEDA 4 months ago

  • Related to Patch #27336: Render previews for audio and video files added

#2 Updated by Go MAEDA 4 months ago

Attaching a quick-and-dirty workaround.

#3 Updated by Antonio McDeal 4 months ago

Go MAEDA wrote:

Attaching a quick-and-dirty workaround.

Thanks :-)
I'll share more feedback after I test it if I'll find more bugs

#4 Updated by Antonio McDeal about 1 month ago

Update: this seems to be working with no problems so far, if somebody else could double check it for security sake, it would be a really cool fix to commit ;-)

#5 Updated by Go MAEDA about 1 month ago

  • Target version set to Candidate for next major release

#6 Updated by Mizuki ISHIKAWA about 1 month ago

If you need tests to commit 32153-workaround.patch, add tests based on the attached patch.

  • 0002-Add-test.patch is a patch that should be applied after 32153-workaround.patch is applied
  • 0002-Add-test.patch adds a .mp4 file to the test subversion_repository.
    • rake test:scm:update command must be executed after applying the patch for the test to succeed

#7 Updated by Go MAEDA 19 days ago

  • Target version changed from Candidate for next major release to 4.1.1

Mizuki ISHIKAWA wrote:

If you need tests to commit 32153-workaround.patch, add tests based on the attached patch.

Thanks. Setting the target version to 4.1.1.

Also available in: Atom PDF