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

Import from JabFox Causes New Instance of JabRef (Linux,FF) #5766

Closed
PorcelainMouse opened this issue Dec 18, 2019 · 6 comments
Closed

Import from JabFox Causes New Instance of JabRef (Linux,FF) #5766

PorcelainMouse opened this issue Dec 18, 2019 · 6 comments

Comments

@PorcelainMouse
Copy link

JabRef version 5
jabref-5.0.30016-1.x86_64 (master)

The issue is that importing from JabFox causes a new instance of JabRef to start running, same as if JabRef isn't running already. However, JabFox reports that it has established communication to JabRef. So, not sure why it is doing that.

Steps to reproduce the behavior:

  1. Start Jab Ref
  2. navigate to importable page
  3. cick JabFox import icon

There is no error, per se, only the behavior is wrong; I assume this is not the intended behavior, but I don't know because I haven't got it working yet. It is certainly atypical behavior, for sure. When the second instance starts, it complains about backup files for all the open jabref databases, which is predictable.

I opened a bug on JabFox for this, but I the team over there thought it could be a JabRef bug because JabFox says that it can communicate with JabRef. They also said they would look into it further, though.

JabRef/JabRef-Browser-Extension#111

(BTW, love the new v5. Super excited. Thanks! Looks a lot better on my system than v4, for starters.)

@bernhard-kleine
Copy link

bernhard-kleine commented Jan 2, 2020

Under "Options->Advanced" you should add 6050 to the port and check "listen for remote operation on port".

@bernhard-kleine
Copy link

bernhard-kleine commented Jan 2, 2020

@tobiasdiez
In the Details part of the Jabfox add-on there is written:

Download jabref.json, JabRef.bat and JabRef.ps1, and copy them to the same directory as JabRef.exe

However the .bat file is burrowed in C:\Program Files\JabRef5\runtime\bin and JabRef.ps1 does not exist in the 5.0 version. Therefore for me JabFox does not work under win7.
Using Zotero (stand alone) and export form Zotero and import into Jabref still works.

@PorcelainMouse
Copy link
Author

@bernhard-kleine, Oh, thanks! Yes, that makes sense. However, JabFox said it was communicating with JabRef before. It still says it can communicate, now that I have checked that box for port 6050.

But, now, JabFox doesn't work! Before, JabFox would be visible on import-able pages. Now, it's grey-ed out on journal article pages, but not on other regular web-pages. For example, it's available on this page for this github issue, but not on this DOI:10.1088/0143-0807/28/3/E01 . So confusing!

@PorcelainMouse
Copy link
Author

Ignoring the problem with JabFox not working on the pages I need, I tested another page, and it does not cause a new instance of JabRef. I seems pretty clear that was the problem. Thanks @bernhard-kleine .

@bernhard-kleine
Copy link

I cannot import via JabFox at all. When using Pubmed (on firefox) the jabfox icon is greyed and does not work. Trying to import DOI 10.1002/cne.20260 from the website fails because the import into jabref is never finished. Please reopen the issue!

@PorcelainMouse
Copy link
Author

@bernhard-kleine doyou have the latest JabRef 5 (beta, from the development branch)? I think there is a new package just about every day. I tried your DOI and, for me JabFox doesn't let me try to import it, it is also grayed out. So, it sounds like you are getting different behavior than me.

Also, your suggestion fixed this bug for me, and it sounds like you aren't getting new instances started, either. So, I think this issue is solved.

Are you following this issue? JabRef/JabRef-Browser-Extension#90

That is my main issue, now, and I think that is one of your problems, too.

It also sounds like you have another problem with JabRef imports failing. I would open a new issue for that. When JabFox does work, import succeeds for me, at least most of the time. So, I think JabRef will want to know about your issue. It's probably something specific about the reference you want to import. I got an error importing from a random web-site, but it wasn't really a journal article, so I figured I would let it go, since other imports worked fine.

Also, I had weird problems before I installed the extra .json file, correctly. Do you know what I'm talking about?

