Текст тру 1С-ника (штатный)    
Если Проведен Тогда
    НачатьТранзакцию();

    НаборОстаткиТоваров = РегистрыНакопления.ТоварыВРезервеНаСкладах.СоздатьНаборЗаписей();
    НаборОстаткиТоваров.Отбор.Регистратор.Установить(Ссылка);
    НаборОстаткиТоваров.Записать();
    НаборЗаказыПоставщикам = РегистрыНакопления.РазмещениеЗаказовПокупателей.СоздатьНаборЗаписей();
    НаборЗаказыПоставщикам.Отбор.Регистратор.Установить(Ссылка);
    НаборЗаказыПоставщикам.Записать();
КонецЕсли;

ЗаполнитьТабличныеЧастиПередПроведениемУпр(СтруктураПараметров);

Если Проведен Тогда
    ОтменитьТранзакцию();
КонецЕсли;

Текст, который написал бы я:
Если Проведен Тогда

    НаборОстаткиТоваров = РегистрыНакопления.ТоварыВРезервеНаСкладах.СоздатьНаборЗаписей();
    НаборОстаткиТоваров.Отбор.Регистратор.Установить(Ссылка);
    НаборОстаткиТоваров.Записать();
    НаборЗаказыПоставщикам = РегистрыНакопления.РазмещениеЗаказовПокупателей.СоздатьНаборЗаписей();
    НаборЗаказыПоставщикам.Отбор.Регистратор.Установить(Ссылка);
    НаборЗаказыПоставщикам.Записать();

    ЗаполнитьТабличныеЧастиПередПроведениемУпр(СтруктураПараметров);
КонецЕсли;

Но так как написано именно так как написано. Не пойму в чём глубокий смысл штатного кода?
Документ.ЗаказПокупателя -> Кнопка Заполнить и провести

да

Описался. Во втором блоке. Тот что мой. Если не Проведен тогда

да пофиг. да

(3) Гуляй

(4) ты мне не указ

(0) у тебя ЗаполнитьТабличныеЧастиПередПроведениемУпр будет выполняться только когда документ не проведён
в оригинале выполняется всегда

(6)+ видимо в этой процедуре данные из регистров РазмещениеЗаказовПокупателей и ТоварыВРезервеНаСкладах должны браться запросом БЕЗ УЧЁТА движений этого документа
отсюда и фокусы с транзакцией

Смысл кода в том чтобы очистить движения документа перед проведением если он раньше был проведен.
А ты очищаешь итак пустые наборы.

Всё разобрался при откате транзакции отменяться все действия(все сохранённые действия). А документ объект ещё не записывался. Поэтому он останется неоткаченым