Merge branch 'ent-11559-error-al-crear-items-de-tipo-graphs-sql-vertical-horizontal-y-pie-graph-en-metaconsola' into 'develop'
Ent 11559 Error al crear items de tipo "Graphs: SQL vertical, horizontal y pie graph" en metaconsola See merge request artica/pandorafms!6098
This commit is contained in:
commit
e9246e4a1d
|
@ -2947,10 +2947,6 @@ switch ($action) {
|
|||
);
|
||||
if ($values['treport_custom_sql_id'] == 0) {
|
||||
$sql = get_parameter('sql', '');
|
||||
if ($sql !== '') {
|
||||
$good_format = db_validate_sql($sql);
|
||||
}
|
||||
|
||||
$values['external_source'] = $sql;
|
||||
}
|
||||
|
||||
|
@ -2958,6 +2954,19 @@ switch ($action) {
|
|||
'historical_db_check'
|
||||
);
|
||||
$values['top_n_value'] = get_parameter('max_items');
|
||||
|
||||
if ($values['type'] === 'sql_graph_hbar'
|
||||
|| ($values['type'] === 'sql_graph_vbar')
|
||||
|| ($values['type'] === 'sql_graph_pie')
|
||||
) {
|
||||
$values['server_name'] = get_parameter('combo_server_sql');
|
||||
} else {
|
||||
$values['server_name'] = get_parameter('combo_server');
|
||||
}
|
||||
|
||||
if ($sql !== '') {
|
||||
$good_format = db_validate_sql($sql, (is_metaconsole() === true) ? $values['server_name'] : false);
|
||||
}
|
||||
} else if ($values['type'] == 'url') {
|
||||
$values['external_source'] = get_parameter('url');
|
||||
} else if ($values['type'] == 'event_report_group') {
|
||||
|
|
|
@ -2560,12 +2560,20 @@ function db_get_column_type(string $table, string $column='')
|
|||
/**
|
||||
* Validate sql query.
|
||||
*
|
||||
* @param string $sql Query for validate.
|
||||
* @param string $sql Query for validate.
|
||||
* @param mixed $server Server name where sql must connect.
|
||||
*
|
||||
* @return boolean True if query is valid.
|
||||
*/
|
||||
function db_validate_sql(string $sql)
|
||||
function db_validate_sql(string $sql, $server=false)
|
||||
{
|
||||
if ($server !== false && is_metaconsole() === true) {
|
||||
$setup = metaconsole_get_connection($server);
|
||||
if (metaconsole_connect($setup) !== NOERR) {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
try {
|
||||
error_reporting(0);
|
||||
db_process_sql_begin();
|
||||
|
@ -2578,5 +2586,9 @@ function db_validate_sql(string $sql)
|
|||
error_reporting(E_ALL);
|
||||
}
|
||||
|
||||
if ($server !== false && is_metaconsole() === true) {
|
||||
metaconsole_restore_db();
|
||||
}
|
||||
|
||||
return ($result !== false) ? true : false;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue