Ticket #1725 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

Error when saving book with video (filename with space)

Reported by: deyan Owned by: sriggins
Priority: critical Milestone: M09_BETA1
Component: uncategorized Version: 2.0
Keywords: Cc:
Ticket_group: Effort:
Category: unknown Importance:
Estimated Number of Hours: Add Hours to Ticket:
Billable?: Total Hours:
Analysis_owners: deyan Design_owners: sriggins
Imp._owners: sriggins Test_owners:
Design_reviewers: meddle Analysis_reviewers: meddle
Changelog: Imp._reviewers: meddle
Test_reviewers: dido Analysis_score: 3.5
Design_score: 3.5 Imp._score: 3.5
Test_score: 0

Description

Create a new book, insert a video frame, save the book as, an error appears

Attachments

videobug.txt Download (1.6 MB) - added by deyan 3 years ago.
demo video.avi Download (1.5 MB) - added by deyan 3 years ago.

Change History

Changed 3 years ago by deyan

Changed 3 years ago by deyan

  • owner set to deyan
  • status changed from new to s1a_analysis_started

Changed 3 years ago by deyan

  • priority changed from major to critical
  • status changed from s1a_analysis_started to s1b_analysis_finished

Changed 3 years ago by deyan

Changed 3 years ago by deyan

Happens when the videofile has space in it's name.

Changed 3 years ago by deyan

  • summary changed from Error when saving book with video to Error when saving book with video (filename with space)

Changed 3 years ago by deyan

Seems to happen with audio too.

Changed 3 years ago by sriggins

The movie is indeed copied, and encoded, but the folder the resource creates is not encoded

Changed 3 years ago by sriggins

File in the file system is:

/var/folders/IY/IYc9-0-WHNKv-n6jayvDRE+++TI/-Tmp-/tempFile5740849472677483542demo%20video.avi

but in the url it is:

/var/folders/IY/IYc9-0-WHNKv-n6jayvDRE+++TI/-Tmp-/tempFile5740849472677483542demo%2520video.avi

with the difference being that the %20 changed to %2520

Changed 3 years ago by sriggins

  • status changed from s1b_analysis_finished to s1c_analysis_ok

BinDataSource>>createFile toURI converts the %20 in the existing URL to %2520

Changed 3 years ago by sriggins

  • owner changed from deyan to sriggins
  • status changed from s1c_analysis_ok to s2a_design_started

Changed 3 years ago by sriggins

  • design_owners set to sriggins
  • status changed from s2a_design_started to s2b_design_finished
  • implementation_owners set to sriggins

Changed 3 years ago by sriggins

  • status changed from s2b_design_finished to s2c_design_ok

Changed 3 years ago by sriggins

  • status changed from s2c_design_ok to s3a_implementation_started

Changed 3 years ago by sriggins

  • status changed from s3a_implementation_started to s3b_implementation_finished

branch /branches/private/sriggins/BUG_1725_ERROR_SAVING_BOOK_WITH_MEDIA

Change BinDataSource >> createFile to decode the filename before creating the file. Otherwise we end up with %20 in some places, and %2520 in others. After seeing bug reports about how 1.6 changed how toURI works in regards to endlessly encoding % to %25, I decided that decoding (via getPath) before passing to File would be the safest bet given the time before this release.

Changed 3 years ago by sriggins

I ran /org.sophie2.main.func.media/src/test/java/org/sophie2/main/func/media/VideoFramePersistTest.java but it fails with or without my change

Changed 3 years ago by meddle

  • status changed from s3b_implementation_finished to s3c_implementation_ok
  • analysis_reviewers set to meddle
  • analysis_score changed from 0 to 3.5
  • implementation_reviewers set to meddle
  • design_score changed from 0 to 3.5
  • design_reviewers set to meddle
  • implementation_score changed from 0 to 3.5

Merged to the trunk in [3936]. Additionally I fixed the test -> /org.sophie2.main.func.media/src/test/java/org/sophie2/main/func/media/VideoFramePersistTest.java. The problem wast that the video content in it was not created in it's frame. That is not a problem connected directly to that bug, but was a problem with the test that tested the persistence of video data.

3.5p (20m)

Changed 3 years ago by dido

  • status changed from s3c_implementation_ok to closed
  • test_reviewers set to dido
  • resolution set to fixed
Note: See TracTickets for help on using tickets.