Использование ACL

[bloodice]

Как я понимаю если не задать параметр, то он отсутствует у тикета соответственно ACL проверку не проходит. Мол нет параметра нет проверки.

[ykolesnikov]

Так проверьте. Это займет немного времени. Отображение на экране и наличие начального значения это, по моему разные вещи.

[bloodice]

Я бы не сказал, что хорошо знаю потроха OTRS, потому и задаю вопрос (Да и perl мне не нравится, и не учил я его). А то мало ли вдруг, что то где то пропустил.
На сколько я понимаю OTRS в ACL не обрабатывает значения которых не существует. Ибо проверка:

Код: Выделить всё

elsif ( defined $UsedChecks{$Key}->{$Data} ) {

Кроме как дописывать код, идей у меня нет. К примеру так:

Код: Выделить всё

} elsif ( !defined $UsedChecks{$Key}->{$Data} && $Item eq 'undef' ) {
    $MatchProperty = 1;
    # debug
    if ( $Self->{ACLDebug} ) {
        $Kernel::OM->Get('Kernel::System::Log')->Log(
            Priority => $Self->{ACLDebugLogPriority},
            Message =>
                "TicketACL '$Acl' $PropertiesHash:'$Key->$Data' Matched ($Item eq undef)",
        );
    }
}

И вроде работает. Но это изменение кода OTRS, а мне бы не хотелось в исполняемых файлах, что либо менять (что бы при апдейтах не париться).

[paver]

... а мне бы не хотелось в исполняемых файлах, что либо менять (что бы при апдейтах не париться).

Хотел было написать про кастомный каталог, но взглянув на дату вашей регистрации на форуме, решил, что я чего-то не понял.

[bloodice]

Про кастомный каталог я знаю. Просто при переходе на новую версию могут меняться и те файлы которые подверглись изменению (изначальный вариант). Могли баги пофиксить, а могли и какие нибудь механизмы поменять. Текущая рабочая версия имеет более десяти измененных файлов, из-за чего обновление становится тем еще гемором. Поэтому в "новом" релизе цель настроить все с минимумом изменений исполняемых файлов.

[alexus]

1. Более 10 - это 11, 50 или 100500? Да если и 500 - это не проблема, а расходы
2. Чтобы давать советы по ACL, нужно как минимум понимать версию OTRS. Вот старожил вроде, а подпись так и нет.

[bloodice]

Озвучивание необходимости доп. расходов руководство воспринимает как попытку отобрать у них последнюю конфетку. Необходимость приходится чуть ли не выгрызать. Так что лучше заранее позаботиться о сохранности своих нервов, к тому же это интересно .
Ради интереса подсчитал 23 pm файла изменено (это в 3 ветке), не считая своих дополнительных модулей. Подпись добавил. Можно еще в 5 версии посмотреть не поменялось ли чего, но в изменениях ничего такого не заметил.