Dominic-DallOsto / zotero-reading-list

Keep track of whether you've read items in Zotero
GNU General Public License v3.0
222 stars 5 forks source link

Adding attachments directly or renaming attachments with the zotero Attanger plugin will cause zotero to crash. #31

Closed Fan-tas-tic closed 4 weeks ago

Fan-tas-tic commented 2 months ago

与zotero style插件同时使用时会出现所有文献条目都不显示的问题,可能存在冲突。

禁用此插件后问题解决。 zotero版本:7.0.0-beta.87+f59a4da7f (64-bit) zotero-reading-list版本:1.3.1


(deepl translation)

When used in conjunction with the zotero style plugin there is a problem where all literature entries are not displayed, there may be a conflict.

Disabling this plugin solves the problem. zotero version: 7.0.0-beta.87+f59a4da7f (64-bit) zotero-reading-list version: 1.3.1

Dominic-DallOsto commented 2 months ago

Hi,

Thanks for the error report.

Could you possibly clarify the issue a little bit? In the issue title you mention the Attanger plugin, but in the issue body you mention the Zotero Style plugin. If there are actually problems with both, could you make a separate github issue for each to make it easier to track?

Also, could you post a picture/video of the issue, a debug log, or more detailed steps showing how to reproduce it?

Thanks!

On 14/06/2024 5:18 am, Fan-tas-tic wrote:

与zotero style插件同时使用时会出现所有文献条目都不显示的问题,可能存在冲突。

禁用此插件后问题解决。 zotero版本:7.0.0-beta.87+f59a4da7f (64-bit) zotero-reading-list版本:1.3.1

— Reply to this email directly, view it on GitHub https://github.com/Dominic-DallOsto/zotero-reading-list/issues/31, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGM5S3DTAHPJ7EF4ISC3QA3ZHJOHDAVCNFSM6AAAAABJJREF6OVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2TENBQGQYDMOI. You are receiving this because you are subscribed to this thread.Message ID: @.***>

--------------mY5aKD7rUXYybPQbGzA5LjJM Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 8bit

<!DOCTYPE html>

Hi,

Thanks for the error report.

Could you possibly clarify the issue a little bit? In the issue title you mention the Attanger plugin, but in the issue body you mention the Zotero Style plugin. If there are actually problems with both, could you make a separate github issue for each to make it easier to track?

Also, could you post a picture/video of the issue, a debug log, or more detailed steps showing how to reproduce it?


Thanks!

On 14/06/2024 5:18 am, Fan-tas-tic wrote:

与zotero style插件同时使用时会出现所有文献条目都不显示的问题,可能存在冲突。

禁用此插件后问题解决。
zotero版本:7.0.0-beta.87+f59a4da7f (64-bit)
zotero-reading-list版本:1.3.1


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <Dominic-DallOsto/zotero-reading-list/issues/31@github.com>


--------------mY5aKD7rUXYybPQbGzA5LjJM--

Qingchan2020 commented 2 months ago

这个问题确实存在,本插件与attanger共同使用,在抓取文献与PDF标注的过程中都会存在卡死的情况。禁用本插件之后就不会卡死


(deepl translation)

This problem does exist, this plug-in and attanger used together, in the process of grabbing the literature and PDF annotation will exist in the case of jamming. Disable the plug-in will not be stuck after

Dominic-DallOsto commented 2 months ago

Ok, thanks for describing the issue. Would you also be able to provide the output in the error console (Tools -> Developer -> Error Console) so I can think what might be causing the issue? I don't use the attanger plugin so I can't think what might be causing the issue

gitlzzz commented 1 month ago

Hi, I also experienced the crash when highlighting words in a PDF on both Linux and Windows before disabling this plugin. I think I didn't have the attanger plugin.

I leave the error report here.


18:00:54.187 1721664054187  addons.xpi  WARN    Checking /home//code/Zotero_linux-x86_64/distribution/extensions for addons
18:00:54.244 1721664054243  addons.xpi-utils    WARN    addMetadata: Add-on undefined is invalid: Error: Invalid addon ID: expected addon ID undefined, found undefined in manifest(resource://gre/modules/addons/XPIDatabase.jsm:3108:15) JS Stack trace: addMetadata@XPIDatabase.jsm:3108:15
processFileChanges@XPIDatabase.jsm:3569:26
checkForChanges@XPIProvider.jsm:3121:60
startup@XPIProvider.jsm:2532:12
callProvider@AddonManager.sys.mjs:228:31
_startProvider@AddonManager.sys.mjs:536:17
startup@AddonManager.sys.mjs:742:14
startup@AddonManager.sys.mjs:3666:26
observe@amManager.sys.mjs:73:29
18:00:54.261 NS_ERROR_ILLEGAL_VALUE: Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsICommandLine.handleFlagWithParam] BrowserContentHandler.sys.mjs:428
    bch_handle resource:///modules/BrowserContentHandler.sys.mjs:428
18:00:54.264 NS_ERROR_ILLEGAL_VALUE: Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsICommandLine.handleFlagWithParam]
    dch_handle resource:///modules/BrowserContentHandler.sys.mjs:1289
BrowserContentHandler.sys.mjs:1308:15
    dch_handle resource:///modules/BrowserContentHandler.sys.mjs:1308
18:00:54.264 Warning: unrecognized command line flag -url
BrowserContentHandler.sys.mjs:1344:17
    dch_handle resource:///modules/BrowserContentHandler.sys.mjs:1344
