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.

Sign up for GitHub

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

Jump to bottom

Incompatible with acrobat plugin #40

Closed
tomyan112 opened this issue Feb 22, 2020 · 10 comments
Closed

Incompatible with acrobat plugin #40

tomyan112 opened this issue Feb 22, 2020 · 10 comments
Labels
Bug Something isn't working Fixed Fixed, but may not be released yet

Comments

@tomyan112
Copy link

image
Everytime I create pdf with the plugin this error pop up.
image
@MikeWilliams-UK
Copy link
Contributor

@tomyan112 Could you please supply details of how we can obtain the Acrobat Plug In?

Without it we would be totally guessing about why Chem4Word crashes.

@tomyan112
Copy link
Author

@tomyan112 Could you please supply details of how we can obtain the Acrobat Plug In?

Without it we would be totally guessing about why Chem4Word crashes.

It is automatically installed with Adobe Acrobat Pro DC

@MikeWilliams-UK
Copy link
Contributor

@tomyan112 further point of clarification.

Does the error occur when you try to create a pdf of any document or just one with our chemistry objects inside it?

@tomyan112
Copy link
Author

@tomyan112 further point of clarification.

Does the error occur when you try to create a pdf of any document or just one with our chemistry objects inside it?

It only occurs when there the document has chemistry objects inside it.

@deadlyvices
Copy link
Contributor

Our chemistry objects are simply a content control with DrawingML, inside, and a matching custom XML part in the document. There is nothing inherently 'chemical' about either. So I can only suppose that Acrobat isn't handling the DrawingML.

@tomyan112
Copy link
Author

Our chemistry objects are simply a content control with DrawingML, inside, and a matching custom XML part in the document. There is nothing inherently 'chemical' about either. So I can only suppose that Acrobat isn't handling the DrawingML.

It did convert the document. However, it crashed afterward.

@MikeWilliams-UK
Copy link
Contributor

@tomyan112 while @deadlyvices is right in what he says it's not the full story.

Chem4Word listens to events such as "DocumentCreated", "DocumentSaved", "DocumentClosed" and many more.
The acid test to see if it is data we have placed in the document or something going wrong in one of our event listeners is to create a document with chemistry in it. Uninstall Chem4Word and see if you get an error.

@tomyan112
Copy link
Author

@tomyan112 while @deadlyvices is right in what he says it's not the full story.

Chem4Word listens to events such as "DocumentCreated", "DocumentSaved", "DocumentClosed" and many more.
The acid test to see if it is data we have placed in the document or something going wrong in one of our event listeners is to create a document with chemistry in it. Uninstall Chem4Word and see if you get an error.

No errors after I uninstalled Chem4Word.

@MikeWilliams-UK
Copy link
Contributor

@tomyan112 thanks for testing this it narrows down where we need to investigate.

@MikeWilliams-UK MikeWilliams-UK added the Bug Something isn't working label Feb 23, 2020
@MikeWilliams-UK MikeWilliams-UK added the Fixed Fixed, but may not be released yet label Mar 4, 2020
MikeWilliams-UK added a commit that referenced this issue Mar 5, 2020
@MikeWilliams-UK
Merged PR 338: V3.1.6 Beta 6
6d3f336 
* ACME displays warning circles when valence violations occur #31
* ACME keyboard focus now returns to the editor #30
* Add ACME Option to render in Monochrome #11
* Add ACME Option to show/hide Implicit Hydrogens #17
* Fix flipping of functional groups #35
* Fix crash when using Create PDF button on Adobe Acrobat Word Add-In #40
* Correct rendering of double bond of norbornene #34
* Correct rendering of terminal double bonds #25
* Silently handle COMException in OnDocumentBeforeSave #42

Related work items: #677, #678, #681, #690, #697, #698, #700, #701
@MikeWilliams-UK
Copy link
Contributor

Fixed by beta-6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Something isn't working Fixed Fixed, but may not be released yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tomyan112 @deadlyvices @MikeWilliams-UK