Documentation update:
- https://www.zabbix.com/documentation/2.2/manual/appendix/macros/supported_by_location?&#overview
- https://www.zabbix.com/documentation/2.2/manual/introduction/whatsnew222?&#daemon_improvements
Not sure if Daemon Improvements was the best place for whatsnew note.
Fixed in development branch svn://svn.zabbix.com/branches/dev/ZBXNEXT-2069
Tested. Looks wonderful.
Released in:
pre-2.2.2rc1 r40967
pre-2.3.0 r40968
(1) [S]
{HOST .PORT}macro should be supported also by trigger name/description
wiper RESOLVED in r41428
asaveljevs Looks good, but please see r41528. Checks for macros where in different order in different places. RESOLVED.
wiper reviewed, CLOSED
(2) [F]
{HOST.PORT} macro should be supported also by trigger name/descriptionoleg.egorov RESOLVED IN r41462
asaveljevs (a) {HOST.PORT}
is expanded in trigger name in "Monitoring" -> "Triggers", but is not expanded in "Monitoring" -> "Events" or "Last 20 issues" in the dashboard.
oleg.egorov RESOLVED IN r41566
asaveljevs CLOSED
asaveljevs (b) I am not sure separating cases into "interfaceWithPriorities" and "interfaceWithPrioritiesPort" performance-wise is worth code complication that it introduces. Maybe just add
{HOST.PORT}to PATTERN_INTERFACE_FUNCTION?
oleg.egorov CLOSED
asaveljevs (c) There is a typo in variable name $paternInterfaceFunction. REOPENED.
oleg.egorov RESOLVED IN r41566
oleg.egorov As discussed, should be renamed from -> to
interfaceWithPriorities -> interfaceWithoutPort
interface -> agentInterface
interfaceWithPrioritiesPort -> interface
asaveljevs It was $paternInterfaceFunction, now it became $paterntInterfaceFunction, which is even worse. 
The correct spelling is $patternInterfaceFunction. Fixed that in r41604. RESOLVED.
oleg.egorov Thanks! CLOSED
asaveljevs Tested
FIXED IN 2.2.2rc1 r41605, 2.3.0 r41606
CLOSED
(3) according to (2),
{HOST.PORT}support was also added in trigger name/description, but there is no changelog entry on that
Documentation also should be updated
wiper updated changelog in pre-2.2.2rc1 r42007 and pre-2.3.0 r42008
wiper marked as supported in overview and added 'supported since' note https://www.zabbix.com/documentation/2.2/manual/appendix/macros/supported_by_location?&#overview
wiper and updated whatsnew https://www.zabbix.com/documentation/2.2/manual/introduction/whatsnew222?&#daemon_improvements
Please review
asaveljevs Fixed typo in changelog entry in r42090.
asaveljevs In https://www.zabbix.com/documentation/2.2/manual/appendix/macros/supported_by_location , I think that "internal and trigger" is a more suitable style for official documentation than "internal/trigger". Simiarly for "names/descriptions".
asaveljevs In "What's new", the change is only listed as a daemon improvement, but it also an improvement in the frontend. REOPENED.
wiper Removed '/' from HOST.PORT macro description. Similar style was already used there, so I though it's acceptable. Regarding frontend documentation - I'm passing that to Oleg.
oleg.egorov RESOLVED
<richlv> now we have footnote "Supported since 2.2.2." and in the description "Supported in trigger names and descriptions since 2.2.2.".
other entries have various combinations of this, too. let's agree on some guidelines and unify this.
martins-v I'd propose the general rule as this - 'supported since' comments go in the Description (as done originally). However, when 'supported since' comments include a long listing (or would have to be repeated in too many cells), it is acceptable to add them in the footnotes and link to them.
martins-v Also, reviewed and removed the 'supported since' duplication in this particular case. And, for 'what's new', reviewed and improved the wording a bit and split it out into 'macro improvements' separately, rather than in 'daemon improvements'. Seeing that developer review of doc content has already taken place, I will set this as CLOSED.