On Friday, June 11, 2004 1:28 PM
Heinzmann, Robert <[EMAIL PROTECTED]> wrote:
>> Heisst, die RegEx erwartet, dass nach dem TicketHook ein
>> Doppelpunkt folgt, dann ein oder kein Zeichen, dann die Ticketnummer.
>
> W�hre es hier nich besser nach
> /$TicketHook:+\s{0,1}($SystemID\d{2,12})/i zu suchen ? Es kann ja
> sonst, je nach Gierigkeit des Perl RegEx'ers, zu Doppeldeutigkeiten
> kommen.

Das h�ngt zuallererst davon ab, wie �blicherweise das Subject im OTRS
zusammengebaut wird:
$ grep -r "\$Subject = .*\$TicketHook" .
./Kernel/Modules/AgentEmail.pm:        $Subject = "[$TicketHook: $Tn]
$Subject";
./Kernel/System/Ticket/Article.pm:    $Subject = "[$TicketHook:
$Article{TicketNumber}] $Subject";
./Kernel/System/Ticket/SendAutoResponse.pm:    $Subject = "[$TicketHook:
$Param{TicketNumber}] $Subject";

Siehe da - der TicketDivider wird nicht benutzt, was uns aber auch dies
sagt:
Kernel/Config/Defaults.pm:    # TicketDivider
Kernel/Config/Defaults.pm:    $Self->{TicketDivider} = ':';
Kernel/System/Ticket/Number/AutoIncrement.pm:    my $TicketDivider =
$Self->{ConfigObject}->Get('TicketDivider') || ': ';

Ausschliesslich AutoIncrement liest den Divider, substituiert aber einen
anderen Standardwert als die Default.pm vorschl�gt (genau hinschauen!).
Genutzt wird er dennoch nicht, wie gesehen.

Zur�ck zu Deinem Vorschlag:
Ich pl�diere daf�r, den TicketDivider auf ': ' zu setzen, also
Doppelpunkt Space. Dann vereinfacht sich dieser Teil der RegEx wie
folgt:

Aus:
    $TicketHook:+.{0,1}

machst Du:
    $TicketHook:+\s{0,1}
Ich find' netter:
   $TicketHook$TicketDivider

Die gesamte RegExp s�he dann f�r Random wie folgt aus:

/$TicketHook$TicketDivider($SystemID\d{2,12})/i

Vorher m�sste nat�rlich $TicketDivider geboren werden:

    my $TicketDivider = $Self->{ConfigObject}->Get('TicketDivider') ||
': ';


>> Das ist ime ein Bug im Code (genauer: vier in vier Modulen),
>> der TicketDivider wird nicht abgefragt.
> Das verwirrt mich jetzt. Nach "analyse" des Codes wird der
> TicketDivider doch gar nirgens ausgewertet oder seh ich das falsch ?

In der Tat heisst es nun, dass $Self{TicketDivider} nicht nur nicht
gelesen, sondern auch nicht geschrieben wird.


Den Bug beheben wir dann wohl f�r's CVS. Der Workaround bleibt bestehen.

> Lauft der Aussage des Entfernen des "[" sollte dies doch aber auch
> nie ein neues Ticket erzeugen oder ?

Das "[" ist vollkommen egal. Das "]" auch.

Es gr��t fr�hlich

Robert Kehl

--
((otrs.de)) :: OTRS GmbH :: Norsk-Data-Str. 1 :: 61352 Bad Homburg
         http://www.otrs.de/ :: Tel. +49 (0)6172 4832388

_______________________________________________
OTRS Mailingliste: otrs-de - Webpage: http://otrs.org/
Archiv: http://lists.otrs.org/pipermail/otrs-de/
Listenabo verwalten: http://lists.otrs.org/cgi-bin/listinfo/otrs-de/
Support oder Consulting fuer Ihr OTRS System?
=> http://www.otrs.de/

Antwort per Email an