glpi
glpi copied to clipboard
Add AdditionalTargets For SoftwareLicense Alert
| Q | A |
|---|---|
| Bug fix? | no |
| New feature? | yes |
| BC breaks? | no |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | #number |
Add AdditionalTargets For SoftwareLicense Alert
I understand. Like certificates, recipients of the alert will receive all expired certificates and not just those for which they are responsible?
My point is that this code will not work. Indeed, the notification is created by NotificationEvent::raiseEvent('alert', new self(), $options), meaning that the $target_object of the NotificationTarget will be an empty SoftwareLicense item. The id of the tech/group in charge is fetched from this $target_object, so this id will always be unset/0.
This PR seems stale, I close it.