[prev in list] [next in list] [prev in thread] [next in thread] 

List:       spacewalk-devel
Subject:    [Spacewalk-devel] [PATCH] - 1072100 - Duplicated packages are displayed on the event's history page 
From:       Marcelo Moreira de Mello <mmello () redhat ! com>
Date:       2014-03-03 22:35:54
Message-ID: 531503CA.8050208 () redhat ! com
[Download RAW message or body]

Hello spacewalkers,

 This patch addresses the issue report on the bugzilla #1072100

 Thank you.
mmello
-- 
Marcelo Moreira de Mello
RHCA RHCSS RHCVA
Senior Technical Support Engineer/GSS

gpg id: 2048R/FDB110E5
gpg fingerprint: 3BE7 EF71 4DD7 6812 D309  8F18 BD42 D095 FDB1 10E5

["0001-1072100-Fix-duplicated-package-list-when-listing-the.patch" (text/x-patch)]

From: Marcelo Moreira de Mello <mmello@redhat.com>
Date: Mon, 3 Mar 2014 17:33:22 -0500
Subject: [PATCH] 1072100 - Fix duplicated package list when listing the
 event's history

---
 web/modules/rhn/RHN/DB/Server.pm | 31 ++++++++++---------------------
 1 file changed, 10 insertions(+), 21 deletions(-)

diff --git a/web/modules/rhn/RHN/DB/Server.pm b/web/modules/rhn/RHN/DB/Server.pm
index 7cbed1e..89cc7ef 100644
--- a/web/modules/rhn/RHN/DB/Server.pm
+++ b/web/modules/rhn/RHN/DB/Server.pm
@@ -574,27 +574,16 @@ sub server_event_package_action {
 
   my $dbh = RHN::DB->connect;
   my $sth = $dbh->prepare(<<EOS);
-SELECT PN.name ||
-       CASE
-         WHEN PE.id IS NULL THEN ''
-         ELSE '-' || evr_t_as_vre_simple(PE.evr) END ||
-       CASE
-         WHEN AP.package_arch_id is NULL THEN ''
-         ELSE '.' || ARCH.label END
-       AS NVREA,
-       AP.id AS ACTION_PACKAGE_ID,
-       PN.id || '|' || PE.id AS id_combo,
-       AP.package_arch_id
-  FROM rhnPackageName PN,
-       rhnActionPackage AP
-  LEFT OUTER JOIN
-       rhnPackageEVR PE
-    ON AP.evr_id = PE.id
-  LEFT OUTER JOIN
-       rhnPackageArch ARCH
-    ON ARCH.id = AP.package_arch_id
- WHERE AP.action_id = :aid
-   AND AP.name_id = PN.id
+SELECT * FROM (
+  SELECT  DISTINCT PN.id || '|' || PE.id AS ID, PN.name || '-' || evr_t_as_vre_simple(PE.evr) AS NVRE
+     FROM  rhnPackageName PN,
+           rhnPackageEVR PE,
+           rhnActionPackage AP
+       WHERE  AP.action_id = :aid
+         AND  PN.id = AP.name_id
+         AND  PE.id = AP.evr_id
+  ) X
+ORDER BY  UPPER(X.NVRE);
 EOS
   $sth->execute_h(aid => $aid);
   while (my $row = $sth->fetchrow_hashref_copy) {
-- 
1.8.5.3



_______________________________________________
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel

[prev in list] [next in list] [prev in thread] [next in thread]