Earlier we did not have different state-balls for critical/down-handled nodes. They were assigned critical/down state-balls. Now, we are introducing new state-balls for these nodes.
Earlier in NodeRemoveAction the states were recalculated for the BpNodes. Hence, the state for parent nodes with unconfirmed deleted monitored nodes (host/service) is set to Unknown. This results in incorrect tile rendering for the parent nodes. Therefore, any call to methods which recalculate the states are removed.
fix#279
The node state is recursively calculated in case the node contains only BpNodes as its children and are empty. The gray tiles and state-balls are used for the entire branch if the children are empty.
In BpNode::recalculateState() the BpNode needs to be recursively checked if its children is empty using BpNode::isEmpty() instead of BpNode::hasChildren().
We need to check recurively whether the BP Nodes contatin only child BpNodes which are empty and set the entire tree to state NODE_EMPTY state in this case.
If the imported node is empty or was deleted at a later time, then in NodeTile::render() $link should not be rendered. Also, NodeTile::addDetailsAction() should not add a `href` to this tile.
Before the child empty nodes where set to missing nodes by BpNode::isMissing().
This has been corrected in this fix by checking if whether a child node is an empty node or not.
An empty node state called `NODE_EMPTY` has been introduced with a value of 128 in class `Node`, and this state has been mapped to value *0* in structure `$stateToSortStateMap`.
Before, the empty nodes were to be considered as missing nodes. Correspondingly, the counter for missing in BpNode::getStateSummary() was increased if an empty node is encountered.
Now, the empty nodes are not assumed as missing nodes and the counter for missing is not increased for this case.
If the imported node does not exist, no arguments is passed to BpConfig::addError() by ImportedNode::createFailedNode(). Added 'if' condition in BpConfig::addError() in case no arguments are passed to vsprintf.
refs #234
Before, we called printf twice because BpConfig::addError()
already makes use of printf. The exception only occured for
nodes that have a percent sign (%) in their name.
fixes#234
It is possible to import process nodes from other processes. But there
are no way, via the GUI, to remove the imported process nodes from the
current process/process node.
This commit makes it possible to remove imported process nodes, no matter
where which process they comes from.
Before, we used setParams() instead of with() to populate the parameters
which is especially problematic with imported nodes where the config
paramter got overriden for every subsequent node.
fixes#228
* Fix drill down of the tile/tree action icon links
* Set base target only on parent containers
This also fixes the dashboard integration where we only override the
base target of the div.tiles containers.
fixes#243
