Skip to content
This repository has been archived by the owner on Mar 12, 2021. It is now read-only.

Начало поддержка выгрузки 8.3.7 в иерархическом формате. #38

Merged
merged 5 commits into from
Dec 23, 2015

Conversation

pumbaEO
Copy link
Collaborator

@pumbaEO pumbaEO commented Dec 2, 2015

На текущий момент сделал обратно совместимым, если необходима выгрузка стандартная в иерерхическом виде, то необходимо указать явно параметр -format .
Дополнительно подправил выгрузку под linux, мелкая ошибка с регистром наименования базы данных.

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

ДобавитьПереименование(Переименования,Файл.Имя,ИмяНовогоФайла);
ДобавитьПереименование(Переименования,ИмяФайлаДляПерименования,ИмяНовогоФайла);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Синтакс-ошибка
вместо ИмяФайлаДляПерименования нужно ИмяФайлаДляПереименования и заменить по тексту процедуры

Убрал отладочные ненужные сообщения, перенес в переменную строку описания иерархии.
@artbear
Copy link
Collaborator

artbear commented Dec 9, 2015

@pumbaEO Пулл-реквест не работает с форматом plain :(
Нужно доработать:
1 Параметр формата не передается в функции unpack.ВыгрузитьМодулиКонфигурации и в вызов /DumpConfigToFiles
Нужно "/DumpConfigToFiles """ + КаталогПлоскойВыгрузки + """ -format Plain"
или "/DumpConfigToFiles """ + КаталогПлоскойВыгрузки + """ -format "+Формат
2 Для 1С важен регистр наименований Hierarchical или Plain (иначе будет ошибка выгрузки), а у тебя используется нижний регистр

@pumbaEO
Copy link
Collaborator Author

pumbaEO commented Dec 9, 2015

Надо переделывать в принципе сам v8runner.os , добавлять туда поддержку
8.3.7 , мои правки - это мелкий фикс, который правильней было-бы в
отдельную ветку пока выделить, т.к. теряется полностью обратная
совместимость с 8.3.6... (хотя и виновата в данном случаи сама 1с,
сломавшая поведение по умолчанию).

9 декабря 2015 г., 12:55 пользователь Artur Ayukhanov <
notifications@github.com> написал:

@pumbaEO https://github.com/pumbaEO Пулл-реквест не работает с форматом
plain :(
Нужно доработать:
1 Параметр формата не передается в функции
unpack.ВыгрузитьМодулиКонфигурации и в вызов /DumpConfigToFiles
Нужно "/DumpConfigToFiles """ + КаталогПлоскойВыгрузки + """ -format
Plain"
или "/DumpConfigToFiles """ + КаталогПлоскойВыгрузки + """ -format
"+Формат
2 Для 1С важен регистр наименований Hierarchical или Plain (иначе будет
ошибка выгрузки), а у тебя используется нижний регистр


Reply to this email directly or view it on GitHub
#38 (comment)
.

З повагою Сосна Євген,
mailto:shenja@sosna.zp.ua
skype:shenjasosna
+380933897103

@artbear
Copy link
Collaborator

artbear commented Dec 9, 2015

Я вернул обратную совместимость в 91ba744
До 8.3.7 параметр format 1С учитывать не будет, а для 8.3.7 уже будет.
у меня заработало.

@EvilBeaver
Copy link
Owner

@pumbaEO @artbear парни, я уже немного запутался. Этот пулреквест можно сливать или он требует доработки?

@pumbaEO
Copy link
Collaborator Author

pumbaEO commented Dec 13, 2015

Еще нет.
13 дек. 2015 г. 18:37 пользователь "Andrei Ovsiankin" <
notifications@github.com> написал:

@pumbaEO https://github.com/pumbaEO @artbear
https://github.com/artbear парни, я уже немного запутался. Этот
пулреквест можно сливать или он требует доработки?


Reply to this email directly or view it on GitHub
#38 (comment)
.

@EvilBeaver
Copy link
Owner

@artbear помоги прояснить: @pumbaEO сказал, что сливать пул-реквест пока рано, однако есть коммит 30c235f, в котором ты этот пулреквест влил. Но сам пулреквест все еще открыт. Я запутался.

@artbear
Copy link
Collaborator

artbear commented Dec 22, 2015

Я слил изменения, дополнил их и теперь новый код работает для старого умолчального (plain) формата.
Для нового формата я только ревьюил код, сам не тестил.

Но я не сливал последние правки из #38 (commits)

И странно, что до сих пор числится много изменений, ведь я фактически все слил ранее :(

EvilBeaver added a commit that referenced this pull request Dec 23, 2015
Поддержка выгрузки 8.3.7 в иерархическом формате. Я сливаю реквест, мне нужен код gitsync для рефакторинга по решению задачи #42
@EvilBeaver EvilBeaver merged commit f623654 into EvilBeaver:develop Dec 23, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants