Это конфликт вашей реализации логики работы интерфейса и предполагаемой архитектурой приложения с точки зрения My Visual Database.
Судя по всему это у вас форма редактирования. (но я как пользователь был бы недоволен отсутствием видимых кнопок "Сохранить" и "Отмена" - непонятно, в какой момент подтверждается продажа и как её отменить )
Так вот, когда вы вызываете отчет с формы редактирования, то, если запись не была сохранена, MVD пытается её сохранить (это момент отменить или изменить не получится) А у вас в логике сохранения (по цепочке кликов) прописано, что при сохранении нужно добавить ещё товар. Вот и открывается форма добавления товара.
procedure Prodaja_Button2_OnAfterClick (Sender: string);
begin
Prodaja.Button3.Click();
end;
procedure Prodaja_Button3_OnAfterClick (Sender: string);
begin
Prodaja.TableGrid1.SelectedRow := 0;
Prodaja.Button4.Click();
end;
P.S. Формы редактирования с детализацией при всей их внешней привлекательности - тот ещё геморрой...
P.S.P.S. Дайте кнопкам понятные имена - это поможет вам в дальнейшем находить ошибки быстрей.