CDR Tickets

Issue Number 4531
Summary Remove duplicate Insertion tags
Created 2018-09-27 12:19:24
Issue Type Bug
Submitted By Osei-Poku, William (NIH/NCI) [C]
Assigned To Englisch, Volker (NIH/NCI) [C]
Status Closed
Resolved 2019-08-05 13:40:56
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.233689
Description

When an Insertion element is added from the the Elements List, it is duplicated. This appears to happen when no you attempt to add the element without selecting another element or text before selecting the Insertion element.

Comment entered 2018-09-27 13:25:29 by Englisch, Volker (NIH/NCI) [C]

Do you know if this started happening with a recent CDR release? I was under the impression the double-insert of the markup tags when double-clicking was related to the last XMetaL upgrade.

Anyway, after taking a quick look at some of the other inline elements it appears that the same behavior is showing up not only for the Insertion/Deletion elements. Elements like GeneName, ForeignWord, or Emphasis are acting the same.

Comment entered 2018-10-04 10:49:41 by Osei-Poku, William (NIH/NCI) [C]

This appears to be a known issue for quite a long time.

Comment entered 2018-10-12 15:50:39 by Englisch, Volker (NIH/NCI) [C]
  • Finding the bug: 5 hours

  • Fixing the bug: 5 seconds

  • Getting it done before the weekend: Priceless! :-)

I believe I have identified and fixed the problem that caused inline elements to be inserted twice. There was a bug in the macro On_Application_ElementList_Insert. This is a macro that is used for the ProcessingStatus of the GlossaryTermConcept and GlossaryTermName documents. Therefore please make sure to take a look at these two document types as well!

As a side benefit, I was able to "link" the Bold and Italic buttons at the top of the menus to our Strong and Emphasis elements and users are now able to toggle italics and bold using these buttons (see attached image).

The changes are ready for review on DEV.

Comment entered 2018-10-15 13:39:50 by Englisch, Volker (NIH/NCI) [C]

The following files have been updated in github (branch cdr4531-duptags):

Comment entered 2019-06-20 11:49:41 by Englisch, Volker (NIH/NCI) [C]

The modified files have been merged into Joule:
https://github.com/NCIOCPL/cdr-client/commit/9ba87934

Comment entered 2019-06-27 12:34:05 by Osei-Poku, William (NIH/NCI) [C]

Could you please re-install the changes? It appears they have been deleted.

Comment entered 2019-06-27 13:21:38 by Englisch, Volker (NIH/NCI) [C]

Same response as my comment for OCECDR-4527.  Once ticket OCECDR-4600 has been verified I'll restore the version of Joule which then will contain the changes for all three requested macro changes.

Comment entered 2019-07-10 13:02:25 by Osei-Poku, William (NIH/NCI) [C]

OCECDR-4600  has been verified. Please reinstall this change for testing.  Thanks!

Comment entered 2019-07-10 14:10:41 by Englisch, Volker (NIH/NCI) [C]

The current Joule version of the macro has been installed on DEV.

(... and the RefreshManifest has been run, too.)

Comment entered 2019-07-10 15:47:48 by Osei-Poku, William (NIH/NCI) [C]

Verified on DEV. Thanks!

Comment entered 2019-08-05 12:23:00 by Osei-Poku, William (NIH/NCI) [C]

It doesn't look like this has been installed on QA. I am getting duplicate insertion tags.

Comment entered 2019-08-05 12:39:46 by Englisch, Volker (NIH/NCI) [C]

I'll take a look to see what's going on.

Comment entered 2019-08-05 12:49:10 by Englisch, Volker (NIH/NCI) [C]

, could you please tell me the steps you're following to create the duplicate insertion tags?
I tried on QA and I'm getting single insertion tags. I've tried both, to highlight text and then create Insertion tags from the CDR Styles menu as well as double-click from the Element List and I was doing the same without highlighting text first but creating empty Insertion tags.

Comment entered 2019-08-05 13:04:41 by Osei-Poku, William (NIH/NCI) [C]

I am getting the duplicate tags when double-clicking on the Insertion tags from the element list. I have logged out and logged back in and still getting the duplicate tags. Using the Insertion tags from the CDR Styles does not duplicate the tags.

Comment entered 2019-08-05 13:08:14 by Englisch, Volker (NIH/NCI) [C]

Could you try to connect to DEV (or PROD) and back to QA? I'm wondering if your client files haven't been updated since the update on QA.

I will do the same on my machine.

Comment entered 2019-08-05 13:16:55 by Englisch, Volker (NIH/NCI) [C]

I switched to STAGE, opened a summary, place the cursor inside a Para element (without highlighting text), and double-click the Insertion element from the Element List. I confirmed the Insertion tag was inserted twice.
I then repeated the same process on QA and the Insertion tag was inserted correctly (once).
We should ask another user to try to see which behavior is the common one.

Comment entered 2019-08-05 13:20:39 by Osei-Poku, William (NIH/NCI) [C]

Looks like it works in Summaries but not in Glossary Terms after logging into PROD and back into QA.

Comment entered 2019-08-05 13:22:10 by Osei-Poku, William (NIH/NCI) [C]

It worked for me in summaries but not in Glossary Terms.

Comment entered 2019-08-05 13:38:03 by Englisch, Volker (NIH/NCI) [C]

Ahh! Good to know there is an easy explanation.
I had assumed this was only a Summary issue and did not make any changes affecting Glossaries. Is doesn't say in your description that this problem was limited to Summaries but that's what we had looked at while testing on DEV. Obviously, the Glossaries hadn't been tested on DEV or we would have seen this issue earlier. The Insertion/Deletion tags are still duplicated on DEV at this point.
I suggest to enter a new ticket for Kepler to address this issue for Glossaries as a next step.

Comment entered 2019-08-05 13:40:41 by Osei-Poku, William (NIH/NCI) [C]

Okay. Sounds good. Thanks!

Comment entered 2019-08-05 13:41:12 by Osei-Poku, William (NIH/NCI) [C]

Verified on QA. Thanks!

Comment entered 2019-09-09 09:43:08 by Osei-Poku, William (NIH/NCI) [C]

Verified on PROD. Thanks!

Attachments
File Name Posted User
BoldItalic.png 2018-10-12 15:49:19 Englisch, Volker (NIH/NCI) [C]

Elapsed: 0:00:00.001495