18:00:54.591 1721664054591  addons.webextension.Knowledge4Zotero@windingwind.com    WARN    Loading extension 'Knowledge4Zotero@windingwind.com': Reading manifest: Warning processing version: version must be a version string consisting of at most 4 integers of at most 9 digits without leading zeros, and separated with dots
18:00:54.595 1721664054595  addons.webextension.zoteropdftranslate@euclpts.com  WARN    Loading extension 'zoteropdftranslate@euclpts.com': Reading manifest: Warning processing version: version must be a version string consisting of at most 4 integers of at most 9 digits without leading zeros, and separated with dots
18:00:54.607 1721664054607  addons.webextension.tara@linxzh.com WARN    Loading extension 'tara@linxzh.com': Reading manifest: Warning processing version: version must be a version string consisting of at most 4 integers of at most 9 digits without leading zeros, and separated with dots
18:00:54.659 unreachable code after return statement util.js:205:4
18:00:54.888 Error: Addon must include an id, version, and type 2 XPIProvider.jsm:1723:13
18:00:55.120 Missing resource in locale en-AU: browser/browserSets.ftl
18:00:55.121 Missing resource in locale en-CA: browser/browserSets.ftl
18:00:55.129 Missing resource in locale en-NZ: browser/browserSets.ftl
18:00:55.384 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:00:55.508 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:00:55.515 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:00:55.522 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:00:56.065 patching  getField tara.js:451:23
18:00:56.066 patching  setField tara.js:451:23
18:00:56.066 patching  isFieldOfBase tara.js:451:23
18:00:56.066 patching  getField tara.js:451:23
18:00:56.066 patching  setField tara.js:451:23
18:00:56.067 patching  isFieldOfBase tara.js:451:23
18:00:56.067 patching  getField tara.js:451:23
18:00:56.067 patching  setField tara.js:451:23
18:00:56.067 patching  isFieldOfBase tara.js:451:23
18:00:56.068
<style id="prompt-style">
tara.js:451:23
18:00:56.069
<div style="position: fixed; left: 0…th: 100%; height: 100%;">
tara.js:451:23
18:00:56.069
<div style="display: none;">
tara.js:451:23
18:00:56.070
<input class="prompt-input" type="text" placeholder="Select a command...">
tara.js:451:23
18:00:56.070
<div class="cta">
tara.js:451:23
18:00:56.070
<div class="input-container">
tara.js:451:23
18:00:56.070
<div class="commands-containers">
tara.js:451:23
18:00:56.071
<span class="key">
tara.js:451:23
18:00:56.071
<span>
tara.js:451:23
18:00:56.071
<div class="instruction">
tara.js:451:23
18:00:56.071
<span class="key">
tara.js:451:23
18:00:56.071
<span>
tara.js:451:23
18:00:56.071
<div class="instruction">
tara.js:451:23
18:00:56.072
<span class="key">
tara.js:451:23
18:00:56.072
<span>
tara.js:451:23
18:00:56.072
<div class="instruction">
tara.js:451:23
18:00:56.072
<div class="instructions">
tara.js:451:23
18:00:56.072
<div id="zotero-plugin-toolkit-prompt" class="prompt-container">
tara.js:451:23
18:00:56.099 patching  getField betterauthors.js:451:23
18:00:56.099 patching  setField betterauthors.js:451:23
18:00:56.099 patching  isFieldOfBase betterauthors.js:451:23
18:00:56.099 patching  getField betterauthors.js:451:23
18:00:56.099 patching  setField betterauthors.js:451:23
18:00:56.099 patching  isFieldOfBase betterauthors.js:451:23
18:00:56.100 patching  getField betterauthors.js:451:23
18:00:56.100 patching  setField betterauthors.js:451:23
18:00:56.100 patching  isFieldOfBase betterauthors.js:451:23
18:00:56.123 patching  getField zoterofigure.js:445:23
18:00:56.123 patching  setField zoterofigure.js:445:23
18:00:56.123 patching  isFieldOfBase zoterofigure.js:445:23
18:00:56.123 patching  getField zoterofigure.js:445:23
18:00:56.123 patching  setField zoterofigure.js:445:23
18:00:56.123 patching  isFieldOfBase zoterofigure.js:445:23
18:00:56.123 patching  getField zoterofigure.js:445:23
18:00:56.123 patching  setField zoterofigure.js:445:23
18:00:56.123 patching  isFieldOfBase zoterofigure.js:445:23
18:00:56.136 patching  getField zotero-reading-list.js:4:5386
18:00:56.136 patching  setField zotero-reading-list.js:4:5386
18:00:56.136 patching  isFieldOfBase zotero-reading-list.js:4:5386
18:00:56.136 patching  getField zotero-reading-list.js:4:5386
18:00:56.136 patching  setField zotero-reading-list.js:4:5386
18:00:56.136 patching  isFieldOfBase zotero-reading-list.js:4:5386
18:00:56.136 patching  getField zotero-reading-list.js:4:5386
18:00:56.136 patching  setField zotero-reading-list.js:4:5386
18:00:56.136 patching  isFieldOfBase zotero-reading-list.js:4:5386
18:00:56.203 patching  refresh tara.js:451:23
18:00:56.204 patching  getField tara.js:451:23
18:00:56.204 patching  setField tara.js:451:23
18:00:56.204 patching  isFieldOfBase tara.js:451:23
18:00:56.204 patching  getField tara.js:451:23
18:00:56.204 patching  setField tara.js:451:23
18:00:56.204 patching  isFieldOfBase tara.js:451:23
18:00:56.204 patching  getField tara.js:451:23
18:00:56.204 patching  setField tara.js:451:23
18:00:56.204 patching  isFieldOfBase tara.js:451:23
18:00:56.205 patching  getField betterauthors.js:451:23
18:00:56.205 patching  setField betterauthors.js:451:23
18:00:56.205 patching  isFieldOfBase betterauthors.js:451:23
18:00:56.206 patching  getField betterauthors.js:451:23
18:00:56.206 patching  setField betterauthors.js:451:23
18:00:56.206 patching  isFieldOfBase betterauthors.js:451:23
18:00:56.206 patching  getField betterauthors.js:451:23
18:00:56.206 patching  setField betterauthors.js:451:23
18:00:56.206 patching  isFieldOfBase betterauthors.js:451:23
18:00:56.207 patching  getField zotero-reading-list.js:4:5386
18:00:56.208 patching  setField zotero-reading-list.js:4:5386
18:00:56.208 patching  isFieldOfBase zotero-reading-list.js:4:5386
18:00:56.208 patching  getField zotero-reading-list.js:4:5386
18:00:56.208 patching  setField zotero-reading-list.js:4:5386
18:00:56.208 patching  isFieldOfBase zotero-reading-list.js:4:5386
18:00:56.208 patching  getField zotero-reading-list.js:4:5386
18:00:56.208 patching  setField zotero-reading-list.js:4:5386
18:00:56.208 patching  isFieldOfBase zotero-reading-list.js:4:5386
18:00:56.219 Missing resource in locale en-AU: browser/browserSets.ftl
18:00:56.220 Missing resource in locale en-CA: browser/browserSets.ftl 2
18:00:56.256 Missing resource in locale en-NZ: browser/browserSets.ftl
18:00:56.259 Missing resource in locale en-AU: browser/browserSets.ftl 3
18:00:56.361 patching  getField zoteropdftranslate.js:458:24
18:00:56.361 patching  setField zoteropdftranslate.js:458:24
18:00:56.361 patching  isFieldOfBase zoteropdftranslate.js:458:24
18:00:56.509 patching  getField zoteropdftranslate.js:458:24
18:00:56.510 patching  setField zoteropdftranslate.js:458:24
18:00:56.510 patching  isFieldOfBase zoteropdftranslate.js:458:24
18:00:56.629 patching  getField zoteroreference.js:1:67817
18:00:56.629 patching  setField zoteroreference.js:1:67817
18:00:56.629 patching  isFieldOfBase zoteroreference.js:1:67817
18:00:56.630 patching  getField zoteroreference.js:1:67817
18:00:56.630 patching  setField zoteroreference.js:1:67817
18:00:56.630 patching  isFieldOfBase zoteroreference.js:1:67817
18:00:56.630 patching  getField zoteroreference.js:1:67817
18:00:56.630 patching  setField zoteroreference.js:1:67817
18:00:56.630 patching  isFieldOfBase zoteroreference.js:1:67817
18:00:56.653 Missing resource in locale en-NZ: browser/browserSets.ftl 3
18:00:56.664 Missing resource in locale en-CA: browser/browserSets.ftl
18:00:56.664 Missing resource in locale en-NZ: browser/browserSets.ftl
18:00:56.671 Uncaught (in promise) undefined
18:00:56.734
<div class="commands-container">
tara.js:451:23
18:00:56.735 patching  viewItems BetterNotes.js:462:24
18:00:56.735 console.trace() BetterNotes.js:463:24
    log jar:file:///home//.zotero/zotero/gh7v08jk.default/extensions/Knowledge4Zotero@windingwind.com.xpi!/chrome/content/scripts/BetterNotes.js:463
    setData jar:file:///home//.zotero/zotero/gh7v08jk.default/extensions/Knowledge4Zotero@windingwind.com.xpi!/chrome/content/scripts/BetterNotes.js:21324
    patchViewItems jar:file:///home//.zotero/zotero/gh7v08jk.default/extensions/Knowledge4Zotero@windingwind.com.xpi!/chrome/content/scripts/BetterNotes.js:35190
    onMainWindowLoad jar:file:///home//.zotero/zotero/gh7v08jk.default/extensions/Knowledge4Zotero@windingwind.com.xpi!/chrome/content/scripts/BetterNotes.js:35964
