search

JChemPaint / jchempaint

Chemical 2D structure editor application/applet based on the Chemistry Development Kit
http://jchempaint.github.com
110 stars 34 forks source link

JS messaging: getSmiles() and getStereoSmiles() functions do not respond in version Applet-3.3-1209 #149

Open neuenschwanm opened 11 years ago

neuenschwanm commented 11 years ago

When using the example html page on http://jchempaint.github.com/EditorApplet.html or when using the corresponding class files in a custom webpage, it appears that while the applet itself works nicely its funcions getSmiles() and getStereoSmiles() do not invoke any response when called. This problem does not appear with the nigthly build version on http://www.ebi.ac.uk/steinbeck-srv/jchempaint-nightly/

rwst commented 11 years ago

This happens with both sun-jdk and icedtea-web. Inclusion of cdk-logging.jar gives:

Imgur

rwst commented 11 years ago

The issue can be worked around by first selecting from menu Tools --> Create SMILES before accessing the function via Javascript. Obviously this suggests an initialization problem. Applies to #140 as well.

rwst commented 11 years ago

I'm pretty sure that this is also an icedtea-web (OpenJDK) issue. I have tested 3.3-1209 with both OpenJDK and Sun JDK and this didn't show with Sun JDK. Can you please try with the latest Sun Java plugin for your browser?

neuenschwanm commented 11 years ago

Hi, with the 3.3-1210 version available now the stereochemistry as well as the GetSmiles() function work fine! Thanks a lot!

Dr. Martin Neuenschwander Screening Unit/ AG Kries Institute of Molecular Pharmacology FMP Campus Berlin-Buch Robert-Roessle Str. 10 13125 Berlin Tel +493094062906 Fax +493094062948

rwst notifications@github.com 08.10.2012 17:43 >>>

I'm pretty sure that this is also an icedtea-web (OpenJDK) issue. I have tested 3.3-1209 with both OpenJDK and Sun JDK and this didn't show with Sun JDK. Can you please try with the latest Sun Java plugin for your browser? — Reply to this email directly or view it on GitHub.