EDIT

this problem appears only when the newly created in-reference is not of the type journal.

Katja Luther wrote:

Katja Luther wrote:

this problem appears only when the newly created in-reference is not of the type journal.

I think the problem is that the selection dialog for the in reference of an article shows only journals and when the newly created reference is not of this type it does not show up.

Could you please check, if this is right

But do we really allow to enter some other ref type if article is the section reference? We shouldn't I think. Definetely by default Journal should be preselected. But maybe changing ref type should not be possible at all in this case.
Also I do not really think that Walter did choose something else. But maybe, if General is stil the default, in this case he might have forgotten to change the type.
I added Walter to cc so he can comment on this.

I got the following NPE when playing around with the dialog:

last remote method : http://test.e-taxonomy.eu:80/cdmserver/rem_conf_am/remoting/reference.service
last remote request client time : 2020-11-27T17:22:52.882
last remote request response header time : Fri, 27 Nov 2020 17:22:52 GMT
client error time : 2020-11-27T17:22:54.278
login : admin
editor version : 5.18.0.202011270848
server : test.e-taxonomy.eu (edit-test) / rem_conf_am
schema version : 5.18.5.0.20201103
os : Windows Server 2012 R2 6.3 amd64
java : 1.8.0_121
java.lang.NullPointerException
    at eu.etaxonomy.taxeditor.ui.dialog.selection.ReferenceSelectionDialog.getNewEntityWizard(ReferenceSelectionDialog.java:258)
    at eu.etaxonomy.taxeditor.ui.dialog.selection.AbstractFilteredCdmResourceSelectionDialog$2.widgetSelected(AbstractFilteredCdmResourceSelectionDialog.java:443)
    at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:249)
    at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
    at org.eclipse.swt.widgets.Display.sendEvent(Display.java:4418)
    at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1079)
    at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4236)
    at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3824)
    at org.eclipse.jface.window.Window.runEventLoop(Window.java:818)
    at org.eclipse.jface.window.Window.open(Window.java:794)
    at eu.etaxonomy.taxeditor.ui.dialog.selection.AbstractFilteredCdmResourceSelectionDialog.getUuidAndTitleCacheSelectionFromDialog(AbstractFilteredCdmResourceSelectionDialog.java:152)
    at eu.etaxonomy.taxeditor.ui.dialog.selection.AbstractFilteredCdmResourceSelectionDialog.getSelectionFromDialog(AbstractFilteredCdmResourceSelectionDialog.java:133)
    at eu.etaxonomy.taxeditor.ui.dialog.selection.NomenclaturalReferenceSelectionDialog.select(NomenclaturalReferenceSelectionDialog.java:127)
    at eu.etaxonomy.taxeditor.ui.dialog.selection.SelectionDialogFactory.getSelectionFromDialog(SelectionDialogFactory.java:98)
    at eu.etaxonomy.taxeditor.ui.selection.EntitySelectionElement.widgetSelected(EntitySelectionElement.java:239)
    at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:249)
    at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
    at org.eclipse.swt.widgets.Display.sendEvent(Display.java:4418)
    at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1079)
    at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4236)
    at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3824)
    at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$4.run(PartRenderingEngine.java:1121)
    at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:336)
    at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.run(PartRenderingEngine.java:1022)
    at org.eclipse.e4.ui.internal.workbench.E4Workbench.createAndRunUI(E4Workbench.java:150)
    at org.eclipse.ui.internal.Workbench$5.run(Workbench.java:693)
    at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:336)
    at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:610)
    at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:148)
    at eu.etaxonomy.taxeditor.Application.start(Application.java:20)
    at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
    at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:134)
    at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104)
    at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:388)
    at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:243)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:498)
    at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:673)
    at org.eclipse.equinox.launcher.Main.basicRun(Main.java:610)
    at org.eclipse.equinox.launcher.Main.run(Main.java:1519)

Andreas Müller wrote:

For "Section" the type is "InProceedings" now which is generally false. We should use "General" as before or even better force the user to explicitly choose a type as discussed recently in other context.

Also Proceedings have "General" but should have "Print Series" accoring to UI interface.

And for WebPage I would again take WebPage as default. Usually this is a specific page which is part of a more general WebPage. It also could be database but this is up to the user to decide. Not so often used anyway.

The type of the in-reference depends on ReferenceType.inReferenceContraints, we should add the constraints mentioned above.

Andreas Müller wrote:

I got the following NPE when playing around with the dialog:

This should be fixed now.

Katja Luther wrote:

The type of the in-reference depends on ReferenceType.inReferenceContraints, we should add the constraints mentioned above.

AM:

Hallo, diese Methode ist von Andreas K. angelegt worden, daher müsste er da auch nochmal drauf schauen. Kritisch finde ich das handling von Book in Journal. Mit diesen extremen Ausnahmen handelt man sich eine Reihe Probleme ein. Ich denke da macht es mehr Sinn, diese Ausnahme irgendwie anders zu behandelt (z.B. doch als Article mit entsprechender Annotation oder als Book in PrintSeries) oder so, aber das müsste man mit den Taxonomen nochmal besprechen und durchdenken, was das für Probleme bereitet in Eingabe und Ausgabe UIs sowie beim Formattieren. Außer anscheinend Henning hat meines Wissens noch niemand eine solche Andorderung gehabt.
Auch sollte man bei >1 Rückgabetypen drüber nachdenken, ob das nicht zumindeste eine Liste sein sollte, um eine gewisse Wertung zu haben. Falls wir bei Journal als möglichkeit bleiben, wäre es ja extrem wichtig, PrintSeries aber als den absoluten Standard festzulegen.
Kannst du das ins Ticket rüberziehen? Hier geht es schnell verloren.

AK:

ich kenne ähnliche Fälle auch aus der Zoologie (90er Jahre) allerdings weiß ich nicht welchem Typ diese Preprint-Auszüge aus dem Journal in diesen Fällen hatten. Ob man diese Bücher als Article handhaben kann hängt natürtlich davon ab, welche Metadaten gebraucht werden, Article ist da sicherlich nicht ausreichend.

AM:
Das stimmt wahrscheinlich. Im exakten Sinne sind sie aber auch nicht in einem Journal veröffentlicht. Dann passt evtl. besser die PrintSeries, welcher ja ein loser Zusammenhang ist.
Aber das sollten wir unbedingt die User nochmal fragen, insbesondere auch Leute wie Norbert, der sich ja mit sowas sehr gut auskennt.
... also vielleicht müssen wir auch wirklich Journal für diesen Typ zusätzlich zulassen, wenn das wirklich öfters vorkommt.

AK:
Ich denke dass man die Definition von Journal weiter fassen muss. Ein Journal kann in seltenen Fällen auch ein preprint Excerpt als Büchlein herausgeben. Wobei mit Büchlein eher ein eingeständiges kleines Heftchen, als ein dicker Schinken, gemeint ist. Vielleicht liegt auch hierin die Unterscheidung Büchlein vs Buch?

ReferenceType.inReferenceContraints() is a list now, so the first element in the list is the default preferred type. Additionally it should be used in the TaxEditor for filling the combobox, but don't forget to always also add "Generic".