From c3ab507841a2f99fc2629adcf63e543cbf935068 Mon Sep 17 00:00:00 2001 From: FusionPBX Date: Fri, 30 Dec 2016 10:54:33 -0700 Subject: [PATCH] Update xml_cdr.php Capitalize the SQL, change tmp_cdr to an alias of c for CDR, and remove the spaces from the indentation. --- app/xml_cdr/resources/classes/xml_cdr.php | 435 +++++++++++----------- 1 file changed, 217 insertions(+), 218 deletions(-) diff --git a/app/xml_cdr/resources/classes/xml_cdr.php b/app/xml_cdr/resources/classes/xml_cdr.php index 0816ca6e79..49b99fc456 100644 --- a/app/xml_cdr/resources/classes/xml_cdr.php +++ b/app/xml_cdr/resources/classes/xml_cdr.php @@ -650,252 +650,251 @@ if (!class_exists('xml_cdr')) { $sql_date_range .= " and start_stamp between '".$this->start_stamp_begin.":00.000' and '".$this->start_stamp_end.":59.999' \n"; } else { - if (strlen($this->start_stamp_begin) > 0) { $sql_date_range .= "and start_stamp >= '".$this->start_stamp_begin.":00.000' \n"; } - if (strlen($this->start_stamp_end) > 0) { $sql_date_range .= "and start_stamp <= '".$this->start_stamp_end.":59.999' \n"; } + if (strlen($this->start_stamp_begin) > 0) { $sql_date_range .= "AND start_stamp >= '".$this->start_stamp_begin.":00.000' \n"; } + if (strlen($this->start_stamp_end) > 0) { $sql_date_range .= "AND start_stamp <= '".$this->start_stamp_end.":59.999' \n"; } } } else { switch ($this->quick_select) { - case 1: $sql_date_range .= "and start_stamp >= '".date('Y-m-d H:i:s.000', strtotime("-1 week"))."' \n"; break; //last 7 days - case 2: $sql_date_range .= "and start_stamp >= '".date('Y-m-d H:i:s.000', strtotime("-1 hour"))."' \n"; break; //last hour - case 3: $sql_date_range .= "and start_stamp >= '".date('Y-m-d')." "."00:00:00.000' \n"; break; //today - case 4: $sql_date_range .= "and start_stamp between '".date('Y-m-d',strtotime("-1 day"))." "."00:00:00.000' and '".date('Y-m-d',strtotime("-1 day"))." "."23:59:59.999' \n"; break; //yesterday - case 5: $sql_date_range .= "and start_stamp >= '".date('Y-m-d',strtotime("this week"))." "."00:00:00.000' \n"; break; //this week - case 6: $sql_date_range .= "and start_stamp >= '".date('Y-m-')."01 "."00:00:00.000' \n"; break; //this month - case 7: $sql_date_range .= "and start_stamp >= '".date('Y-')."01-01 "."00:00:00.000' \n"; break; //this year + case 1: $sql_date_range .= "AND start_stamp >= '".date('Y-m-d H:i:s.000', strtotime("-1 week"))."' \n"; break; //last 7 days + case 2: $sql_date_range .= "AND start_stamp >= '".date('Y-m-d H:i:s.000', strtotime("-1 hour"))."' \n"; break; //last hour + case 3: $sql_date_range .= "AND start_stamp >= '".date('Y-m-d')." "."00:00:00.000' \n"; break; //today + case 4: $sql_date_range .= "AND start_stamp between '".date('Y-m-d',strtotime("-1 day"))." "."00:00:00.000' and '".date('Y-m-d',strtotime("-1 day"))." "."23:59:59.999' \n"; break; //yesterday + case 5: $sql_date_range .= "AND start_stamp >= '".date('Y-m-d',strtotime("this week"))." "."00:00:00.000' \n"; break; //this week + case 6: $sql_date_range .= "AND start_stamp >= '".date('Y-m-')."01 "."00:00:00.000' \n"; break; //this month + case 7: $sql_date_range .= "AND start_stamp >= '".date('Y-')."01-01 "."00:00:00.000' \n"; break; //this year } } //calculate the summary data $sql = "SELECT \n"; $sql .= "e.domain_uuid, \n"; - $sql .= "d.domain_name, \n"; - $sql .= "e.extension, \n"; - $sql .= "e.number_alias, \n"; - $sql .= "count(*) \n"; - $sql .= "filter( \n"; - $sql .= " where tmp_cdr.domain_uuid = e.domain_uuid \n"; - $sql .= " and ((\n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or ( \n"; - $sql .= " e.number_alias is not null and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - $sql .= " and (\n"; - $sql .= " tmp_cdr.answer_stamp is not null \n"; - $sql .= " and \n"; - $sql .= " tmp_cdr.bridge_uuid is not null) \n"; + $sql .= "COUNT(*) \n"; + $sql .= "FILTER( \n"; + $sql .= " WHERE c.domain_uuid = e.domain_uuid \n"; + $sql .= " AND ((\n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR ( \n"; + $sql .= " e.number_alias IS NOT NULL and ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + $sql .= " AND (\n"; + $sql .= " c.answer_stamp IS NOT NULL \n"; + $sql .= " and \n"; + $sql .= " c.bridge_uuid IS NOT NULL) \n"; + if ($this->include_internal) { - $sql .= " and (direction = 'inbound' or direction = 'local')) \n"; + $sql .= " AND (direction = 'inbound' OR direction = 'local')) \n"; } else { - $sql .= "and direction = 'inbound') \n"; + $sql .= "AND direction = 'inbound') \n"; } - $sql .= "as answered, \n"; + $sql .= "AS answered, \n"; - $sql .= "count(*) \n"; - $sql .= "filter( \n"; - $sql .= " where (( \n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or (\n"; - $sql .= " e.number_alias is not null \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.answer_stamp is null \n"; - $sql .= " and \n"; - $sql .= " tmp_cdr.bridge_uuid is null) \n"; - if ($this->include_internal) { - $sql .= " and (direction = 'inbound' or direction = 'outbound'))"; - } else { - $sql .= " and direction = 'inbound')"; - } - $sql .= "as missed, \n"; - - $sql .= "count(*) \n"; - $sql .= "filter( \n"; - $sql .= " where (( \n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or ( \n"; - $sql .= " e.number_alias is not null \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - $sql .= " and tmp_cdr.hangup_cause = 'NO_ANSWER' \n"; + $sql .= "COUNT(*) \n"; + $sql .= "FILTER( \n"; + $sql .= " WHERE (( \n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR (\n"; + $sql .= " e.number_alias IS NOT NULL \n"; + $sql .= " AND ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + $sql .= " AND ( \n"; + $sql .= " c.answer_stamp IS NULL \n"; + $sql .= " AND \n"; + $sql .= " c.bridge_uuid IS NULL) \n"; + if ($this->include_internal) { + $sql .= " AND (direction = 'inbound' OR direction = 'outbound'))"; + } else { + $sql .= " AND direction = 'inbound')"; + } + $sql .= "AS missed, \n"; + + $sql .= "COUNT(*) \n"; + $sql .= "FILTER( \n"; + $sql .= " WHERE (( \n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR ( \n"; + $sql .= " e.number_alias IS NOT NULL \n"; + $sql .= " AND ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + $sql .= " AND c.hangup_cause = 'NO_ANSWER' \n"; if ($this->include_internal) { - $sql .= " and (direction = 'inbound' or direction = 'local') \n"; + $sql .= " AND (direction = 'inbound' OR direction = 'local') \n"; + } + else { + $sql .= "AND direction = 'inbound' \n"; + } + $sql .= ") AS no_answer, \n"; + + $sql .= "COUNT(*) \n"; + $sql .= "FILTER( \n"; + $sql .= " WHERE (( \n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR ( \n"; + $sql .= " e.number_alias IS NOT NULL \n"; + $sql .= " AND ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + $sql .= " AND \n"; + $sql .= " c.hangup_cause = 'USER_BUSY' \n"; + if ($this->include_internal) { + $sql .= " AND (direction = 'inbound' OR direction = 'local')) \n"; } else { - $sql .= "and direction = 'inbound' \n"; - } - $sql .= ") as no_answer, \n"; + $sql .= " AND direction = 'inbound') \n"; + } + $sql .= "AS busy, \n"; - $sql .= "count(*) \n"; - $sql .= "filter( \n"; - $sql .= " where (( \n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or ( \n"; - $sql .= " e.number_alias is not null \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - $sql .= " and \n"; - $sql .= " tmp_cdr.hangup_cause = 'USER_BUSY' \n"; - if ($this->include_internal) { - $sql .= " and (direction = 'inbound' or direction = 'local')) \n"; - } - else { - $sql .= " and direction = 'inbound') \n"; - } - $sql .= "as busy, \n"; + $sql .= "SUM(c.billsec) \n"; + $sql .= "FILTER ( \n"; + $sql .= " WHERE (( \n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR ( \n"; + $sql .= " e.number_alias IS NOT NULL \n"; + $sql .= " AND ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + if ($this->include_internal) { + $sql .= " AND (direction = 'inbound' OR 'direction = 'outbound') \n"; + } + $sql .= " ) / \n"; + $sql .= "COUNT(*) \n"; + $sql .= "FILTER ( \n"; + $sql .= " WHERE (( \n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR ( \n"; + $sql .= " e.number_alias IS NOT NULL \n"; + $sql .= " AND ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + if ($this->include_internal) { + $sql .= " AND (direction = 'inbound' OR 'direction = 'outbound') \n"; + } + $sql .= " ) AS aloc, \n"; - $sql .= "SUM(tmp_cdr.billsec) \n"; - $sql .= "filter ( \n"; - $sql .= " where (( \n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or ( \n"; - $sql .= " e.number_alias is not null \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - if ($this->include_internal) { - $sql .= " and (direction = 'inbound' or 'direction = 'outbound') \n"; - } - $sql .= " ) / \n"; - $sql .= "count(*) \n"; - $sql .= "filter ( \n"; - $sql .= " where (( \n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or ( \n"; - $sql .= " e.number_alias is not null \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - if ($this->include_internal) { - $sql .= " and (direction = 'inbound' or 'direction = 'outbound') \n"; - } - $sql .= " ) as aloc, \n"; + $sql .= "COUNT(*) \n"; + $sql .= "FILTER ( \n"; + $sql .= " WHERE (( \n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR ( \n"; + $sql .= " e.number_alias IS NOT NULL \n"; + $sql .= " AND ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + if ($this->include_internal) { + $sql .= " AND (direction = 'inbound' OR direction = 'local')) \n"; + } + else { + $sql .= " AND direction = 'inbound') \n"; + } + $sql .= "AS inbound_calls, \n"; - $sql .= "count(*) \n"; - $sql .= "filter ( \n"; - $sql .= " where (( \n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or ( \n"; - $sql .= " e.number_alias is not null \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - if ($this->include_internal) { - $sql .= " and (direction = 'inbound' or direction = 'local')) \n"; - } - else { - $sql .= " and direction = 'inbound') \n"; - } - $sql .= "as inbound_calls, \n"; + $sql .= "SUM(c.billsec) \n"; + $sql .= "FILTER ( \n"; + $sql .= " WHERE (( \n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR ( \n"; + $sql .= " e.number_alias IS NOT NULL \n"; + $sql .= " AND ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + if ($this->include_internal) { + $sql .= " AND (direction = 'inbound' OR direction = 'local')) \n"; + } + else { + $sql .= " AND direction = 'inbound') \n"; + } + $sql .= "AS inbound_duration, \n"; - $sql .= "SUM(tmp_cdr.billsec) \n"; - $sql .= "filter ( \n"; - $sql .= " where (( \n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or ( \n"; - $sql .= " e.number_alias is not null \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - if ($this->include_internal) { - $sql .= " and (direction = 'inbound' or direction = 'local')) \n"; - } - else { - $sql .= " and direction = 'inbound') \n"; - } - $sql .= "as inbound_duration, \n"; + $sql .= "COUNT(*) \n"; + $sql .= "FILTER ( \n"; + $sql .= " WHERE (( \n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR ( \n"; + $sql .= " e.number_alias IS NOT NULL \n"; + $sql .= " AND ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + $sql .= " AND \n"; + $sql .= " c.direction = 'outbound') \n"; + $sql .= "AS outbound_calls, \n"; - $sql .= "count(*) \n"; - $sql .= "filter ( \n"; - $sql .= " where (( \n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or ( \n"; - $sql .= " e.number_alias is not null \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - $sql .= " and \n"; - $sql .= " tmp_cdr.direction = 'outbound') \n"; - $sql .= "as outbound_calls, \n"; + $sql .= "SUM(c.billsec) \n"; + $sql .= "FILTER ( \n"; + $sql .= " WHERE (( \n"; + $sql .= " c.caller_id_number = e.extension \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.extension) \n"; + $sql .= " OR ( \n"; + $sql .= " e.number_alias IS NOT NULL \n"; + $sql .= " AND ( \n"; + $sql .= " c.caller_id_number = e.number_alias \n"; + $sql .= " OR \n"; + $sql .= " c.destination_number = e.number_alias))) \n"; + $sql .= " AND ( \n"; + $sql .= " c.direction = 'outbound')) \n"; + $sql .= "AS outbound_duration, \n"; + + $sql .= "e.description \n"; + + $sql .= "FROM v_extensions AS e, v_domains AS d, \n"; + $sql .= "( SELECT \n"; + $sql .= " domain_uuid, \n"; + $sql .= " caller_id_number, \n"; + $sql .= " destination_number, \n"; + $sql .= " answer_stamp, \n"; + $sql .= " bridge_uuid, \n"; + $sql .= " direction, \n"; + $sql .= " start_stamp, \n"; + $sql .= " hangup_cause, \n"; + $sql .= " billsec \n"; + $sql .= " FROM v_xml_cdr \n"; + $sql .= " WHERE domain_uuid = '".$this->domain_uuid."' \n"; + $sql .= $sql_date_range; + $sql .= ") AS c \n"; + + $sql .= "WHERE \n"; + $sql .= "d.domain_uuid = e.domain_uuid \n"; + if (!($_GET['showall'] && permission_exists('xml_cdr_all'))) { + $sql .= "AND e.domain_uuid = '".$this->domain_uuid."' \n"; + } + $sql .= "GROUP BY e.extension, e.domain_uuid, d.domain_uuid, e.number_alias, e.description \n"; + $sql .= "ORDER BY extension ASC \n"; - $sql .= "SUM(tmp_cdr.billsec) \n"; - $sql .= "filter ( \n"; - $sql .= " where (( \n"; - $sql .= " tmp_cdr.caller_id_number = e.extension \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.extension) \n"; - $sql .= " or ( \n"; - $sql .= " e.number_alias is not null \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.caller_id_number = e.number_alias \n"; - $sql .= " or \n"; - $sql .= " tmp_cdr.destination_number = e.number_alias))) \n"; - $sql .= " and ( \n"; - $sql .= " tmp_cdr.direction = 'outbound')) \n"; - $sql .= "as outbound_duration, \n"; - - $sql .= "e.description \n"; - - $sql .= "from v_extensions as e, v_domains as d, \n"; - $sql .= "( select \n"; - $sql .= " domain_uuid, \n"; - $sql .= " caller_id_number, \n"; - $sql .= " destination_number, \n"; - $sql .= " answer_stamp, \n"; - $sql .= " bridge_uuid, \n"; - $sql .= " direction, \n"; - $sql .= " start_stamp, \n"; - $sql .= " hangup_cause, \n"; - $sql .= " billsec \n"; - $sql .= " from v_xml_cdr \n"; - $sql .= " where domain_uuid = '".$this->domain_uuid."' \n"; - $sql .= $sql_date_range; - $sql .= ") tmp_cdr \n"; - $sql .= "where \n"; - $sql .= "d.domain_uuid = e.domain_uuid \n"; - if (!($_GET['showall'] && permission_exists('xml_cdr_all'))) { - $sql .= "AND e.domain_uuid = '".$this->domain_uuid."' \n"; - } - $sql .= "group by e.extension, e.domain_uuid, d.domain_uuid, e.number_alias, e.description \n"; - $sql .= "order by extension asc \n"; - //echo $sql; $prep_statement = $this->db->prepare(check_sql($sql)); $prep_statement->execute(); $summary = $prep_statement->fetchAll(PDO::FETCH_NAMED);