koppor pushed a commit that referenced this issue Jan 1, 2022
f78c707 Update Turabian (full note) for 17.1.3.2 Reprint Editions (#5809)
2169ddc Create art-libraries-society-of-north-america-arlisna-reviews.csl (#5791)
94119a6 Create university-of-south-wales-harvard.csl (#5787)
470925c Update fachhochschule-sudwestfalen.csl (#5811)
1a9eff7 Create annals-of-public-and-cooperative-economics.csl (#5747)
d7477f2 Merge pull request #5368 from POBrien333/patch-862
113e683 Merge pull request #5812 from citation-style-language/apa-no-initials
62dacff Create universitat-oberta-de-catalunya-apa.csl
046482c Fix Organization Studies title
1213b95 Make oranization-studies dependent
c9641f4 Create apa-no-initials.csl
0b4e047 Modify american-journal-of-archaeology.csl (#5719)
ddeda4f Create journal-of-economic-impact.csl (#5707)
40c2696 Update gallia.csl (#5782)
5d5927a Create serbian-archives-of-medicine.csl (#5721)
2777954 Merge pull request #5807 from dhacker29/master
77e3fca Update Society of Biblical Literature Full Note for 6.1.6
4479a5a haaga-helia-university-of-applied-sciences-harvard.csl: no date cites need to be unique (#5726)
d00e046 Create university-of-roehampton-harvard.csl (#5732)
e76da40 Medicinski Razgledi -- fix locale
75b1554 Add original-date extra to proper place as per SBL Handbook 6.2.17-18 (#5725)
d43676e Update harvard-bournemouth-university.csl (#5667)
0312a39 Create ABNT NBR 6023:2018. UNESP – Faculdade de Engenharia de Guarati… (#5705)
6585ff9 Create sinergie-italian-journal-of-management.csl (#5776)
a7dc678 Create journal-of-global-health.csl (#5775)
02080ce Merge pull request #5800 from citation-style-language/validation-fixes
8b51d85 change s. d. to s.d. in french style for INBO reports (issue #2) (#5634)
1b4ec09 Create bloomsbury-academic.csl (#5741)
1f28d87 Create zeithistorische-forschungen.csl (#5766)
bb11942 Remove superfluous 'container' data from wiley-vch-books
06038a7 Remove et-al affixes
be96bd8 Remove et-al affixes from bibliotecae-it.csl
03a94e4 Remove more et-al affixes
07d7423 Revert non-et-al changes to universitat-basel-iberomanistik
f8e5fd8 Create medicinski-razgledi.csl (#5789)
b4df8ee Create dut-harvard.csl (#5762)
40855c4 Fix affixes on et-al
b4fe9b0 Merge pull request #5798 from citation-style-language/uris
3dc4aaf URL-encode documentation URLs
408a3a8 fixes citation-style-language/Sheldon#38
42aca5b Update biophysics-and-physicobiology.csl (#5751)
706bea0 Create universidade-do-porto-faculdade-de-engenharia-chicago-pt.csl (#5742)
269ff91 Update universidade-do-porto-faculdade-de-engenharia-chicago.csl (#5797)
0ba51e7 Update el-profesional-de-la-informacion.csl (#5785)
dac3baf Create arthropod-systematics-and-phylogeny.csl (#5772)
9dff241 Merge pull request #5796 from dhacker29/patch-1
35949f4 Update turabian-fullnote-bibliography-no-ibid.csl
ccfb661 Merge pull request #5793 from benjaminmoon/patch-2
520e127 Update frontiers-medical-journals.csl (#5740)
ab629ab Update natura-croatica.csl (#5795)
60b2b17 Rename turabian-fullnote-bibliography-no-ibid to turabian-fullnote-bibliography-no-ibid.csl
ad4edff Create turabian-fullnote-bibliography-no-ibid
d9b956c Update monographs-of-the-palaeontographical-society.csl
3648ae6 Merge pull request #5792 from StianOby/fix-no-page-number-articles
e15b2f7 Update norsk-henvisningsstandard-for-rettsvitenskapelige-tekster.csl
d8778cf Removed "page" (s. ) term for articles that do not have page numbers.
409017c add medium field for eBooks to MLA 8th

git-subtree-dir: buildres/csl/csl-styles
git-subtree-split: f78c707
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

2 participants