Bug 794349 (JBEPP-1393) - Application Registry has double entries for some applications
Summary: Application Registry has double entries for some applications
Keywords:
Status: CLOSED NEXTRELEASE
Alias: JBEPP-1393
Product: JBoss Enterprise Portal Platform 5
Classification: JBoss
Component: unspecified
Version: 5.2.0.CR01
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 5.2.1.GA
Assignee: Matt Wringe
QA Contact:
URL: http://jira.jboss.org/jira/browse/JBE...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-11-23 18:53 UTC by Matt Wringe
Modified: 2012-04-04 20:51 UTC (History)
3 users (show)

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.
Clone Of:
Environment:
Last Closed: 2012-01-23 20:38:12 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
double_currency_gadget.png (123.22 KB, image/png)
2011-11-23 18:53 UTC, Matt Wringe
no flags Details
double_app_reg.png (130.50 KB, image/png)
2011-11-23 18:53 UTC, Matt Wringe
no flags Details
appreg.patch (1.53 KB, text/x-patch)
2011-11-23 20:24 UTC, Matt Wringe
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 793667 0 high CLOSED Bad names for some administration portlets in app.registry after import 2021-02-22 00:41:40 UTC
Red Hat Issue Tracker JBEPP-1393 0 None Closed Application Registry has double entries for some applications 2012-04-19 20:22:26 UTC

Internal Links: 793667

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.


Note You need to log in before you can comment on or make changes to this bug.