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

Deleted code related to the pdf and ps fields #865

Merged
merged 2 commits into from
Feb 25, 2016

Conversation

oscargus
Copy link
Contributor

Dropping the support for ps and pdf fields (which seems dropped already as hardcoding a pdf-field in a .bib-file doesn't even show up in the entry editor) seemed to remove quite a bit of code.

@oscargus oscargus added cleanup-ops status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers labels Feb 25, 2016
@simonharrer
Copy link
Contributor

I am suprised that so much code is unused. But I do like it that we can throw so much unused code away! :)

👍

lenhard added a commit that referenced this pull request Feb 25, 2016
Deleted code related to the pdf and ps fields
@lenhard lenhard merged commit 6e3b189 into JabRef:master Feb 25, 2016
@ajbelle
Copy link

ajbelle commented Feb 28, 2016

I suspect that in all that code JabRef was providing the multiple root directory access functionality discussed in #496 :-)

@oscargus
Copy link
Contributor Author

Could be like that, yes. Doesn't it work for you anymore?

@ajbelle
Copy link

ajbelle commented Mar 1, 2016

Sorry, so near end of my PhD I don't dare update, for fear of loosing access to thousands of references. My understanding is that the functionality I am using was 'never there' (none on the team understood what I was on about, and hence the lengthy #496). I thought you may have seen something that may have been enabling it, and was forewarning.

@ajbelle
Copy link

ajbelle commented Mar 25, 2016

@oscargus Over Easter I updated to ver3.2 to try fix a Search issue. My install still finds the files I have residing in the old legacy pdf file structure that is not in the same tree as the JabRef files! No symbolic or hardlinks present, so I believe in magic! :-) It must be coded elsewhere. happy Easter.

@oscargus
Copy link
Contributor Author

@ajbelle this PR is not in 3.2 so maybe a bit too early to be fully relieved. ;-) However, I know of some code which currently supports different directories for different file types and I won't remove that. Not sure if that code is enough, but it might in fact be.

@oscargus
Copy link
Contributor Author

This code was primarily related to the UI side, so any buttons magically showing up next to the PDF-field etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup-ops status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants