Project

General

Profile

Actions

bug #9353

closed

Copy & Paste creates dirty data in term trees

Added by Katja Luther almost 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Highest
Assignee:
Category:
taxeditor
Target version:
Start date:
Due date:
% Done:

80%

Estimated time:
Severity:
normal
Found in Version:

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

Related to EDIT - feature request #9337: Open issues for reimplementation of termtree editorsClosedKatja Luther

Actions
Actions #1

Updated by Andreas Müller almost 2 years ago

Actions #2

Updated by Katja Luther almost 2 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50
Actions #3

Updated by Katja Luther almost 2 years ago

  • Assignee changed from Katja Luther to Andreas Müller
Actions #4

Updated by Andreas Müller almost 2 years ago

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

Actions #5

Updated by Andreas Müller almost 2 years ago

The error message on duplicate also appears if the tree allows duplicates:

Actions #6

Updated by Andreas Müller almost 2 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

Actions #7

Updated by Andreas Müller almost 2 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.

Actions #8

Updated by Andreas Müller almost 2 years ago

In general the copy&past works now.

Actions #9

Updated by Katja Luther almost 2 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.

Actions #10

Updated by Katja Luther almost 2 years ago

  • Assignee changed from Katja Luther to Andreas Müller
Actions #11

Updated by Andreas Müller almost 2 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.

Actions #12

Updated by Katja Luther almost 2 years ago

  • Status changed from Feedback to Closed
Actions #13

Updated by Katja Luther almost 2 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.

Actions

Also available in: Atom PDF