bug #10060
openException when adding kind-of-unit information to a field unit
0%
Description
When trying to edit the "kind of unit" field of a field unit in specimen-tree-editor details view the following error occurs.
We need to check if this is only a result of using the facade instead of explicit details view for field units or if kind-of-unit should really not be used for field units.
last remote method : https://api.cybertaxonomy.org:443/salvador/remoting/taxon.service last remote request client time : 2022-05-12T14:28:02.214 last remote request response header time : Thu, 12 May 2022 12:28:02 GMT client error time : 2022-05-12T14:28:29.256 login : admin editor version : 5.31.0 server : api.cybertaxonomy.org (cybertaxonomy.org) / salvador schema version : 5.29.0.0.20211122 os : Windows Server 2012 R2 6.3 amd64 java : 1.8.0_131 java.lang.IllegalStateException: This method is not allowed for this specimen or observation type. Probably you have tried to add specimen(derived unit) information to a field unit at eu.etaxonomy.cdm.api.facade.DerivedUnitFacade.testDerivedUnit(DerivedUnitFacade.java:2535) at eu.etaxonomy.cdm.api.facade.DerivedUnitFacade.setDerivedUnitKindOfUnit(DerivedUnitFacade.java:1845) at eu.etaxonomy.taxeditor.ui.section.occurrence.AbstractSpecimenOrObservationDetailElement.handleEvent(AbstractSpecimenOrObservationDetailElement.java:45) at eu.etaxonomy.taxeditor.ui.section.occurrence.FieldUnitFacadeDetailElement.handleEvent(FieldUnitFacadeDetailElement.java:77) at eu.etaxonomy.taxeditor.ui.section.AbstractCdmDetailElement.propertyChange(AbstractCdmDetailElement.java:209) at eu.etaxonomy.taxeditor.ui.element.AbstractCdmFormElement.firePropertyChangeEvent(AbstractCdmFormElement.java:230) at eu.etaxonomy.taxeditor.ui.combo.term.TermComboElement.widgetSelected(TermComboElement.java:420) 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)
Related issues
Updated by Katja Luther over 1 year ago
- Status changed from New to In Progress
I think we should not allow to set a kind of unit for a field unit, so I would remove the possibility to edit the kind of unit for field units. The kind of unit describes "Part(s), physical state, or class of materials represented by this specimen" (https://wiki.bgbm.org/bps/index.php/ABCD2Concepts).
Updated by Katja Luther over 1 year ago
- Related to feature request #6721: Show kindOfUnit, Sex and Lifestage also for DerivedUnits in DUFacade added
Updated by Katja Luther over 1 year ago
- Related to bug #6725: Bug in DU facade for setting / retrieving kind of unit added
Updated by Katja Luther over 1 year ago
Katja Luther wrote in #note-1:
I think we should not allow to set a kind of unit for a field unit, so I would remove the possibility to edit the kind of unit for field units. The kind of unit describes "Part(s), physical state, or class of materials represented by this specimen" (https://wiki.bgbm.org/bps/index.php/ABCD2Concepts).
Maybe I am wrong... we should discuss this issue again.
Updated by Andreas Müller over 1 year ago
- Target version changed from Release 5.32 to Release 5.33
Updated by Andreas Müller over 1 year ago
- Target version changed from Release 5.33 to Release 5.44