From ec90e73696fdf6b0ab9c26b2aa9a1f90f6f66281 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= <adrian@suse.de>
Date: Thu, 3 Sep 2015 15:39:50 +0200
Subject: [PATCH] [api] simplify channel_binary search as well

---
 src/api/lib/xpath_engine.rb | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/src/api/lib/xpath_engine.rb b/src/api/lib/xpath_engine.rb
index ca78437c944..643299a1a05 100644
--- a/src/api/lib/xpath_engine.rb
+++ b/src/api/lib/xpath_engine.rb
@@ -158,27 +158,19 @@ def initialize
         '@binaryarch' => {:cpart => 'channel_binaries.binaryarch'},
         '@package' => {:cpart => 'channel_binaries.package'},
         '@project' => {:cpart => 'cprj.name', :joins => [
-          'LEFT join channel_binary_lists cblp on cblp.id=channel_binaries.channel_binary_list_id',
-          'LEFT join channels cp on cp.id=cblp.channel_id',
-          'LEFT join packages cpkg on cpkg.id=cp.package_id',
+          'LEFT join packages cpkg on cpkg.id=channel.package_id',
           'LEFT join projects cprj on cprj.id=cpkg.project_id']},
         '@supportstatus' => {:cpart => 'supportstatus'},
         'target/disabled' => {:cpart => 'ufdct.disabled', :joins => [
-          'LEFT join channel_binary_lists ufdcbl on ufdcbl.id=channel_binaries.channel_binary_list_id',
-          'LEFT join channels ufdc on ufdc.id=ufdcbl.channel_id',
-          'LEFT join channel_targets ufdct on ufdct.channel_id=ufdc.id']},
+          'LEFT join channel_targets ufdct on ufdct.channel_id=channel.id']},
         'updatefor/@project' => {:cpart => 'puprj.name', :joins => [
-          'LEFT join channel_binary_lists ufcbl on ufcbl.id=channel_binaries.channel_binary_list_id',
-          'LEFT join channels ufc on ufc.id=ufcbl.channel_id',
-          'LEFT join channel_targets ufct on ufct.channel_id=ufc.id',
+          'LEFT join channel_targets ufct on ufct.channel_id=channel.id',
           'LEFT join product_update_repositories pur on pur.repository_id=ufct.repository_id',
           'LEFT join products pun on pun.id=pur.product_id ',
           'LEFT join packages pupkg on pupkg.id=pun.package_id ',
           'LEFT join projects puprj on puprj.id=pupkg.project_id ']},
         'updatefor/@product' => {:cpart => 'pupn.name', :joins => [
-          'LEFT join channel_binary_lists ufncbl on ufncbl.id=channel_binaries.channel_binary_list_id',
-          'LEFT join channels ufnc on ufnc.id=ufncbl.channel_id',
-          'LEFT join channel_targets ufnct on ufnct.channel_id=ufnc.id',
+          'LEFT join channel_targets ufnct on ufnct.channel_id=channel.id',
           'LEFT join product_update_repositories pnur on pnur.repository_id=ufnct.repository_id',
           'LEFT join products pupn on pupn.id=pnur.product_id ']},
       },
@@ -374,6 +366,9 @@ def find(xpath)
       relation = Issue.all
     when 'channel_binaries'
       relation = ChannelBinary.all
+      @joins = [ 'LEFT join channel_binary_lists channel_binary_list on channel_binary_list.id=channel_binaries.channel_binary_list_id',
+                 'LEFT join channels channel on channel.id=channel_binary_list.channel_id',
+               ] << @joins
     when 'released_binaries'
       relation = BinaryRelease.all