Mezomish / juffed

JuffEd text editor
GNU General Public License v2.0
56 stars 19 forks source link

Fix start crash if file manager plugin is disabled #86

Closed philippwiesemann closed 2 years ago

philippwiesemann commented 8 years ago

The file manager plugin already created widgets while it was temporarily loaded for getting its attributes on application start. The new TreeView widget then caused a crash later because its implementation was unloaded with the deactivated plugin. This was fixed by now creating the widgets in the plugin's init() method which is only called for enabled plugins.

This might also fix issue #82.

Thanks!

q4a commented 2 years ago

@philippwiesemann Hi. I see that this commit was made more than five years old. I cherry-picked your commit, include it in big PR and it was accepted to master reccently: https://github.com/Mezomish/juffed/commit/718e2c7883167f2f94d7744208a9c59d0437ac87 @Mezomish I think, that you can close it.