mirror of
https://github.com/Icinga/icingaweb2-module-businessprocess.git
synced 2026-02-02 16:39:25 -05:00
Declare parameters that are null by default as nullable
Implicitly nullable parameters are deprecated since PHP-8.4
This commit is contained in:
Bastian Lederer
parent
b4b41f7c6b
commit
615d0afd10
16 changed files with 23 additions and 23 deletions
|
|
@ -185,7 +185,7 @@ class ProcessCommand extends Command
|
|||
}
|
||||
}
|
||||
|
||||
protected function renderProblemTree($tree, $useColors = false, $depth = 0, BpNode $parent = null)
|
||||
protected function renderProblemTree($tree, $useColors = false, $depth = 0, ?BpNode $parent = null)
|
||||
{
|
||||
$output = '';
|
||||
|
||||
|
|
|
|||
|
|
@ -264,7 +264,7 @@ class ProcessController extends Controller
|
|||
$bp->applySimulation($simulation);
|
||||
}
|
||||
|
||||
protected function loadActionForm(BpConfig $bp, Node $node = null)
|
||||
protected function loadActionForm(BpConfig $bp, ?Node $node = null)
|
||||
{
|
||||
$action = $this->params->get('action');
|
||||
$form = null;
|
||||
|
|
@ -720,7 +720,7 @@ class ProcessController extends Controller
|
|||
return $tabs;
|
||||
}
|
||||
|
||||
protected function handleFormatRequest(BpConfig $bp, BpNode $node = null)
|
||||
protected function handleFormatRequest(BpConfig $bp, ?BpNode $node = null)
|
||||
{
|
||||
$desiredContentType = $this->getRequest()->getHeader('Accept');
|
||||
if ($desiredContentType === 'application/json') {
|
||||
|
|
|
|||
|
|
@ -73,7 +73,7 @@ class AddNodeForm extends CompatForm
|
|||
*
|
||||
* @return $this
|
||||
*/
|
||||
public function setParentNode(BpNode $node = null): self
|
||||
public function setParentNode(?BpNode $node = null): self
|
||||
{
|
||||
$this->parent = $node;
|
||||
|
||||
|
|
|
|||
|
|
@ -84,7 +84,7 @@ class DeleteNodeForm extends BpConfigBaseForm
|
|||
* @param BpNode|null $node
|
||||
* @return $this
|
||||
*/
|
||||
public function setParentNode(BpNode $node = null)
|
||||
public function setParentNode(?BpNode $node = null)
|
||||
{
|
||||
$this->parentNode = $node;
|
||||
return $this;
|
||||
|
|
|
|||
|
|
@ -76,7 +76,7 @@ class EditNodeForm extends CompatForm
|
|||
*
|
||||
* @return $this
|
||||
*/
|
||||
public function setParentNode(BpNode $node = null): self
|
||||
public function setParentNode(?BpNode $node = null): self
|
||||
{
|
||||
$this->parent = $node;
|
||||
|
||||
|
|
|
|||
|
|
@ -110,7 +110,7 @@ class MoveNodeForm extends BpConfigBaseForm
|
|||
* @param BpNode|null $node
|
||||
* @return $this
|
||||
*/
|
||||
public function setParentNode(BpNode $node = null)
|
||||
public function setParentNode(?BpNode $node = null)
|
||||
{
|
||||
$this->parentNode = $node;
|
||||
return $this;
|
||||
|
|
|
|||
|
|
@ -112,7 +112,7 @@ class Metadata
|
|||
return Auth::getInstance();
|
||||
}
|
||||
|
||||
public function canModify(Auth $auth = null)
|
||||
public function canModify(?Auth $auth = null)
|
||||
{
|
||||
if ($auth === null) {
|
||||
if (Icinga::app()->isCli()) {
|
||||
|
|
@ -128,7 +128,7 @@ class Metadata
|
|||
);
|
||||
}
|
||||
|
||||
public function canRead(Auth $auth = null)
|
||||
public function canRead(?Auth $auth = null)
|
||||
{
|
||||
if ($auth === null) {
|
||||
if (Icinga::app()->isCli()) {
|
||||
|
|
|
|||
|
|
@ -69,12 +69,12 @@ class ProcessChanges
|
|||
}
|
||||
|
||||
/**
|
||||
* @param Node $node
|
||||
* @param ?Node $node
|
||||
* @param $properties
|
||||
*
|
||||
* @return $this
|
||||
*/
|
||||
public function addChildrenToNode($children, Node $node = null)
|
||||
public function addChildrenToNode($children, ?Node $node = null)
|
||||
{
|
||||
$action = new NodeAddChildrenAction($node);
|
||||
$action->setChildren($children);
|
||||
|
|
@ -88,7 +88,7 @@ class ProcessChanges
|
|||
*
|
||||
* @return $this
|
||||
*/
|
||||
public function createNode($nodeName, $properties, Node $parent = null)
|
||||
public function createNode($nodeName, $properties, ?Node $parent = null)
|
||||
{
|
||||
$action = new NodeCreateAction($nodeName);
|
||||
$action->setProperties($properties);
|
||||
|
|
|
|||
|
|
@ -7,7 +7,7 @@ use Icinga\Module\Businessprocess\Monitoring\Backend\Ido\Query\HostStatusQuery;
|
|||
|
||||
class HostStatus extends \Icinga\Module\Monitoring\DataView\Hoststatus
|
||||
{
|
||||
public function __construct(ConnectionInterface $connection, array $columns = null)
|
||||
public function __construct(ConnectionInterface $connection, ?array $columns = null)
|
||||
{
|
||||
parent::__construct($connection, $columns);
|
||||
|
||||
|
|
|
|||
|
|
@ -7,7 +7,7 @@ use Icinga\Module\Businessprocess\Monitoring\Backend\Ido\Query\ServiceStatusQuer
|
|||
|
||||
class ServiceStatus extends \Icinga\Module\Monitoring\DataView\Servicestatus
|
||||
{
|
||||
public function __construct(ConnectionInterface $connection, array $columns = null)
|
||||
public function __construct(ConnectionInterface $connection, ?array $columns = null)
|
||||
{
|
||||
parent::__construct($connection, $columns);
|
||||
|
||||
|
|
|
|||
|
|
@ -536,12 +536,12 @@ abstract class Node
|
|||
/**
|
||||
* Export the node to array
|
||||
*
|
||||
* @param array $parent The node's parent. Used to construct the path to the node
|
||||
* @param ?array $parent The node's parent. Used to construct the path to the node
|
||||
* @param bool $flat If false, children will be added to the array key children, else the array will be flat
|
||||
*
|
||||
* @return array
|
||||
*/
|
||||
public function toArray(array $parent = null, $flat = false)
|
||||
public function toArray(?array $parent = null, $flat = false)
|
||||
{
|
||||
$data = [
|
||||
'name' => $this->getAlias(),
|
||||
|
|
|
|||
|
|
@ -47,7 +47,7 @@ abstract class Renderer extends HtmlDocument
|
|||
* @param BpConfig $config
|
||||
* @param BpNode|null $parent
|
||||
*/
|
||||
public function __construct(BpConfig $config, BpNode $parent = null)
|
||||
public function __construct(BpConfig $config, ?BpNode $parent = null)
|
||||
{
|
||||
$this->config = $config;
|
||||
$this->parent = $parent;
|
||||
|
|
|
|||
|
|
@ -112,11 +112,11 @@ class TreeRenderer extends Renderer
|
|||
|
||||
/**
|
||||
* @param Node $node
|
||||
* @param array $path
|
||||
* @param BpNode $parent
|
||||
* @param ?array $path
|
||||
* @param ?BpNode $parent
|
||||
* @return BaseHtmlElement[]
|
||||
*/
|
||||
public function getNodeIcons(Node $node, array $path = null, BpNode $parent = null)
|
||||
public function getNodeIcons(Node $node, ?array $path = null, ?BpNode $parent = null)
|
||||
{
|
||||
$icons = [];
|
||||
if (empty($path) && $node instanceof BpNode) {
|
||||
|
|
|
|||
|
|
@ -8,7 +8,7 @@ use Icinga\Exception\ProgrammingError;
|
|||
|
||||
class FormLoader
|
||||
{
|
||||
public static function load($name, Module $module = null)
|
||||
public static function load($name, ?Module $module = null)
|
||||
{
|
||||
if ($module === null) {
|
||||
$basedir = Icinga::app()->getApplicationDir('forms');
|
||||
|
|
|
|||
|
|
@ -140,7 +140,7 @@ abstract class QuickBaseForm extends Zend_Form implements ValidHtml
|
|||
return $this;
|
||||
}
|
||||
|
||||
protected function loadForm($name, Module $module = null)
|
||||
protected function loadForm($name, ?Module $module = null)
|
||||
{
|
||||
if ($module === null) {
|
||||
$module = $this->icingaModule;
|
||||
|
|
|
|||
|
|
@ -325,7 +325,7 @@ abstract class QuickForm extends QuickBaseForm
|
|||
return $this;
|
||||
}
|
||||
|
||||
public function handleRequest(Request $request = null)
|
||||
public function handleRequest(?Request $request = null)
|
||||
{
|
||||
if ($request === null) {
|
||||
$request = $this->getRequest();
|
||||
|
|
|
|||
Loading…
Reference in a new issue