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.

Sign up for GitHub

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

Jump to bottom

Strip off stray \r when reading lines via Zotero.read() #584

Open
wants to merge 1 commit into
base: 4.0
Choose a base branch
from
Open

Strip off stray \r when reading lines via Zotero.read() #584

wants to merge 1 commit into from

Conversation

aurimasv
Copy link
Contributor

Ran into this problem with RIS and APA PsychNET translator, but I worked around it for now.

@simonster
Copy link
Member

Do we have to strip any \r, or just \r at the end? Not that I can think of a good reason \r in the middle of a line.

@aurimasv
Copy link
Contributor Author

Fairly sure this can only happen at the end of the string (otherwise, \r would be an end-of-line character). But I can't really find why this would be happening at all on MXR, tbh. Might be looking in the wrong place

@dstillman dstillman closed this Mar 3, 2021
@dstillman dstillman deleted the branch zotero:4.0 March 3, 2021 08:51
@dstillman dstillman reopened this Mar 3, 2021
@dstillman dstillman deleted the branch zotero:4.0 April 5, 2023 07:34
@dstillman dstillman closed this Apr 5, 2023
@dstillman dstillman reopened this Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aurimasv @simonster @dstillman