10 Commits

Author	SHA1	Message	Date
Julian Brost	0ebcd2662d	No longer allow overriding the frozen attribute of containers ... The Array, Dictionary, and Namespace types provide a Freeze() method that makes them read-only. So far, there was the possibility to call some methods with `overrideFrozen=true` which would then bypass the corresponding check and allow modification of the data structures nonetheless. With 24b57f0d3a222835178e88489eabd595755ed883, this possibility was already removed from the Namespace type. However, for interface compatibility, it kept the parameter and just ignores it, throwing an exception on any modification on a frozen instance. The only place using `overrideFrozen` was processing of the `-D`/`--define` command line flag that allows setting additional variables in the DSL. At the time it is evaluated, there are no user-created data structures yet that could be frozen, so the only frozen objects that could be encountered are Namespaces (Icinga doesn't freeze other types by itself) and for these, `overrideFrozen` already has no effect. Hence, there is no harm in removing `overrideFrozen` altogether. This simplifies the code and also means that frozen objects are now indeed read-only without exceptions, allowing further optimizations regarding locking in the future.	2025-07-08 14:16:20 +02:00
Alexander A. Klimov	97cd05db7a	Notification#BeginExecuteNotification(): on recovery clear last_notified_state_per_user	2023-12-13 13:21:22 +01:00
Alexander A. Klimov	44e9c6f40d	Notification#BeginExecuteNotification(): discard likely duplicate problem notifications	2023-12-13 13:21:19 +01:00
Michael Friedrich	9c5f280a85	Tests: Add Notification state/type to string cases	2019-07-16 13:46:22 +02:00
Michael Friedrich	d14a88235d	Replace Copyright header with a short version, part I ... CLion -> replace in path	2019-02-25 14:48:22 +01:00
Michael Friedrich	dab53448bc	icinga.com: Update *.{h,c}pp	2018-10-18 09:27:04 +02:00
Michael Insel	158ae2188e	Change copyright header for 2018	2018-01-02 12:08:55 +01:00
Michael	2d036e6886	Fixed missing include statement in unit test ... This adds the missing iostream include to the notification unit test. refs #5613	2017-09-26 22:01:07 +02:00
Michael Friedrich	b7caf0820d	Ensure that *.icinga.com is used everywhere ... fixes #13897 fixes #13277	2017-01-10 17:19:12 +01:00
Michael Friedrich	f8e6d2d603	Add unit test for notification state/type filters ... fixes #13225	2016-11-17 14:36:55 +01:00