In Bearbeitung Workflow funktioniert nicht - Bedingung: Seriennummer

[Buinger]

Hallo,

wir haben einen Workflow (siehe Bild), wo aufgrund der erweiterten Eigenschaft (siehe folgend) dem Kunden eine Rechnung per Email geschickt wird.

Eine Zeitverzögerung wurde später hinzugefügt, ohne hat es aber auch nicht geklappt. Der Worker läuft natürlich, Tag und Nacht.

Der Workflow funktioniert aber nicht, die Kunden erhalten keine Rechnung. Woran könnte das liegen?


{% for Artikelbeschreibungen in Vorgang.AuftragsPositionen.ArtikelPositionen -%}
{% capture query -%}
SELECT nSeriennummernstatus from tartikel where kArtikel = '{{ Artikelbeschreibungen.InterneArtikelnummer }}'
{% endcapture -%}
{% assign Ergebnis = query | DirectQueryScalar -%}
{% if Ergebnis != 0 -%}
true
{% else -%}
false
{% endif -%}
{{ Ergbnis }}
{% endfor -%}

 

[frankell]

Hallo @Buinger,

hier sind die Hochkommata zu viel, da kArtikel kein String, sondern ein Integer ist:

SELECT nSeriennummernstatus from tartikel where kArtikel = '{{ Artikelbeschreibungen.InterneArtikelnummer }}'

In der Theorie sollte man auch escapen, also:

{{ Artikelbeschreibungen.InterneArtikelnummer | SqlEscape }}

Aber ich habe zumindest bei Integers noch nicht erlebt, dass es da Probleme gibt, wenn man das nicht machen.

Außerdem ist hier ein Typo:

{{ Ergbnis }}

Ist kein Fehler, aber wenn Du eh dabei bist, SQL zu benutzen, dann kannst Du auch den Rest durch SQL machen lassen, also:

{% for Artikelbeschreibungen in Vorgang.AuftragsPositionen.ArtikelPositionen -%}
{% capture query -%}
SELECT
    CASE nSeriennummernstatus
    WHEN 0 THEN 'false'
    ELSE 'true'
    END
FROM
    tartikel
WHERE
    kArtikel = {{ Artikelbeschreibungen.InterneArtikelnummer | SqlEscape  }}
{% endcapture -%}
{% assign Ergebnis = query | DirectQueryScalar -%}
{{ Ergebnis }}
{% endfor -%}

Weitere Möglichkeit:

{% capture query -%}
IF EXISTS (
    SELECT
        *
    FROM
        tLieferscheinPos AS LP
        JOIN Verkauf.tAuftragPosition AS AP ON LP.kBestellPos = AP.kAuftragPosition
        JOIN tArtikel AS ART ON AP.kArtikel = ART.kArtikel
    WHERE
        LP.kLieferschein =  {{ Vorgang.InterneLieferscheinnummer | SqlEscape  }}
        AND ART.nSeriennummernstatus = 1
)
BEGIN SELECT 'true' END
ELSE BEGIN SELECT 'false' END
{% endcapture -%}
{% assign Ergebnis = query | DirectQueryScalar -%}
{{ Ergebnis }}

Damit sparst Du Dir die Schleife in DotLiquid und lässt alles SQL machen. Du kannst dann auch die Erweiterte Eigenschaft zu einem Boolean machen.

 

[Buinger]

Hallo @Buinger,

hier sind die Hochkommata zu viel, da kArtikel kein String, sondern ein Integer ist:



In der Theorie sollte man auch escapen, also:

{{ Artikelbeschreibungen.InterneArtikelnummer | SqlEscape }}

Aber ich habe zumindest bei Integers noch nicht erlebt, dass es da Probleme gibt, wenn man das nicht machen.

Außerdem ist hier ein Typo:



Ist kein Fehler, aber wenn Du eh dabei bist, SQL zu benutzen, dann kannst Du auch den Rest durch SQL machen lassen, also:

{% for Artikelbeschreibungen in Vorgang.AuftragsPositionen.ArtikelPositionen -%}
{% capture query -%}
SELECT
    CASE nSeriennummernstatus
    WHEN 0 THEN 'false'
    ELSE 'true'
    END
FROM
    tartikel
WHERE
    kArtikel = {{ Artikelbeschreibungen.InterneArtikelnummer | SqlEscape  }}
{% endcapture -%}
{% assign Ergebnis = query | DirectQueryScalar -%}
{{ Ergebnis }}
{% endfor -%}

Weitere Möglichkeit:

{% capture query -%}
IF EXISTS (
    SELECT
        *
    FROM
        tLieferscheinPos AS LP
        JOIN Verkauf.tAuftragPosition AS AP ON LP.kBestellPos = AP.kAuftragPosition
        JOIN tArtikel AS ART ON AP.kArtikel = ART.kArtikel
    WHERE
        LP.kLieferschein =  {{ Vorgang.InterneLieferscheinnummer | SqlEscape  }}
        AND ART.nSeriennummernstatus = 1
)
BEGIN SELECT 'true' END
ELSE BEGIN SELECT 'false' END
{% endcapture -%}
{% assign Ergebnis = query | DirectQueryScalar -%}
{{ Ergebnis }}

Damit sparst Du Dir die Schleife in DotLiquid und lässt alles SQL machen. Du kannst dann auch die Erweiterte Eigenschaft zu einem Boolean machen.

Vielen Dank für die ausführliche Antwort.

"Du kannst dann auch die Erweiterte Eigenschaft zu einem Boolean machen." - Ich meine mich erinnern zu können, dass das Skript direkt von JTL gemacht wurde. Hier kann sich niemand erinnern wo es sonst herstammen könnte. Ich hatte ein Ticket geschrieben (schon lange her) und jemand wollte mal "reinschauen". Was genau meinst Du mit diesem Satz?

 

[frankell]

Was genau meinst Du mit diesem Satz?

Da, wo Du den Code der Erweiterten Eigenschaft eingibst, gibt es rechts oben ein Dropdown für den "Rückgabetyp". Boolean steht für Wahr/Falsch. Dem passen sich die Möglichkeiten bei den Workflowbedingungen einer ausgewählten Erweiterten Eigenschaft an.

 

[Thomas Berge]

Hallo @Buinger,

prüfe bitte im Workflow Log, ob der Workflow ausgeführt wurde. Bei zeitverzögerten Workflows solltest du zwei Einträge für deinen Workflow finden. Einmal, dass der Workflow in die Queue eingefügt worden ist und wenn der Workflow durch den Worker abgearbeitet und Aktionen ausgeführt worden sind, wird im Workflow Log ein zweiter Eintrag geschrieben. Gibt es evtl. Fehlermeldungen im Workflow Log? Wenn es zwei Einträge im Log gibt, dann liegt es nicht an der Bedingung, dann könntest du prüfen, ob die Ausgabe an sich funktioniert. Kann die Rechnung manuell an den Kunden versendet werden?

 

[Buinger]

Da, wo Du den Code der Erweiterten Eigenschaft eingibst, gibt es rechts oben ein Dropdown für den "Rückgabetyp". Boolean steht für Wahr/Falsch. Dem passen sich die Möglichkeiten bei den Workflowbedingungen einer ausgewählten Erweiterten Eigenschaft an.

Alles klar. Hast Du evtl. noch eine Zeile so dass der Versand bei diesen Seriennummer-Artikel-Bestellungen automatisch mit dem Workflow gesetzt wird wenn sie verschickt werden?

 

[frankell]

Weitere Möglichkeit:

{% capture query -%}
IF EXISTS (
    SELECT
        *
    FROM
        tLieferscheinPos AS LP
        JOIN Verkauf.tAuftragPosition AS AP ON LP.kBestellPos = AP.kAuftragPosition
        JOIN tArtikel AS ART ON AP.kArtikel = ART.kArtikel
    WHERE
        LP.kLieferschein =  {{ Vorgang.InterneLieferscheinnummer | SqlEscape  }}
        AND ART.nSeriennummernstatus = 1
)
BEGIN SELECT 'true' END
ELSE BEGIN SELECT 'false' END
{% endcapture -%}
{% assign Ergebnis = query | DirectQueryScalar -%}
{{ Ergebnis }}

Damit sparst Du Dir die Schleife in DotLiquid und lässt alles SQL machen. Du kannst dann auch die Erweiterte Eigenschaft zu einem Boolean machen.

Das war fälschlicherweise der Code für einen Versand- Workflow. Hier der korrekte für einen Auftrags-Workflow:

{% capture query -%}
IF EXISTS (
    SELECT
        *
    FROM
        Verkauf.tAuftragPosition AS AP
        JOIN tArtikel AS ART ON AP.kArtikel = ART.kArtikel
    WHERE
        AP.kAuftrag = {{ Vorgang.Stammdaten.InterneAuftragsnummer | SqlEscape  }}
        AND AP.nType = 1
        AND ART.nSeriennummernstatus = 1
)
BEGIN SELECT 'true' END
ELSE BEGIN SELECT 'false' END
{% endcapture -%}
{% assign Ergebnis = query | DirectQueryScalar -%}
{{ Ergebnis }}

so dass der Versand bei diesen Seriennummer-Artikel-Bestellungen automatisch mit dem Workflow gesetzt wird wenn sie verschickt werden

Das verstehe ich leider nicht ganz. Der Versand soll gesetzt werden, wenn verschickt wird?

 

[Buinger]

Das war fälschlicherweise der Code für einen Versand- Workflow. Hier der korrekte für einen Auftrags-Workflow:

{% capture query -%}
IF EXISTS (
    SELECT
        *
    FROM
        Verkauf.tAuftragPosition AS AP
        JOIN tArtikel AS ART ON AP.kArtikel = ART.kArtikel
    WHERE
        AP.kAuftrag = {{ Vorgang.Stammdaten.InterneAuftragsnummer | SqlEscape  }}
        AND AP.nType = 1
        AND ART.nSeriennummernstatus = 1
)
BEGIN SELECT 'true' END
ELSE BEGIN SELECT 'false' END
{% endcapture -%}
{% assign Ergebnis = query | DirectQueryScalar -%}
{{ Ergebnis }}

Das verstehe ich leider nicht ganz. Der Versand soll gesetzt werden, wenn verschickt wird?

Ja genau, irgendwie müssen wir das bei Seriennummer-Artikeln immer manuell setzen.

 

[frankell]

Ja genau, irgendwie müssen wir das bei Seriennummer-Artikeln immer manuell setzen.

Das ist bei Seriennummernartikeln aber auch nicht anders zu erwarten. Du musst ja an irgendeiner Stelle eingeben, um welche Seriennummer es sich handelt. Wenn Du bspw. den Packtisch nutzen würdest (hört sich so an, als würdet Ihr den nicht nutzen), müsstest Du auch da eine Auswahl der Seriennummer treffen.

 

[Buinger]

Das ist bei Seriennummernartikeln aber auch nicht anders zu erwarten. Du musst ja an irgendeiner Stelle eingeben, um welche Seriennummer es sich handelt. Wenn Du bspw. den Packtisch nutzen würdest (hört sich so an, als würdet Ihr den nicht nutzen), müsstest Du auch da eine Auswahl der Seriennummer treffen.

Doch, wir benutzen den Packtisch. Ist es nicht möglich, dass nach Auswahl der Seriennummer das zu automatisieren?

 

[frankell]

Doch, wir benutzen den Packtisch. Ist es nicht möglich, dass nach Auswahl der Seriennummer das zu automatisieren?

Ich habe schlicht Probleme zu verstehen, was Du möchtest. Wenn Du Packtisch nutzt, dann wird ja versendet.

Daher vielleicht noch mal nen Schritt zurück: Was genau wird für Dich wo genau wie genau nicht so angezeigt, wie es Deiner Meinung nach sollte? Gerne auch unterstützt durch Screenshots und die Angabe, wie es aussehen sollte.