bug #9353
closedCopy & Paste creates dirty data in term trees
Added by Katja Luther over 3 years ago. Updated about 3 years ago.
80%
Description
It was missed to implement copy & paste for the dto implementation of the term tree editors, the menu items are still available but produce termnodes without terms.
Files
| picture135-1.png (18.7 KB) picture135-1.png | Andreas Müller, 01/21/2021 06:31 PM | ||
| picture752-1.png (16.6 KB) picture752-1.png | Andreas Müller, 01/21/2021 06:35 PM |
Related issues
Updated by Andreas Müller over 3 years ago
- Related to feature request #9337: Open issues for reimplementation of termtree editors added
Updated by Katja Luther over 3 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 50
Applied in changeset taxeditor|27ca515332a1068e6f49ccff9a4466bce556a548.
Updated by Katja Luther over 3 years ago
- Assignee changed from Katja Luther to Andreas Müller
Updated by Andreas Müller over 3 years ago
- File picture135-1.png picture135-1.png added
There is a duplicate error message when c%p a duplicate term to a non-duplicate allowing tree.
last remote method : http://test.e-taxonomy.eu:80/cdmserver/rem_conf_am/remoting/termnode.service
last remote request client time : 2021-01-21T18:29:20.452
last remote request response header time : Thu, 21 Jan 2021 18:29:20 GMT
client error time : 2021-01-21T18:29:20.493
login : admin
editor version : 5.19.0.202101211659
server : test.e-taxonomy.eu (edit-test) / rem_conf_am
schema version : 5.18.6.0.20201124
os : Windows Server 2012 R2 6.3 amd64
java : 1.8.0_121
java.lang.Exception: This term tree does not allow duplicate terms.
at eu.etaxonomy.cdm.api.service.TermNodeServiceImpl.addChildNode(TermNodeServiceImpl.java:160)
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.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:333)
at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:190)
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:157)
at org.springframework.transaction.interceptor.TransactionInterceptor$1.proceedWithInvocation(TransactionInterceptor.java:99)
at org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:283)
at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:96)
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:179)
at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:213)
at com.sun.proxy.$Proxy809.addChildNode(Unknown Source)
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.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:333)
at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:190)
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:157)
Only 1 error message should appear
Updated by Andreas Müller over 3 years ago
- File picture752-1.png picture752-1.png added
The error message on duplicate also appears if the tree allows duplicates:
Updated by Andreas Müller over 3 years ago
Andreas Müller wrote:
There is a duplicate error message when c%p a duplicate term to a non-duplicate allowing tree.
...
I could not reproduce this later
Updated by Andreas Müller over 3 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
Paste requires save first. Is this really needed? Most task are now handled without save-requirement so it breaks a bit the look&feel.
Updated by Katja Luther over 3 years ago
- Status changed from Feedback to Resolved
Duplicates can be copied now (for trees allowing duplicates)
The problem of two error dialogs I could not reproduce.
Updated by Katja Luther over 3 years ago
- Assignee changed from Katja Luther to Andreas Müller
Updated by Andreas Müller over 3 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 50 to 80
Andreas Müller wrote:
Paste requires save first. Is this really needed? Most task are now handled without save-requirement so it breaks a bit the look&feel.
This seem not yet being answered. If difficult to implement please move to new ticket or include in #9337.
Ticket can be closed except for this issue.
Updated by Katja Luther about 3 years ago
- Status changed from Feedback to Closed
Updated by Katja Luther about 3 years ago
Andreas Müller wrote:
Andreas Müller wrote:
Paste requires save first. Is this really needed? Most task are now handled without save-requirement so it breaks a bit the look&feel.
This seem not yet being answered. If difficult to implement please move to new ticket or include in #9337.
moved this issue to #9337.
Ticket can be closed except for this issue.