Bug 556719

Created attachment 385331 [details]
This patch has included the query current partner and make of hwcert.

The return vendor/partner name is reading from cf_partner.value directly. 
For example, the result will be returned:"Sun", but in the listing application UI we may need a exact name: "Sun Microsystems" for showing. 

Question:
Need we add a new table named "vendor_partner_map" to map "Sun" vs "Sun Microsystems"  in the database? or other method?

Thoughts?


Thanks.
Nicho

Created attachment 385621 [details]
Patch improves:

1. Add the logger to log the sql statement.
2. Improve the usage information in comment.
3. Change the return value:partner_info to HASH from Array including make infomation

This patch will probably need to be updated with today's discussion...


We agreed to add two tables, a vendor names table and a pass through map table.

Vendor names {id(seq),publish_name(notnull),cf_partner_id}

o vendor_names.cf_partner is blank or provides the cf_partner_id that goes with
  the vendor publish_name text

o vendor_name_id replaces cf_partner in hwcert_make


Pass-Through map {id(seq),cf_partner(notnull),vendor_name_id(notnull)}

pass-through map controls the list of other vendors that a partner may select for a pass-through name.

Created attachment 386766 [details]
Patch improve:

1. To match the new "partner" issue db change. 
2. Impact:getCertLst & getPartInfo interface.


PS: 
  A) bugs34's table schema has been changed to db schema in the wiki page and add some fake data.
  B) this patch has been deployed in https://bugdev.devel.redhat.com/bugzilla_xisun3/
 
Branch committed revision 217.


Xmlrpc test script result:

[xisun@localhost ~]$ perl test_xmlrpc.pl
$VAR1 = {
          'Cisco' => [],
          'Sun Microsystems, Inc.' => [
                                      'Netra'
                                    ],
          'Dell' => [
                    'PowerEdge'
                  ],
          'IBM' => [
                   'Xseires'
                 ],
          'Intel Corporation' => [
                                 'Server Board'
                               ],
          'Fujitsu' => [
                       'PRIMEQUEST',
                       'PRIMERGY'
                     ],
          'Fujitsu Siemens Computers' => []
        };

This bug is done