18:00:56.740 Missing resource in locale en-CA: browser/browserSets.ftl 3
18:00:56.824 TypeError: _0x472b26 is undefined zoteroreference.js:1:1957752
18:00:56.824 Uncaught (in promise) undefined 2
18:00:56.846 Missing resource in locale en-AU: browser/browserSets.ftl 2
18:00:56.890 Uncaught (in promise) undefined
18:00:56.890 Error: Failed to load resource://services-settings/remote-settings.sys.mjs PartitioningExceptionListService.sys.mjs:88:14
18:00:56.890 Missing resource in locale en-NZ: browser/browserSets.ftl
18:00:57.018 Uncaught (in promise) undefined 2
18:00:57.068 original is undefined 2 tara.js:5187
18:00:57.069
<div class="commands-container">
tara.js:451:23
18:00:57.070 Missing resource in locale en-AU: citation-counts.ftl
18:00:57.070 Missing resource in locale en-NZ: citation-counts.ftl
18:00:57.070 Missing resource in locale en-AU: citation-counts.ftl
18:00:57.071 Missing resource in locale en-CA: citation-counts.ftl 2
18:00:57.072 Missing resource in locale en-GB: citation-counts.ftl
18:00:57.072 Missing resource in locale en-CA: citation-counts.ftl
18:00:57.074 Missing resource in locale en-GB: citation-counts.ftl
18:00:57.074 Missing resource in locale en-NZ: citation-counts.ftl
18:00:57.076 Uncaught (in promise) undefined 2
18:00:57.084 Missing resource in locale en-NZ: citation-counts.ftl
18:00:57.102 Missing resource in locale en-AU: citation-counts.ftl
18:00:57.129 Missing resource in locale en-GB: citation-counts.ftl
18:00:57.141 Uncaught (in promise) undefined
18:00:57.995 SyntaxError: JSON.parse: unexpected character at line 1 column 10 of the JSON data 8
18:01:01.049 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.104 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.120 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.129 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.163 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.170 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.178 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.190 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.202 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.210 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.234 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.242 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.250 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.257 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.265 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.272 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.280 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.317 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.325 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.333 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.346 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.383 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.424 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.435 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.455 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.465 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.476 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.484 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.505 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.583 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.592 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.601 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.611 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.660 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.713 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.827 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.852 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.886 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.894 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.902 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.911 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.967 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.975 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:01.983 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.037 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.046 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.054 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.101 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.114 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.147 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.156 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.164 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.171 InstallTrigger is deprecated and will be removed in the future. reader.js:20134:16
18:01:02.287 Security Error: Content at resource://zotero/reader/reader.html may not load or link to chrome://zoterofigure/content/icons/favicon.png.
18:01:02.299 TypeError: this._pdfThumbnails is undefined reader.js:41190:7
18:01:02.408 Missing resource in locale en-CA: zoteropdftranslate-panel.ftl
18:01:02.408 Missing resource in locale en-CA: citation-counts.ftl
18:01:02.411 Partitioned cookie or storage access was provided to “resource://zotero/reader/pdf/web/viewer.html” because it is loaded in the third-party context and dynamic state partitioning is enabled. 3
18:01:02.415 Missing resource in locale en-NZ: zoteropdftranslate-panel.ftl
18:01:02.415 Missing resource in locale en-NZ: citation-counts.ftl
18:01:02.416 Missing resource in locale en-AU: zoteropdftranslate-panel.ftl
18:01:02.416 Missing resource in locale en-AU: citation-counts.ftl
18:01:02.422 Missing resource in locale en-GB: zoteropdftranslate-panel.ftl
18:01:02.422 Missing resource in locale en-GB: citation-counts.ftl
18:01:02.895 PDF fd387b95aea2aebf21edac81064acb89 [1.4 PDFlib+PDI 9.1.2p4 (C++/Win64); modified using iText 4.2.0 by 1T3XT / Arbortext Publishing Engine] (PDF.js: 4.5.0 [b658eb6]) viewer.mjs:18418:13
18:01:21.040 Missing resource in locale en-CA: devtools/client/toolbox.ftl
18:01:21.040 Missing resource in locale en-NZ: devtools/client/toolbox.ftl
18:01:21.041 Missing resource in locale en-AU: devtools/client/toolbox.ftl
18:01:51.254 DOMException: Could not get children of file(/home//.zotero/zotero/Crash Reports/events) because it does not exist CrashManager.sys.mjs:663:17
Dominic-DallOsto commented 1 month ago

Hi @gitlzzz, thanks for the error report! Unfortunately I don't see anything in the error log that looks like an error related to zotero-reading-list. Are you able to reliably reproduce this issue by highlighting specific text in a specific PDF when the reading list plugin is active, but not when you disable it?

Can you say a bit more about exactly how you triggered this issue and what exactly happened. And which versions of Zotero / zotero-reading-list you're using?

gitlzzz commented 1 month ago

@Dominic-DallOsto Thank you for your response and your development. Yes, it is reliably reproduced by enabling the zotero-reading-list. I have tried to disable the plugins I had one by one, and found the problem happens when enabling zotero-reading-list.

Every time I highlight text in the pdf file, not a specific pdf but anyone i tried, the whole zotero is fails to respond, but sometimes if I wait a long time, it responds again. On windows, I noticed the CPU useage become very high for zotero to 10%.

I met this problem on both windows11/zotero7 and ubuntu/zotero7 with zotero-reading-list 1.3.3

Dominic-DallOsto commented 1 month ago

Hmm, that's really strange because zotero-reading-list shouldn't connect with the PDF reader at all, apart from possibly changing the read status when opening an item's PDF attachment.

I can't reproduce this on windows 10. Which version of Zotero 7 are you using?

Do you have the option "Automatically change the read status of items when opening an attached file" enabled? If so, could you disable it and see if that changes the behaviour?

On 23/07/2024 3:06 pm, gitlzzz wrote:

@Dominic-DallOsto https://github.com/Dominic-DallOsto Thank you for your response and your development. Yes, it is reliably reproduced by enabling the zotero-reading-list. I have tried to disable the plugins I had one by one, and found the problem happens when enabling zotero-reading-list.

Every time I highlight text in the pdf file, not a specific pdf but anyone i tried, the whole zotero is fails to respond, but sometimes if I wait a long time, it responds again. On windows, I noticed the CPU useage become very high for zotero to 10%.

I met this problem on both windows11/zotero7 and ubuntu/zotero7 with zotero-reading-list 1.3.3

— Reply to this email directly, view it on GitHub https://github.com/Dominic-DallOsto/zotero-reading-list/issues/31#issuecomment-2245198947, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGM5S3G5P6BJCYAZPYEG7QLZNZILXAVCNFSM6AAAAABJJREF6OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBVGE4TQOJUG4. You are receiving this because you were mentioned.Message ID: @.***>

--------------cFEslnYlfcWuIkzrgwB5ecOc Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 8bit

<!DOCTYPE html>

Hmm, that's really strange because zotero-reading-list shouldn't connect with the PDF reader at all, apart from possibly changing the read status when opening an item's PDF attachment.

I can't reproduce this on windows 10. Which version of Zotero 7 are you using?

Do you have the option "Automatically change the read status of items when opening an attached file" enabled? If so, could you disable it and see if that changes the behaviour?


On 23/07/2024 3:06 pm, gitlzzz wrote:

Thank you for your response and your development. Yes, it is reliably reproduced by enabling the zotero-reading-list. I have tried to disable the plugins I had one by one, and found the problem happens when enabling zotero-reading-list.

Every time I highlight text in the pdf file, not a specific pdf but anyone i tried, the whole zotero is fails to respond, but sometimes if I wait a long time, it responds again. On windows, I noticed the CPU useage become very high for zotero to 10%.

I met this problem on both windows11/zotero7 and ubuntu/zotero7 with zotero-reading-list 1.3.3


Reply to this email directly,
view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: <Dominic-DallOsto/zotero-reading-list/issues/31/2245198947@github.com>


--------------cFEslnYlfcWuIkzrgwB5ecOc--

gitlzzz commented 1 month ago

disable "Automatically change the read status of items when opening an attached file" didn't help; However, disabling "automatically label items as 'New' when you add then to Zotero" helps, it didn't crash. I think maybe when I make some changes in pdf, some other plugin did something to act like I added a new item?

The Zotero is 7.0.0-beta.105+c5c613c82 (64-bit). The plugins I enabled are here: image

Dominic-DallOsto commented 1 month ago

That's really interesting, hmmm. Thanks for finding that out!

Does this issue still happen if you disable all other addons except zotero-reading-list?

If not, can you find by elimination which combination of plugins causes this issue then I can try to investigate more?

On 24/07/2024 10:25 am, gitlzzz wrote:

disable "Automatically change the read status of items when opening an attached file" didn't help; However, disabling "automatically label items as 'New' when you add then to Zotero" helps, it didn't crash. I think maybe when I make some changes in pdf, some other plugin did something to act like I added a new item?

The Zotero is 7.0.0-beta.105+c5c613c82 (64-bit). The plugins I enabled are here: image.png (view on web) https://github.com/user-attachments/assets/fda540eb-998e-4eff-871e-9a19d3d4b070

— Reply to this email directly, view it on GitHub https://github.com/Dominic-DallOsto/zotero-reading-list/issues/31#issuecomment-2247208498, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGM5S3BNC4XE4ELI3TF23MTZN5QH7AVCNFSM6AAAAABJJREF6OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBXGIYDQNBZHA. You are receiving this because you were mentioned.Message ID: @.***>

--------------BhREZQfpj1yirpKxEd00lSLB Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 8bit

<!DOCTYPE html>

That's really interesting, hmmm. Thanks for finding that out!

Does this issue still happen if you disable all other addons except zotero-reading-list?

If not, can you find by elimination which combination of plugins causes this issue then I can try to investigate more?


On 24/07/2024 10:25 am, gitlzzz wrote:

disable "Automatically change the read status of items when opening an attached file" didn't help;
However, disabling "automatically label items as 'New' when you add then to Zotero" helps, it didn't crash. I think maybe when I make some changes in pdf, some other plugin did something to act like I added a new item?

The Zotero is 7.0.0-beta.105+c5c613c82 (64-bit).
The plugins I enabled are here:
image.png (view on web)


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: <Dominic-DallOsto/zotero-reading-list/issues/31/2247208498@github.com>


--------------BhREZQfpj1yirpKxEd00lSLB--

gitlzzz commented 1 month ago

It seems the situation is a little complicated...

Only zotero-reading-list, no problem. only others, no problem.

The combination of the following five plugins will reproduce the problem. (pdf figure, ethereal reference, translate for zotero, Tara and zotero-reading-list.) image

It seems that both Tara and zotero-reading-list are crucial for the problem. However, when only Tara and zotero-reading-list are enabled, there is no issue.

With all the five plugins turned on, zotero freezes for a long time. If tara is disabled, the problem is less obvious. The less the other three plugins (pdf figure, ethereal reference, translate for zotero) are turned on, the shorter the freeze seems to be.

Probably it is not the problem of zotero-reading-list.

cq01 commented 1 month ago

Problem

I meet same problem while add a new item via zotero connector.

Result

Dominic-DallOsto commented 1 month ago

Thanks a lot for the super detailed information, that seems to help narrow it down a bit.

Now I'm thinking that it could be a problem that multiple extensions are doing things when a new item is added, and somehow they're conflicting.

I'll see if I can find a possible cause for this

On 25/07/2024 10:35 am, cq01 wrote:

Problem

I meet same problem while add a new item via zotero connector.

Result

  • Memory Leak

  • Zotero no response

    Setting

  • Enable Automatically label items as "New• when you add them to Zotero

  • with some other plugins o problem not appear if only enable zotero-reading-list o problem not appear if only enable zotero-reading-list and only one another plugin o problem appear if enable some plugins

    Action

  • Add a new item from Zotero Connector

    Maybe useful information in log

|zotero(1) (+0000002): Error : 'extra' is not a valid field for type 'note' |

Environment Information in logs

|appName => Zotero, version => 7.0.0-beta.105+c5c613c82, os => Windows 10.0 26100, locale => zh-CN, extensions => Wandering File (0.2.8, extension), Easier Citation (0.4.3, extension), Jasminum (1.0.0-16, extension), Zotero 插件市场 (1.6.2, extension), Better BibTex for Zotero (6.7.212, extension), Translate for Zotero (1.1.0-beta.56, extension), Awesome GPT (1.0.5, extension), Ethereal Style (4.6.0, extension), Zotero Reading List (1.3.4, extension), 蒲公英 (1.0.0-9, extension), Actions and Tags for Zotero (1.0.0-beta.44, extension), Better Notes for Zotero (1.1.4-beta.104, extension), OpenH264 Video Codec provided by Cisco Systems, Inc. (1.8.1.2, plugin), Widevine Content Decryption Module provided by Google Inc. (4.10.2710.0, plugin) |

Logs

image.png (view on web) https://github.com/user-attachments/assets/24b5fd40-02ef-4caa-a8b6-c26c17771d23 image.png (view on web) https://github.com/user-attachments/assets/540aaa4f-d5d6-4f2d-82bf-bff6aaac26cd image.png (view on web) https://github.com/user-attachments/assets/95d62075-7dec-4367-99ec-1d874b94484c image.png (view on web) https://github.com/user-attachments/assets/a0b43d1b-c951-4fdf-9cce-9f1e7a1d9dc1 image.png (view on web) https://github.com/user-attachments/assets/cf8afff1-0cd9-4b45-91c2-fac0a64dc306

— Reply to this email directly, view it on GitHub https://github.com/Dominic-DallOsto/zotero-reading-list/issues/31#issuecomment-2249776095, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGM5S3GU7D24PTQ4Q6AZGM3ZOC2FNAVCNFSM6AAAAABJJREF6OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBZG43TMMBZGU. You are receiving this because you were mentioned.Message ID: @.***>

--------------Q3kMQMebKno4EEqgt5sFWH4j Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 8bit

<!DOCTYPE html>

Thanks a lot for the super detailed information, that seems to help narrow it down a bit.

Now I'm thinking that it could be a problem that multiple extensions are doing things when a new item is added, and somehow they're conflicting.

I'll see if I can find a possible cause for this


On 25/07/2024 10:35 am, cq01 wrote:

Problem

I meet same problem while add a new item via zotero connector.

Result

  • Memory Leak
  • Zotero no response

Setting

  • Enable Automatically label items as "New• when you add them to Zotero
  • with some other plugins
    • problem not appear if only enable zotero-reading-list
    • problem not appear if only enable zotero-reading-list and only one another plugin
    • problem appear if enable some plugins

Action

  • Add a new item from Zotero Connector

Maybe useful information in log

zotero(1) (+0000002): Error : 'extra' is not a valid field for type 'note'

Environment Information in logs

appName => Zotero, version => 7.0.0-beta.105+c5c613c82, os => Windows 10.0 26100, locale => zh-CN, 
extensions => Wandering File (0.2.8, extension), 
Easier Citation (0.4.3, extension), 
Jasminum (1.0.0-16, extension), 
Zotero 插件市场 (1.6.2, extension), 
Better BibTex for Zotero (6.7.212, extension), 
Translate for Zotero (1.1.0-beta.56, extension), 
Awesome GPT (1.0.5, extension), 
Ethereal Style (4.6.0, extension), 
Zotero Reading List (1.3.4, extension), 
蒲公英 (1.0.0-9, extension), 
Actions and Tags for Zotero (1.0.0-beta.44, extension), 
Better Notes for Zotero (1.1.4-beta.104, extension), 
OpenH264 Video Codec provided by Cisco Systems, Inc. (1.8.1.2, plugin), 
Widevine Content Decryption Module provided by Google Inc. (4.10.2710.0, plugin)

Logs

image.png (view on web)
image.png (view on web)
image.png (view on web)
image.png (view on web)
image.png (view on web)


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: <Dominic-DallOsto/zotero-reading-list/issues/31/2249776095@github.com>


--------------Q3kMQMebKno4EEqgt5sFWH4j--

Dominic-DallOsto commented 1 month ago

@cq01 @gitlzzz I've fixed the 'extra' is not a valid field for type 'note' error in release 1.3.6. Could you try the same thing with this release and see if you still experience the crashing/freezing?

I also just realised that v1.3.4 broke the auto updating feature, so you'll need to manually update the extension to v1.3.6. Sorry about that!

gitlzzz commented 1 month ago

@cq01 @gitlzzz I've fixed the 'extra' is not a valid field for type 'note' error in release 1.3.6. Could you try the same thing with this release and see if you still experience the crashing/freezing?

I also just realised that v1.3.4 broke the auto updating feature, so you'll need to manually update the extension to v1.3.6. Sorry about that!

@Dominic-DallOsto @cq01 Hi, the 1.3.6 sloved the problem for me! Thanks!

Dominic-DallOsto commented 1 month ago

@cq01 @gitlzzz I've fixed the 'extra' is not a valid field for type 'note' error in release 1.3.6. Could you try the same thing with this release and see if you still experience the crashing/freezing? I also just realised that v1.3.4 broke the auto updating feature, so you'll need to manually update the extension to v1.3.6. Sorry about that!

@Dominic-DallOsto @cq01 Hi, the 1.3.6 sloved the problem for me! Thanks!

Good to hear! Thanks a lot for your detailed investigation

cq01 commented 4 weeks ago

The problem don't appear on v1.3.6! Thanks!

Dominic-DallOsto commented 4 weeks ago

Thanks a lot for your help as well! I think this issue is fixed then.

To summarise, I think what was happening is that the callback that was registered to automatically set a newly added item's read status to New was crashing when running on things that weren't normal Zotero items (like notes and attachments), which was negatively interacting with other plugins that also had callbacks registered for new items and causing Zotero itself to slowdown or crash.