I am going to have to change the view, rhnPaidOrgs to account for the fact that you no longer get 1 update slot for free. The issue is that we don't want orgs that are not paid, to show up as paid if we have to give them a year free of update. I was going to have to do some dirty hackery in the view to make it work. Once the view is right, rhn_bel.is_org_paid should be right. However, the java code doesn't seem to use this procedure, and rewrote the logic of the view into the code in com/redhat/rhn/domain/org/OrgImpl.java. Yes Yes, I know that you all hate using stored procedures, but it came back and bit you :) . So once I fix the view, you can either incorporate whatever I come up with into OrgImpl.java, or switch it to use rhn_bel.is_org_paid. I can't seem to think of a better way to determine if an org is paid except to special case the scenario where they have 1 update slot, it matches the SKU we gave for free and was granted by RHN on whatever day we run the script in production. Any other better ideas? -- -- James Slagle --
I wrote a new view rhnDemoOrgs that will read from a log table that I'll be populating during the removal script. rhnPaidOrgs will now make use of rhnDemoOrgs. The change sql is at: eng/schema/changes/dev/410/sql/rhnPaidOrgs.sql eng/schema/changes/dev/410/sql/rhnDemoOrgs.sql That java code needs to be updated accordingly.
Changed to use the stored proc in subversion. This code is not used in the Java code anywhere yet.
Reassign QA Responsibility to Partha Aji
Test Plan?
Aligning this against 411 as it is untestable in 410
I normally don't ask, but may I have a test plan please sir?
This is not a "testable" bug, it is one requiring code inspection. I have verified that we now call the stored procedure.
closing -- current release. RHN414 phase 14 gazillion is now in production.