-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Исправил ошибку "папка не пуста" при инкрементальной выгрузке #38
Исправил ошибку "папка не пуста" при инкрементальной выгрузке #38
Conversation
src/Классы/increment.os
Outdated
// КаталогВыгрузки = ВременныеФайлы.СоздатьКаталог(); | ||
|
||
ТекущийФайлВерсийМетаданных = Новый Файл(ОбъединитьПути(КаталогВыгрузки, "ConfigDumpInfo.xml")); | ||
ТекущийФайлВерсийМетаданных = Новый Файл(ОбъединитьПути(КаталогПроверки, "ConfigDumpInfo.xml")); | ||
|
||
Если НЕ ТекущийФайлВерсийМетаданных.Существует() И ПустаяСтрока(ПутьКФайлуВерсийДляСравнения) Тогда |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
а как он может существовать, если мы каталог только создали двумя строками выше?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Каталог для того и создали чтобы проверка не мусорила в каталоге выгрузки.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
я не совсем про это. ТекущийФайлВерсийМетаданных собирается из "Каталога проверки", а не "каталога выгрузки". там никогда не может быть файла версий метаданных, соответственно первая часть условий всегда будет "истина"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это условие, по моему, изначально лишнее было. Убрал.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@khorevaa
Эни бади хоум?)))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Уехал :)) в Сочи.. как вернуся... так и сделаю :))
@khorevaa сделаешь релиз гитсинка с новыми плагинами? |
@nixel2007 сделаю |
При инкрементальной выгрузке появляется ошибка - "папка не пуста".
Причина - при проверки возможности выгрузке в папке выгрузки появляется файл ConfigDumpInfo.xml.
Чтобы этого не происходило и на всякий случай для проверки используем новый временный каталог