#3983 [RiskGraph] add: add position to order charts #3985
Conversation
| @@ -639,7 +642,7 @@ public function getDigiriskElementListsByDepth(): array | |||
| ]; | |||
| } | |||
| } | |||
|
|
|||
There was a problem hiding this comment.
Un tab en trop au lieu d'un espace vide, pareil au dessus
| @@ -604,7 +608,7 @@ public function getRisksByCotation(): array | |||
| $array['dataset'] = 1; | |||
| $array['labels'] = $this->cotations; | |||
|
|
|||
| $riskAssessmentList = $riskAssessment->fetchAll('', '', 0, 0, ['customsql' => 'status = 1']); | |||
| $riskAssessmentList = $riskAssessment->fetchAll('', '', 0, 0, ['customsql' => 'status > 0']); | |||
|
|
||
| $array['graphs'] = [$getRisksByCotation, $getRisksByDangerCategoriesAndCriticality, $getRisksByDangerCategories]; | ||
| $array['graphs'] = [$getRisksByDangerCategoriesAndCriticality, $getRisksByCotation , $getRisksByDangerCategories]; |
There was a problem hiding this comment.
Si l'ordre est géré par la position, plus besoin de changer l'ordre dans l'array ?
class/digiriskelement.class.php
Outdated
| } | ||
| } | ||
|
|
||
| $array['data'] = array_count_values($children['all']); |
There was a problem hiding this comment.
Si jamais le "contient GP 6 UT 9" est fait dans une autre issue, penser à le retirer
There was a problem hiding this comment.
L'idée est très intéressante mais il faut revoir la conception car la position comme la visibilité est une donné configurable par l'utilisateur
De plus, on souhaite ici positionner les graphs par rapport au autre donc il s'agit d'une donnée du graph par de son chargement
|
Je vais poser ça ici parce que peut-être que je ne comprends pas bien le code et que je me complique la vie.
Le problème, c'est que quelle que soit la position des graphs, ils seront quand même bloqués par le type de donnée, e.g. même si un graph du type 2 est en position 1 et un graph du type 1 est en position 10, le graph de type 1 sera malgré tout affiché en premier. Du coup, c'est compliqué de laisser le choix à l'utilisateur de l'ordre. Cependant, peut que de juste "flatten" la liste des graphs après les avoir récupérés serait la solution, sinon il faut revoir comment on récupère les graphs. |
No description provided.