Fixes throwing an exception when 'id' field is present in bib file

[CaptainDaVinci]

Fixes #4905

---

• Change in CHANGELOG.md described
• Tests created for changes
• Manually tested changed features in running JabRef
• Screenshots added in PR description (for bigger UI changes)
• Ensured that the git commit message is a good one
• Check documentation status (Issue created for outdated help page at help.jabref.org?)

[tobiasdiez]

Thanks for your PR.

Please also change the variable

jabref/src/main/java/org/jabref/model/entry/BibEntry.java

Line 48 in 23e9e52

protected static final String ID_FIELD = "id";

to INTERNAL_ID_FIELD = JabRef-internal-id. Thanks

[CaptainDaVinci]

Should the exception also be removed from the clearField method?

jabref/src/main/java/org/jabref/model/entry/BibEntry.java

Lines 463 to 468 in 23e9e52

	public Optional<FieldChange> clearField(String name, EntryEventSource eventSource) {
	String fieldName = toLowerCase(name);
	if (BibEntry.ID_FIELD.equals(fieldName)) {
	throw new IllegalArgumentException("The field name '" + name + "' is reserved");
	}

[tobiasdiez]

Yes, please also remove it from the clear field method!

[LinusDietz]

LGTM