(1) new code doesn't follow https://zabbix.org/wiki/Docs/specs/coding_style (specifically, spacing)
<richlv> for the record, new/modified still has things like :
if (!check_db_fields($host_db_fields, $host)){
<pavel> RESOLVED.
<Vedmak> CLOSED
(2) there should be a different fix for trunk (this might be be already be planned, just noting so that it's not forgotten) because of gettext usage
(2) I believe it's already fixed in the trunk. I'll double check it.
(3) i don't think i like the following 
> svn log --stop-on-copy svn://svn.zabbix.com/branches/dev/ZBX-3473
------------------------------------------------------------------------
r22661 | pavels | 2011-10-26 11:01:01 +0300 (Wed, 26 Oct 2011) | 1 line
ZBX-3504Adds a more generic error message when no maps are defined
<pavel> Sorry about that. RESOLVED
<Vedmak> CLOSED
(4) [GUI]
Some code reformation we discussed. (IF clauses)
Done. [RESOLVED]
<Vedmak> review please my last commit
<pavels> CLOSED.
Merged to /branches/1.8 revision r22796. CLOSED.