Attachment 385331 Details for Bug 556719 – This patch has included the query current partner and make of hwcert.

[patch] This patch has included the query current partner and make of hwcert.

c.patch (text/plain), 2.67 KB, created by XINSUN on 2010-01-19 08:13:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: XINSUN

Created: 2010-01-19 08:13:42 UTC

Size: 2.67 KB

patch

obsolete

>Index: webservice.pl
>===================================================================
>--- webservice.pl	(revision 209)
>+++ webservice.pl	(working copy)
>@@ -193,9 +193,83 @@
>         }
>         push(@certlist, \%cert_info);
>     }
>-
>+     
>+    $logger->debug("Ending: hwcert-getCertList");
>+    
>     return \@certlist; 
> }
> 
>+# function to return all partners information
>+# $call = $rpc->call( 'hwcert.getPartnerInfo',
>+#                                          {  vendor => ['hp', 'dell'],
>+#                                             return_field => ["vendor", "make"]
>+#                                           });
>+
>+sub getPartnerInfo {
>+    my ($self, $params) = @_;
>+
>+    $logger->debug("Starting: hwcert-getPartnerInfo");
>+
>+    my %legal_field_map = (
>+        'vendor'        => 'cf_partner.value',
>+        'make'          => 'rh_hwcert_make.name'       
>+    );
>+    #Parse retrun_field for sql_field.
>+    my @return_fields;
>+    if (defined $params->{return_field}) {
>+        @return_fields = ref($params->{return_field}) =~ /ARRAY/ ? @{$params->{return_field}} : ($params->{return_field});
>+    } else {
>+        @return_fields = keys %legal_field_map;
>+    }
>+   
>+    my @inner_fields;
>+    my @valid_return_field;
>+    foreach my $field_name (@return_fields) {
>+        if (defined($legal_field_map{$field_name})) {
>+            push(@valid_return_field, $field_name);
>+            push(@inner_fields, $legal_field_map{$field_name});
>+        }
>+    }
>+
>+    my $sql_fields = join(',', @inner_fields);
>+    my $sql_join_table = "rh_hwcert_make, cf_partner";
>+   
>+    my $sql_cond =  " rh_hwcert_make.cf_partner_id = cf_partner.id 
>+                    AND cf_partner.isactive = 1";
>+
>+    #Parse the parameters for sql_cond; 
>+    my @vendor;
>+    if (defined $params->{vendor}) {
>+        @vendor = ref($params->{vendor}) =~ /ARRAY/ ? @{$params->{vendor}} : ($params->{vendor});
>+    }
>+     
>+    if (scalar(@vendor) > 0) {
>+        my $vendors = '\'';
>+        $vendors .= join( '\', \'', @vendor);
>+        $vendors .= '\'';
>+        $sql_cond .= qq{  AND cf_partner.value in ($vendors) };
>+    }
>+
>+    my $dbh = Bugzilla->dbh;
>+    my $query =  "SELECT $sql_fields FROM $sql_join_table WHERE $sql_cond";
>+    my $sth = $dbh->prepare($query);
>+    $sth->execute();
>+
>+    my @partnerlist;
>+    while ( my @row = $sth->fetchrow_array() ) {
>+        my %partner_info;
>+        foreach my $field_name (@valid_return_field) {
>+           if (scalar(@row) > 0) {
>+               $partner_info{$field_name} = shift(@row);
>+           }
>+        }
>+        push(@partnerlist, \%partner_info);
>+    }
>+
>+    $logger->debug("Ending: hwcert-getPartnerInfo");
>+    return \@partnerlist;  
>+    
>+}
>+
> 1;
> 
>

Actions: View | Diff

Attachments on bug 556719: 385331 | 385621 | 386766