Defect #32153

Repository browser does not render previews for audio/video files

Added by Go MAEDA 7 months ago. Updated 2 months ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Go MAEDA% Done:

0%

Category:SCM
Target version:4.2.0
Resolution:Fixed 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

Associated revisions

Revision 19492
Added by Go MAEDA 2 months ago

Repository browser does not render previews for audio/video files (#32153).

Patch by Mizuki ISHIKAWA.

History

#1 Updated by Go MAEDA 7 months ago

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

#2 Updated by Go MAEDA 7 months ago

Attaching a quick-and-dirty workaround.

#3 Updated by Anonymous 6 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 Anonymous 4 months 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 4 months ago

  • Target version set to Candidate for next major release

#6 Updated by Mizuki ISHIKAWA 4 months 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 3 months 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.

#8 Updated by Go MAEDA 2 months ago

  • Status changed from New to Resolved
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the patch. Thank you for your contribution.

#9 Updated by Go MAEDA 2 months ago

  • Status changed from Resolved to Closed
  • Target version changed from 4.1.1 to 4.2.0

Also available in: Atom PDF