Bug 794349 (JBEPP-1393)

Summary: Application Registry has double entries for some applications
Product: [JBoss] JBoss Enterprise Portal Platform 5 Reporter: Matt Wringe <mwringe>
Component: unspecifiedAssignee: Matt Wringe <mwringe>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: high Docs Contact:
Priority: high    
Version: 5.2.0.CR01CC: epp-bugs, jmorgan, mwringe
Target Milestone: ---   
Target Release: 5.2.1.GA   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/JBEPP-1393
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
An issue with the 'Import Applications' request of the Application Registry caused some applications already added to a category to be added again. The fix implements logic changes to the 'Import Applications' request that check for already existing applications in a category.
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-01-23 20:38:12 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
double_currency_gadget.png
none
double_app_reg.png
none
appreg.patch none

Description Matt Wringe 2011-11-23 18:53:02 UTC
project_key: JBEPP

The application registry, after hitting import all applications, will have some applications listed twice. Note that this does not appear to happen in GateIn trunk.

Comment 1 Matt Wringe 2011-11-23 18:53:37 UTC
screenshots showing the Application Registry, and the currency gadget appearing twice.

Comment 2 Matt Wringe 2011-11-23 18:53:37 UTC
Attachment: Added: double_currency_gadget.png
Attachment: Added: double_app_reg.png


Comment 3 Matt Wringe 2011-11-23 19:02:38 UTC
actually, the currency converter may be because there is one there by default and I may have added another one via the dashboard. But the issue with the Application Registry is still a problem.

Comment 4 Matt Wringe 2011-11-23 20:24:05 UTC
patch to fix issue

Comment 5 Matt Wringe 2011-11-23 20:24:05 UTC
Attachment: Added: appreg.patch


Comment 6 Matt Wringe 2011-11-24 16:15:43 UTC
Link: Added: This issue is related to JBEPP-744


Comment 7 Thomas Heute 2012-01-13 13:14:35 UTC
Labels: Added: EPP_5_2_1_Candidate


Comment 8 Thomas Heute 2012-01-13 13:16:05 UTC
Labels: Removed: EPP_5_2_1_Candidate 


Comment 9 Matt Wringe 2012-01-23 20:38:12 UTC
Fixed in commit 8306

Comment 10 Jared MORGAN 2012-03-27 04:13:21 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
== IS THIS INCLUDED IN EPP 5.2.1, AND DOES IT REQUIRE RN? ==

Comment 13 Matt Wringe 2012-04-03 20:41:50 UTC
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1 +1,7 @@
-== IS THIS INCLUDED IN EPP 5.2.1, AND DOES IT REQUIRE RN? ==+Cause: We were not checking if an application was already added to a category before preforming a 'import all applications' request
+
+Consequence: Some applications would appear twice in the application registry.
+
+Fix: check if an application already exists in a category before when preforming 'import all applications'
+
+Result: We no longer have duplicate entries in the application registry

Comment 14 Jared MORGAN 2012-04-04 00:28:49 UTC
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1,7 +1 @@
-Cause: We were not checking if an application was already added to a category before preforming a 'import all applications' request
+An issue with the Import all Applications request of the Application Registry caused some applications already added to a category to appear twice once an import was performed. The fix implements logic changes to the Import All Applications request that checks the category before performing the Import all Applications request, which ensures no duplicates are imported.-
-Consequence: Some applications would appear twice in the application registry.
-
-Fix: check if an application already exists in a category before when preforming 'import all applications'
-
-Result: We no longer have duplicate entries in the application registry

Comment 15 Matt Wringe 2012-04-04 14:06:14 UTC
Updated the release notes a bit, it should be 'Import Applications' not 'Import All Applications' and I didn't understand what exactly the original RN was trying to say. I think its more clear now what the change is, but maybe a bit awkwardly written.

Comment 16 Matt Wringe 2012-04-04 14:06:14 UTC
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1 +1 @@
-An issue with the Import all Applications request of the Application Registry caused some applications already added to a category to appear twice once an import was performed. The fix implements logic changes to the Import All Applications request that checks the category before performing the Import all Applications request, which ensures no duplicates are imported.+An issue with the 'Import Applications' request of the Application Registry caused some applications already added to a category to be added again. The fix implements logic changes that does checks while performing the 'Import Applications' request for already existing applications in a category.

Comment 18 Jared MORGAN 2012-04-04 20:51:37 UTC
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1 +1 @@
-An issue with the 'Import Applications' request of the Application Registry caused some applications already added to a category to be added again. The fix implements logic changes that does checks while performing the 'Import Applications' request for already existing applications in a category.+An issue with the 'Import Applications' request of the Application Registry caused some applications already added to a category to be added again. The fix implements logic changes to the 'Import Applications' request that check for already existing applications in a category.