Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: dropping pdf url to board #1157

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

letsfindaway
Copy link
Collaborator

This PR solves a problem when dropping a PDF url to the board. Expected behavior is that the same functions are executed as when dropping a PDF document to the board. Instead in most cases just nothing happens.

  • when dropping a url to the board, the data is downloaded and then processed in UBBoardController::downloadFinished
  • for a pdf, the data is written to a temporary file and then imported
  • before importing, the file must be closed to write pending data from buffers to disk
  • however the file was only closed after the import, so the data was not completely written to disk

- when dropping a url to the board, the data is downloaded and
  then processed in UBBoardController::downloadFinished
- for a pdf, the data is written to a temp file and then imported
- before importing, the file must be closed to write pending data
  from buffers to disk
@letsfindaway
Copy link
Collaborator Author

See also #1209 reporting another way to provoke this bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant