Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 617548 - Product Model has artifacts of earlier models
Product Model has artifacts of earlier models
Status: RELEASE_PENDING
Product: Candlepin
Classification: Community
Component: candlepin (Show other bugs)
0.5
All Solaris
low Severity medium
: ---
: ---
Assigned To: Bryan Kearney
Katello QA List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2010-07-23 08:11 EDT by Bryan Kearney
Modified: 2015-05-14 11:22 EDT (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed:
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
possible patch from Devan (4.72 KB, patch)
2010-07-23 12:43 EDT, Bryan Kearney
no flags Details | Diff

  None (edit)
Description Bryan Kearney 2010-07-23 08:11:03 EDT
The product model has a join table at cp_product_cp_product_attribute AND a FK from cp_product_attribute to cp_product. Per Devan, the join table is correct. Attached is a possible, but untested, patch.
Comment 1 Devan Goodwin 2010-07-23 08:39:41 EDT
Patch doesn't seem to work, the tables come out right but the join table no longer gets populated, hopefully someone can spot my mistake.

Also tried with no join table, just a mapped by, this time the join table disappeared from the schema, but again the join column on cp_product_attributes doesn't get populated...
Comment 2 Bryan Kearney 2010-07-23 12:43:32 EDT
Created attachment 434012 [details]
possible patch from Devan
Comment 3 Bryan Kearney 2010-07-23 12:47:36 EDT
Devan:

Is the goal a 1-n product -> attribute or m-n product -> attribute?
Comment 4 Devan Goodwin 2010-08-03 07:55:26 EDT
1-n
Comment 5 Devan Goodwin 2010-08-04 10:57:12 EDT
Fixed for both product and pool attributes. Now uses a join column instead of a table as well. Should see that column in cp_product_attributes and cp_pool_attributes.

57e91b3682424fd17d6989996156c9141991d4d3
Comment 6 wes hayutin 2010-08-10 11:09:32 EDT
dev verified
Comment 7 John Sefler 2011-05-04 10:41:10 EDT
Group move of VERIFIED Candlepin component bugs to RELEASE_PENDING

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