Skip to content

Conversation

mspirkov
Copy link
Contributor

Q A
Is bugfix? ✔️
New feature?
Breaks BC?
Fixed issues

private $_registered = false;
/**
* @var string the current working directory
* @var string|null the current working directory
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

private $_errors;
/**
* @var ArrayObject list of validators
* @var ArrayObject|null list of validators
Copy link
Contributor Author

@mspirkov mspirkov Oct 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

private $_value;
/**
* @var string|int the key for the current iteration
* @var string|int|null the key for the current iteration
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

* @param string $sql
* @param array $params
* @return string[]|false
* @return array[]|false
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

$items[$row['name']] = $this->populateItem($row);
/** @var Role|Permission */
$item = $this->populateItem($row);
$items[$row['name']] = $item;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image


foreach ($this->items as $name => $item) {
/** @var Item $item */
/** @var Role|Permission $item */
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

/** @var static */
$result = $view->registerAssetBundle(get_called_class());

return $result;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

@mspirkov mspirkov marked this pull request as ready for review October 13, 2025 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant