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.

 

[Buinger]

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.

Da hast Du Recht - noch mal einen Schritt zurück - viele Köchte verderben den Brei (hier intern): Also erst mal vorab - der Packtisch wird hierfür NICHT genutzt (sorry).

Hier direkt von der Kollegin im Laden:

"Es sollen halt alle Selbstabholer im Laden gleich verschickt werden (als "verschickt" markiert werden meint sie).

Da hat es mal geheißen, dass man Ser.nr. Artikel ausschließen muss, da das bei denen net geht (die Info ist alt - stimmt das noch so?

Aber der Rest sollte mitm Workflow verschickt werden" (eben als "verschickt" markiert werden).

Das sind Kunden, die ihre Ware im Laden persönlich abholen.

 

[Buinger]

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.

Könntest Du bitte noch mal schauen (vorherige Antwort)?!

 

[frankell]

Also, wenn ich es nun korrekt verstehe, möchtest einen Workflow für Aufträge, der wie folgt aussieht:

Bedingung: keine Seriennummerartikel enthalten
Aktion 1: "Ausliefern"
Aktion 2: Rechnung mailen

Das Thema Bedingung hatten wir ja geklärt.

Mit der Workflow-Aktion "Ausliefern" kannst Du auch gleichzeitig das Mailen der Rechnung abhandeln: Wenn Du die Workflow-Aktion auswählst, dann auf "Konfigurieren" und dann auf "Erweiterte Optionen". Dort hast Du alle Optionen und kannst im Bereich "Lagereinstellungen - Eigenes Lager - Versand" das "Versanddatum für alle Pakete setzen (Versand setzen)".

 

[Buinger]

Also, wenn ich es nun korrekt verstehe, möchtest einen Workflow für Aufträge, der wie folgt aussieht:

Bedingung: keine Seriennummerartikel enthalten
Aktion 1: "Ausliefern"
Aktion 2: Rechnung mailen

Das Thema Bedingung hatten wir ja geklärt.

Mit der Workflow-Aktion "Ausliefern" kannst Du auch gleichzeitig das Mailen der Rechnung abhandeln: Wenn Du die Workflow-Aktion auswählst, dann auf "Konfigurieren" und dann auf "Erweiterte Optionen". Dort hast Du alle Optionen und kannst im Bereich "Lagereinstellungen - Eigenes Lager - Versand" das "Versanddatum für alle Pakete setzen (Versand setzen)".

So war es auch hinterlegt (siehe Bilder). Ich verstehe leider nicht, warum es dann nicht funktioniert.

 

[frankell]

Gut, aber den Haken bei "E-Mail" hast Du nicht gesetzt. Probier mal bitte mit dem Haken.

Wenn das immer noch nichts hilft, bitte auch mal einen Screenshot von der Erweiterten Eigenschaft und vom Workflow- Log.

 

[Buinger]

Gut, aber den Haken bei "E-Mail" hast Du nicht gesetzt. Probier mal bitte mit dem Haken.

Wenn das immer noch nichts hilft, bitte auch mal einen Screenshot von der Erweiterten Eigenschaft und vom Workflow-Log.

Es geht leider immer noch nicht, wir hatte heute viele Selbstabholer, der Workflow "greift" nicht.

Hier ist der Screenshot von den "Erweiterten Eigenschaften":

 

[frankell]

Testest Du die Workflows eigentlich? Wenn nicht, ist das das Erste, das Du machen solltest. Dafür gibt es die Funktion " Workflow testen". Auch Erweiterte Eigenschaften können "getestet" werden (Vorschau).

Wenn es nicht an einem Bug liegt, bekommst Du angezeigt, wie sich die einzelnen Bedingungen ausgewirkt haben. Dann am besten nach dem Ausschlussprinzip verfahren, um so dem Übeltäter auf die Spur zu kommen. Ansonsten bekommst Du außerhalb eines Tests im Workflow- Log eine Ausgabe, was beim Workflow passiert ist (Admin - Hintergrundprozesse - Workflow Log).

 

[Buinger]

Testest Du die Workflows eigentlich? Wenn nicht, ist das das Erste, das Du machen solltest. Dafür gibt es die Funktion " Workflow testen". Auch Erweiterte Eigenschaften können "getestet" werden (Vorschau).

Wenn es nicht an einem Bug liegt, bekommst Du angezeigt, wie sich die einzelnen Bedingungen ausgewirkt haben. Dann am besten nach dem Ausschlussprinzip verfahren, um so dem Übeltäter auf die Spur zu kommen. Ansonsten bekommst Du außerhalb eines Tests im Workflow-Log eine Ausgabe, was beim Workflow passiert ist (Admin - Hintergrundprozesse - Workflow Log).

Doch, in der Regel teste ich, aber hier braucht's ja erst mal einen Selbstabholer.

Sorry, das mit dem Log hatte ich vergessen, in dem Log waren einige wenige Meldungen - alle gleich:

Die Bedingung "Auftragspositionen.ArtikelPositionen.enthält.SeriennummernAlsText" enthält keine Einträge.

 

[frankell]

Doch, in der Regel teste ich, aber hier braucht's ja erst mal einen Selbstabholer.

Bei einem Test geht es nur um die Bedingungen. Das bedeutet, dass auch ein "alter" Auftrag als Testobjekt herhalten kann.

Sorry, das mit dem Log hatte ich vergessen, in dem Log waren einige wenige Meldungen - alle gleich:
Die Bedingung "Auftragspositionen.ArtikelPositionen.enthält.SeriennummernAlsText" enthält keine Einträge.

Das hatte ich ganz vergessen zu schreiben: Diese Bedingung ist überflüssig, weil bei einem noch nicht versendeten Auftrag (präzise: ohne Lieferschein) normalerweise auch noch keine Konkretisierung der Seriennummer stattgefunden hat.

Aber "keine Einträge" dürfte ja "ist leer" entsprechen und damit True ergeben, oder?