Bugfix inventoryreport3.php

found in version 12.1.5

in <SEPM>\Inetpub\Reporting\Inventory\inventoryreport3.php

lines 460, 462, 464 and 466.

Translation is done twice: one at calling "printFirstColumn", and again in the function "printFirstColumn" again.

This is correct: the function does the translation:

function printFirstColumn($lookup) {
    print "<td class=UCDTableHeader width='120'>".I18nInventory::translateInventory($lookup)."</td>";
}

So there is no need for a tranlation when calling the function, so replace lines 459 - 467:

if ($reporttype==PROFILE_UPTODATE && $reportflavor==GROUP) {
    printFirstColumn(I18nInventory::translateInventory("COLUMN_GROUP_SLASH_DOMAIN"));
} else if ($reportflavor==SUBNET) {
    printFirstColumn(I18nInventory::translateInventory("SUBNET"));
} else if ($reportflavor==GROUP) {
    printFirstColumn(I18nInventory::translateInventory("GROUP"));
} else {
    printFirstColumn(I18nInventory::translateInventory("UNDEFINED"));
}

into:

if ($reporttype==PROFILE_UPTODATE && $reportflavor==GROUP) {
    printFirstColumn("COLUMN_GROUP_SLASH_DOMAIN");
} else if ($reportflavor==SUBNET) {
    printFirstColumn("SUBNET");
} else if ($reportflavor==GROUP) {
    printFirstColumn("GROUP");
} else {
    printFirstColumn("UNDEFINED");
}

This is similar to where it was properly coded in firewallreport1.php and firewallreport1_guts.php.