Hallo Andreas,
das erste, was mir an deinem parametrisierten
SQL-String auffällt, ist das mehrfache Vorkommen von :USER. Da durch ParamByName() nur das erste Vorkommen gefunden werden dürfte, sollte das Statement gar nicht ausführbar sein. Um den Fehler auszuschließen könntest du die Parameter User1, User2, User3 einführen.
#39 ist sehr platzsparend, aber QuotedStr('inactive') finde ich irgendwie selbstdokumentierend - du nicht auch?
Grüße vom marabu