Project

General

Profile

Actions

bug #10060

open

Exception when adding kind-of-unit information to a field unit

Added by Andreas Müller over 1 year ago. Updated over 1 year ago.

Status:
In Progress
Priority:
New
Assignee:
Category:
taxeditor
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Severity:
normal
Found in Version:
Tags:

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

Related to EDIT - feature request #6721: Show kindOfUnit, Sex and Lifestage also for DerivedUnits in DUFacadeClosedKatja Luther

Actions
Related to EDIT - bug #6725: Bug in DU facade for setting / retrieving kind of unitClosedAndreas Müller

Actions
Actions #1

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).

Actions #2

Updated by Katja Luther over 1 year ago

  • Related to feature request #6721: Show kindOfUnit, Sex and Lifestage also for DerivedUnits in DUFacade added
Actions #3

Updated by Katja Luther over 1 year ago

  • Related to bug #6725: Bug in DU facade for setting / retrieving kind of unit added
Actions #4

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.

Actions #5

Updated by Andreas Müller over 1 year ago

  • Target version changed from Release 5.32 to Release 5.33
Actions #6

Updated by Andreas Müller over 1 year ago

  • Target version changed from Release 5.33 to Release 5.44
Actions

Also available in: Atom PDF