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

Crashes Zotero when adding items #26

Open
champagnealexandre opened this issue Sep 22, 2022 · 8 comments
Open

Crashes Zotero when adding items #26

champagnealexandre opened this issue Sep 22, 2022 · 8 comments

Comments

@champagnealexandre
Copy link

Hi,

The citationcounts plugin crashes Zotero when used with some other plugins at the same time.

See issue opened in the repo of one of those plugins: windingwind/zotero-actions-tags#36

I have also posted a discussion on the Zotero forum here: https://forums.zotero.org/discussion/99865/plugins-causing-segmentation-fault-when-used-together-citation-counts-manager-zotero-tag?new=1

Any idea what might be causing this?

Thanks.

@retorquere
Copy link

Removing this line may help. The Zotero object is already loaded in the overlay, and there seems to be a problem when it's reloaded when already available.

@champagnealexandre
Copy link
Author

Thanks @retorquere ; I’m not familiar with Zotero plugin development so I can’t build/test but I hope @eschnett will take that into account.

@pencilheart
Copy link

pencilheart commented Oct 4, 2022

Removing this line may help. The Zotero object is already loaded in the overlay, and there seems to be a problem when it's reloaded when already available.

It doesn't work. The modified .xpi can't be installed (show below).
The add-on "%S" could not be installed. It may be incompatible with this version of Zotero.

@retorquere
Copy link

That's zotero's way of telling you the xpi you have is not a zip file. If you changed the xpi, you haven't put it back together right.

@pencilheart
Copy link

@retorquere Thank you. You're right. It works when I "select the files and folders at root level and zip from here" instead of "zip from outside the plugin folder".

@champagnealexandre Here are the steps. #17 (comment)

@champagnealexandre
Copy link
Author

@pencilheart worked perfectly, thank you! Thanks @retorquere too for your help.

@champagnealexandre
Copy link
Author

Hmm, for some reason the first article I added after doing the modification @retorquere suggested was added correctly, but now I’ve just added a second article and Zotero crashed again. Disabling again the plugin and no more crashes..

@pencilheart
Copy link

me too....

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

No branches or pull requests

3 participants