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

Add names of save actions #318

Closed
koppor opened this issue Oct 14, 2020 · 14 comments · Fixed by #322
Closed

Add names of save actions #318

koppor opened this issue Oct 14, 2020 · 14 comments · Fixed by #322
Labels
needs-addition Help texts need to be expanded

Comments

@koppor
Copy link
Member

koppor commented Oct 14, 2020

A user reported at JabRef/jabref#7012 (comment)

11:54:05.303 [JavaFX Application Thread] WARN org.jabref.logic.citationkeypattern.BracketedPattern - Key generator warning: unknown modifier 'remove_braces_formatter'.

I think, the documentation page for the formatters is https://github.com/JabRef/user-documentation/blob/master/en/finding-sorting-and-cleaning-entries/saveactions.md. There, the intenral name of the formatter (getKey(), https://github.com/JabRef/jabref/blob/e955c46850e4738830198a889c3f09870b88ab30/src/main/java/org/jabref/logic/formatter/bibtexfields/RemoveBracesFormatter.java#L16) is not listed. I think, it should be listed as the user can include it at https://docs.jabref.org/setup/citationkeypatterns#modifiers (can't he?).

Should there also be a link from https://docs.jabref.org/setup/citationkeypatterns#modifiers to the save actions?

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

k3KAW8Pnf7mkmdSMPHz27 commented Oct 14, 2020

Should there also be a link from https://docs.jabref.org/setup/citationkeypatterns#modifiers to the save actions?

Yes, there should be a link. I did not know "save actions" were a thing 😛

I think, it should be listed as the user can include it at https://docs.jabref.org/setup/citationkeypatterns#modifiers (can't he?).

They can all be accessed as "modifiers", yes. But the user, most likely, should not. The "special field markers" sometimes remove latex, sometimes remove spaces, and sometimes do neither (I am speculating that they were intended as citation keys, and the use of the "bracketed expressions" has expanded?). I don't think they can be used with consistent results, but perhaps I am being overly paternalistic on this subject.

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

Oups, and I missed the point. Yes, I'd agree that the getKey() should be listed.

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

@Siedlerchr
Copy link
Member

I think it makes sense to have them all in one place and just link from the sorting and cleaning to them

@mlep
Copy link
Contributor

mlep commented Oct 19, 2020

It looks like the good page (the one displayed at help.jabref.org is this one: https://github.com/JabRef/user-documentation/blob/master/en/finding-sorting-and-cleaning-entries/saveactions.md

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

It looks like the good page (the one displayed at help.jabref.org is this one: https://github.com/JabRef/user-documentation/blob/master/en/finding-sorting-and-cleaning-entries/saveactions.md

That does seem to be the case. I don't really know what is going. I'll link to this one for now.

@koppor
Copy link
Member Author

koppor commented Oct 19, 2020

There should be only one file. The other one can be deleted. Please add a redirect from the deleted page to the new page into https://github.com/JabRef/user-documentation/blob/master/.gitbook.yml

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

I am taking a look at this

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

Among others, I cannot use RemoveNewlinesFormatter for abstracts, should that be addressed?

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

Note, if it cannot be accessed from Formatters#getAll() (which it cannot), it also cannot be used as a modifier.

@Siedlerchr
Copy link
Member

It's an internal formatter only. It should then not be documented.

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

So the documentation for the ones that cannot (currently) be used should be removed? (e.g., https://github.com/JabRef/user-documentation/blob/master/en/finding-sorting-and-cleaning-entries/saveactions.md#remove-line-breaks)

@koppor
Copy link
Member Author

koppor commented Nov 3, 2020

@k3KAW8Pnf7mkmdSMPHz27 Yes, please remove the ones not available to the user. Currently, the description does not contain any examples, thus it is no loss to just delete the text.

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

Ok. I'll not add "Replace Unicode ligature" which is the default for "All-text-fields", but cannot be added manually. If it gets added to Formatters#getAll it can be included.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-addition Help texts need to be expanded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants