Warum hast du dich entschieden, für jeden Aufruf jeweils eine neue Instanz von TRESTRequest, TRestClient und TRESTResponse zu verwenden?
Gab es dafür gewichtige Gründe?
Instinktiv hätte ich den Request als Klassenvariable angelegt und vor jeden Ausführen geresetet.
Wie gesagt, der Quettext mag syntaktisch richtig sein, schön ist er nicht. Das ist beispielsweise einer der Punkte, die man noch besser machen sollte.
Folgende Fehler habe ich heute noch behoben:
Arrays werden jetzt korrekt beachtet
Request wird nur einmal erstellt und immer wieder verwendet.
Fehler mit ToString bei Strings behoben
Das Problem mit dem getSalesTax existiert immer noch. Das sehe ich mir mal als nächstes die Tage an.