корректный вызов класса

[Dron AKA andy]

1. В цикле дергать menuitem необязательно, и даже как-то некрасиво. Можно передать в класс предварительно отобранный select`ом набор записей и перебирать его командой next. И временная таблица тоже не нужна. Посмотрите внимательней на классы, кот. я рекомендовал раньше, напр. SalesAutoCreate_Basket.

2. Если через menuitem открывается форма, то система не ждет, пока форма закроется, и в clicked() после super() выполняются последующие команды. super() сделал свое дело - открыл форму, и ждать ее закрытия ему никто не говорил.