cfsimplicity / spreadsheet-cfml

Standalone library for working with spreadsheets and CSV in CFML
MIT License
127 stars 36 forks source link

OSGI bundle error #289

Closed Daemach closed 2 years ago

Daemach commented 2 years ago

I occasionally get the following error when trying to use cfml-spreadsheet. I don't know what causes it, but it requires a service restart to clear.

cfsimplicity commented 2 years ago

Did you upload the right screenshot? Not seeing any OSGi bundle error there or anything to do with the library??

cfsimplicity commented 2 years ago

If you are getting OSGi errors with the library, particularly after upgrading, try calling this method:

spreadsheet.flushOsgiBundle()
Daemach commented 2 years ago

Bah - no, this is the correct one. I wonder if that could go in a try/catch in the library itself?

image

cfsimplicity commented 2 years ago

Possibly. Let me give it some thought.

cfsimplicity commented 2 years ago

OK, that's done. I'm not sure why it happens as when a new bundle version is requested it should just be loaded alongside the previous one. But I have seen that error myself when upgrading, hence the flushing method.

The condition should be caught now. Difficult to auto-test, but I did a dummy bundle version upgrade and tested manually and it no longer errored.