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

Save callback is not called by client-browser when there are images to encode #187

Closed
Malangs opened this issue Sep 15, 2017 · 2 comments
Closed
Assignees
Milestone

Comments

@Malangs
Copy link

Malangs commented Sep 15, 2017

I've been upgrading from an older version to the latest release and encountered the following bug: If I embed an image in the presentation, the callback in save does not work. I've only tested it in the browser but I suspect the same behavior occurs in Node.js. After reverse engineering the code a bit I think I found the problem. If there is an image to encode, the last asynchronous image convert function calls doExportPresentation, otherwise the latter will be called by save directly. However, the image converting functions (encodeImageRelations --> convertImgToDataURLviaCanvas --> callbackImgToDataURLDone) do not propagate the callback. Hope that helps.

@gitbrent gitbrent self-assigned this Sep 15, 2017
@gitbrent gitbrent changed the title Callback is not called if there are images to encode Save callback is not called by client-browser when there are images to encode Sep 17, 2017
@gitbrent gitbrent added this to the 1.9.0 milestone Sep 17, 2017
gitbrent pushed a commit that referenced this issue Sep 17, 2017
@gitbrent
Copy link
Owner

Hi @Malangs ,

Thanks for reporting this and for taking the time to identify the root cause. As you discovered the callback was not being propagated.

Callbacks were working on Node.js (using a callback is the default for the node demo), I had just neglected to test this functionality in browsers for awhile.

@Malangs
Copy link
Author

Malangs commented Sep 18, 2017

Hi @gitbrent, thanks for the fix! Glad I could help.

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

No branches or pull requests

2 participants