mirror of
https://github.com/Nikolay-Shirokov/cc-1c-skills.git
synced 2026-09-29 13:35:54 +03:00
fix(template-add,help-add): закрыть находки ревью по коду языка и странице
help-add терял текст справки. Отказ смотрит только на Help.xml, а страница может пережить его (удалённый дескриптор, частичная выгрузка, справка, сделанная руками): тогда запись шла безусловно и стирала содержимое с кодом 0 и рапортом [OK]. Теперь существующая страница сохраняется, создаётся только дескриптор — та же защита, что уже стоит в template-add. Прежняя оценка «в help-add потери данных нет» была неверной: она опиралась на чтение кода, а не на замер. Код языка: регулярка пропускала имена устройств Windows. При -Lang nul py-порт молча писал <Page>nul</Page> и пустой каталог с кодом 0 (страница уходила в NUL), а PS падал исключением — то есть порты ещё и расходились. Якоря \A…\z вместо ^…$: последние в обоих языках допускают перевод строки в конце. Проверка вынесена в Test-LangCode / is_valid_lang и внесена в реестр check-inline-drift: inline-блок гард не видел, а копий у него две. Плюс мелочи оттуда же: мёртвый дизъюнкт в $pageExists убран; полумигрированное дерево (дескриптор есть, старый Ext/Template.html остался рядом) теперь получает предупреждение, а не молчание; существование страницы в py сверяется без учёта регистра — иначе на Linux -Lang RU писал бы вторую страницу мимо дескриптора. Четыре кейса на каждый сценарий. Существующие эталоны не сдвинулись. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
Nick Shirokov
co-authored by
Claude Opus 5
Claude Opus 5
parent
c619da4a0a
commit
dc470a3803
@@ -1,4 +1,4 @@
|
||||
# help-add v1.21 — Add built-in help to 1C object (+write_xml_file/write_utf8_bom: общий эталон записи)
|
||||
# help-add v1.22 — Add built-in help to 1C object (+write_xml_file/write_utf8_bom: общий эталон записи)
|
||||
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
|
||||
param(
|
||||
[Parameter(Mandatory)]
|
||||
@@ -185,13 +185,22 @@ if (-not (Test-Path $extDir)) {
|
||||
}
|
||||
|
||||
# Код языка идёт и в текст XML, и в имя файла страницы, поэтому проверяем его до записи:
|
||||
# пустое значение дало бы файл «.html» и пустой <Page></Page>, а разделитель пути —
|
||||
# запись мимо каталога Ext/Help. Оба отказа платформы были бы тихими.
|
||||
# пустое значение дало бы файл «.html» и пустой <Page></Page>, разделитель пути — запись мимо
|
||||
# каталога страниц, а зарезервированное имя устройства (nul, con, prn, aux, com1…, lpt1…) на
|
||||
# Windows уводит запись в само устройство. Все отказы платформы были бы тихими.
|
||||
#
|
||||
# Копия этой проверки есть в template-add (навыки автономны, формат «дескриптор +
|
||||
# страница» у них общий). Держать копии одинаковыми — сознательно.
|
||||
if ($Lang -notmatch '^[A-Za-z0-9_-]+$') {
|
||||
Write-Error "Недопустимый код языка: '$Lang'`nОжидается код вида ru, en (буквы, цифры, дефис, подчёркивание)"
|
||||
# Якоря `\A…\z`, а не `^…$`: последние в обоих языках допускают перевод строки в конце.
|
||||
#
|
||||
# Копия этой функции есть в template-add (навыки автономны, формат «дескриптор + страница»
|
||||
# у них общий). Держать копии одинаковыми — сознательно; за дрейфом следит check-inline-drift.
|
||||
function Test-LangCode([string]$code) {
|
||||
if ($code -notmatch '\A[A-Za-z0-9_-]+\z') { return $false }
|
||||
if ($code -match '\A(?i)(con|prn|aux|nul|com[1-9]|lpt[1-9])\z') { return $false }
|
||||
return $true
|
||||
}
|
||||
|
||||
if (-not (Test-LangCode $Lang)) {
|
||||
Write-Error "Недопустимый код языка: '$Lang'`nОжидается код вида ru, en (буквы, цифры, дефис, подчёркивание; имена устройств Windows недопустимы)"
|
||||
exit 1
|
||||
}
|
||||
|
||||
@@ -250,7 +259,14 @@ $helpHtml = @"
|
||||
</html>
|
||||
"@
|
||||
|
||||
[System.IO.File]::WriteAllText($helpHtmlPath, $helpHtml, $encBom)
|
||||
# Файл страницы НЕ перезаписываем: отказ выше смотрит только на Help.xml, а страница может
|
||||
# пережить его (удалённый дескриптор, частичная выгрузка, справка, сделанная руками) — и тогда
|
||||
# безусловная запись молча стирала бы текст справки с кодом 0.
|
||||
if (Test-Path $helpHtmlPath) {
|
||||
Write-Host "[WARN] Страница $Lang.html уже лежала на диске — содержимое сохранено, создан только дескриптор."
|
||||
} else {
|
||||
[System.IO.File]::WriteAllText($helpHtmlPath, $helpHtml, $encBom)
|
||||
}
|
||||
|
||||
# --- 3. Проверка IncludeHelpInContents в метаданных форм ---
|
||||
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
#!/usr/bin/env python3
|
||||
# help-add v1.21 — Add built-in help to 1C object (+write_xml_file/write_utf8_bom: общий эталон записи)
|
||||
# help-add v1.22 — Add built-in help to 1C object (+write_xml_file/write_utf8_bom: общий эталон записи)
|
||||
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
|
||||
|
||||
import argparse
|
||||
@@ -293,6 +293,26 @@ def write_utf8_bom(path, content):
|
||||
|
||||
|
||||
|
||||
def is_valid_lang(code):
|
||||
"""Код языка идёт и в текст XML, и в имя файла страницы.
|
||||
|
||||
Пустое значение дало бы файл «.html» и пустой <Page></Page>, разделитель пути — запись
|
||||
мимо каталога страниц, а зарезервированное имя устройства (nul, con, prn, aux, com1…, lpt1…)
|
||||
на Windows уводит запись в само устройство: при -Lang nul этот порт молча писал
|
||||
<Page>nul</Page> и пустой каталог с кодом 0. Все отказы платформы были бы тихими.
|
||||
|
||||
fullmatch, а не match: последний с `$` допускает перевод строки в конце.
|
||||
|
||||
Копия этой функции есть в template-add (навыки автономны, формат «дескриптор + страница»
|
||||
у них общий). Держать копии одинаковыми — сознательно; за дрейфом следит check-inline-drift.
|
||||
"""
|
||||
if not re.fullmatch(r"[A-Za-z0-9_-]+", code):
|
||||
return False
|
||||
if re.fullmatch(r"(?i)(con|prn|aux|nul|com[1-9]|lpt[1-9])", code):
|
||||
return False
|
||||
return True
|
||||
|
||||
|
||||
def write_xml_file(path, content):
|
||||
"""XML в каноне выгрузки Конфигуратора: CRLF в разделителях, без перевода в конце.
|
||||
|
||||
@@ -329,15 +349,9 @@ def main():
|
||||
print(f"Каталог объекта не найден: {ext_dir}. Проверьте путь ObjectName (например Catalogs/МойСправочник).", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
# Код языка идёт и в текст XML, и в имя файла страницы, поэтому проверяем его до записи:
|
||||
# пустое значение дало бы файл «.html» и пустой <Page></Page>, а разделитель пути —
|
||||
# запись мимо каталога Ext/Help. Оба отказа платформы были бы тихими.
|
||||
#
|
||||
# Копия этой проверки есть в template-add (навыки автономны, формат «дескриптор +
|
||||
# страница» у них общий). Держать копии одинаковыми — сознательно.
|
||||
if not re.match(r"^[A-Za-z0-9_-]+$", lang):
|
||||
if not is_valid_lang(lang):
|
||||
print(f"Недопустимый код языка: '{lang}'", file=sys.stderr)
|
||||
print("Ожидается код вида ru, en (буквы, цифры, дефис, подчёркивание)", file=sys.stderr)
|
||||
print("Ожидается код вида ru, en (буквы, цифры, дефис, подчёркивание; имена устройств Windows недопустимы)", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
help_xml_path = os.path.join(ext_dir, "Help.xml")
|
||||
@@ -382,7 +396,13 @@ def main():
|
||||
'</html>'
|
||||
)
|
||||
|
||||
write_utf8_bom(help_html_path, help_html)
|
||||
# Файл страницы НЕ перезаписываем: отказ выше смотрит только на Help.xml, а страница может
|
||||
# пережить его (удалённый дескриптор, частичная выгрузка, справка, сделанная руками) — и тогда
|
||||
# безусловная запись молча стирала бы текст справки с кодом 0.
|
||||
if os.path.exists(help_html_path):
|
||||
print(f"[WARN] Страница {lang}.html уже лежала на диске — содержимое сохранено, создан только дескриптор.")
|
||||
else:
|
||||
write_utf8_bom(help_html_path, help_html)
|
||||
|
||||
# --- 3. Check IncludeHelpInContents in form metadata ---
|
||||
|
||||
|
||||
@@ -202,10 +202,23 @@ $templatesDir = Join-Path $processorDir "Templates"
|
||||
$templateMetaPath = Join-Path $templatesDir "$TemplateName.xml"
|
||||
|
||||
# Код языка идёт и в текст XML, и в имя файла страницы, поэтому проверяем его до записи:
|
||||
# пустое значение дало бы файл «.html» и пустой <Page></Page>, а разделитель пути —
|
||||
# запись мимо каталога Ext/Template. Оба отказа платформы были бы тихими.
|
||||
if ($Lang -notmatch '^[A-Za-z0-9_-]+$') {
|
||||
Write-Error "Недопустимый код языка: '$Lang'`nОжидается код вида ru, en (буквы, цифры, дефис, подчёркивание)"
|
||||
# пустое значение дало бы файл «.html» и пустой <Page></Page>, разделитель пути — запись мимо
|
||||
# каталога страниц, а зарезервированное имя устройства (nul, con, prn, aux, com1…, lpt1…) на
|
||||
# Windows уводит запись в само устройство: py-порт при -Lang nul молча писал <Page>nul</Page>
|
||||
# и пустой каталог с кодом 0. Все отказы платформы были бы тихими.
|
||||
#
|
||||
# Якоря `\A…\z`, а не `^…$`: последние в обоих языках допускают перевод строки в конце.
|
||||
#
|
||||
# Копия этой функции есть в help-add (навыки автономны, формат «дескриптор + страница» у них
|
||||
# общий). Держать копии одинаковыми — сознательно; за дрейфом следит check-inline-drift.
|
||||
function Test-LangCode([string]$code) {
|
||||
if ($code -notmatch '\A[A-Za-z0-9_-]+\z') { return $false }
|
||||
if ($code -match '\A(?i)(con|prn|aux|nul|com[1-9]|lpt[1-9])\z') { return $false }
|
||||
return $true
|
||||
}
|
||||
|
||||
if (-not (Test-LangCode $Lang)) {
|
||||
Write-Error "Недопустимый код языка: '$Lang'`nОжидается код вида ru, en (буквы, цифры, дефис, подчёркивание; имена устройств Windows недопустимы)"
|
||||
exit 1
|
||||
}
|
||||
|
||||
@@ -358,6 +371,12 @@ if ($addLangMode) {
|
||||
# Версию берём из самого дескриптора: правка чужой выгрузки не должна менять формат.
|
||||
if ($descText -match '<Help[^>]+version="(\d+\.\d+)"') { $descVersion = $Matches[1] }
|
||||
$langs = @([regex]::Matches($descText, '<Page>([^<]+)</Page>') | ForEach-Object { $_.Groups[1].Value })
|
||||
# Полумигрированное дерево: дескриптор уже есть, а старый Ext/Template.html остался рядом.
|
||||
# Платформа его игнорирует, поэтому текст в нём пропадёт незаметно — говорим вслух.
|
||||
if (Test-Path $legacyPagePath) {
|
||||
Write-Host "[WARN] Рядом лежит старый Ext/Template.html — платформа его игнорирует."
|
||||
Write-Host " Перенесите нужное в Template/<язык>.html и удалите его."
|
||||
}
|
||||
} elseif (Test-Path $legacyPagePath) {
|
||||
# Раскладка до v1.24 — одиночный Ext/Template.html, который платформа молча игнорирует.
|
||||
# Языка у него нет, но создать его могла только версия навыка без параметра -Lang,
|
||||
@@ -372,8 +391,8 @@ if ($addLangMode) {
|
||||
|
||||
# Файл страницы НИКОГДА не перезаписываем: в нём может лежать текст макета.
|
||||
# Ошибка — только когда добавлять нечего: язык уже в дескрипторе И страница на диске.
|
||||
$pageExists = (Test-Path $pagePath) -or ($legacyPending -and $Lang -eq "ru")
|
||||
if (($langs -contains $Lang) -and $pageExists -and -not $legacyPending) {
|
||||
# При миграции проверять нечего: langs там задан нами же, а страница появится переносом.
|
||||
if (($langs -contains $Lang) -and (Test-Path $pagePath) -and -not $legacyPending) {
|
||||
Write-Error "Страница макета на языке '$Lang' уже существует: $pagePath"
|
||||
exit 1
|
||||
}
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
#!/usr/bin/env python3
|
||||
# template-add v1.26 — Add template to 1C object (+write_xml_file/write_utf8_bom: общий эталон записи)
|
||||
# template-add v1.27 — Add template to 1C object (+write_xml_file/write_utf8_bom: общий эталон записи)
|
||||
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
|
||||
|
||||
import argparse
|
||||
@@ -276,6 +276,26 @@ def write_utf8_bom(path, content):
|
||||
|
||||
|
||||
|
||||
def is_valid_lang(code):
|
||||
"""Код языка идёт и в текст XML, и в имя файла страницы.
|
||||
|
||||
Пустое значение дало бы файл «.html» и пустой <Page></Page>, разделитель пути — запись
|
||||
мимо каталога страниц, а зарезервированное имя устройства (nul, con, prn, aux, com1…, lpt1…)
|
||||
на Windows уводит запись в само устройство: при -Lang nul этот порт молча писал
|
||||
<Page>nul</Page> и пустой каталог с кодом 0. Все отказы платформы были бы тихими.
|
||||
|
||||
fullmatch, а не match: последний с `$` допускает перевод строки в конце.
|
||||
|
||||
Копия этой функции есть в help-add (навыки автономны, формат «дескриптор + страница»
|
||||
у них общий). Держать копии одинаковыми — сознательно; за дрейфом следит check-inline-drift.
|
||||
"""
|
||||
if not re.fullmatch(r"[A-Za-z0-9_-]+", code):
|
||||
return False
|
||||
if re.fullmatch(r"(?i)(con|prn|aux|nul|com[1-9]|lpt[1-9])", code):
|
||||
return False
|
||||
return True
|
||||
|
||||
|
||||
def write_xml_file(path, content):
|
||||
"""XML в каноне выгрузки Конфигуратора: CRLF в разделителях, без перевода в конце.
|
||||
|
||||
@@ -379,12 +399,9 @@ def main():
|
||||
templates_dir = os.path.join(processor_dir, "Templates")
|
||||
template_meta_path = os.path.join(templates_dir, f"{template_name}.xml")
|
||||
|
||||
# Код языка идёт и в текст XML, и в имя файла страницы, поэтому проверяем его до записи:
|
||||
# пустое значение дало бы файл «.html» и пустой <Page></Page>, а разделитель пути —
|
||||
# запись мимо каталога Ext/Template. Оба отказа платформы были бы тихими.
|
||||
if not re.match(r"^[A-Za-z0-9_-]+$", lang):
|
||||
if not is_valid_lang(lang):
|
||||
print(f"Недопустимый код языка: '{lang}'", file=sys.stderr)
|
||||
print("Ожидается код вида ru, en (буквы, цифры, дефис, подчёркивание)", file=sys.stderr)
|
||||
print("Ожидается код вида ru, en (буквы, цифры, дефис, подчёркивание; имена устройств Windows недопустимы)", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
# Существующий HTML-макет — не всегда повод отказать: в один макет платформа кладёт
|
||||
@@ -491,6 +508,11 @@ def main():
|
||||
if m_dv:
|
||||
desc_version = m_dv.group(1)
|
||||
langs = re.findall(r"<Page>([^<]+)</Page>", desc_text)
|
||||
# Полумигрированное дерево: дескриптор уже есть, а старый Ext/Template.html остался рядом.
|
||||
# Платформа его игнорирует, поэтому текст в нём пропадёт незаметно — говорим вслух.
|
||||
if os.path.exists(legacy_page_path):
|
||||
print("[WARN] Рядом лежит старый Ext/Template.html — платформа его игнорирует.")
|
||||
print(" Перенесите нужное в Template/<язык>.html и удалите его.")
|
||||
elif os.path.exists(legacy_page_path):
|
||||
# Раскладка до v1.24 — одиночный Ext/Template.html, который платформа молча игнорирует.
|
||||
# Языка у него нет, но создать его могла только версия навыка без параметра -Lang,
|
||||
@@ -506,7 +528,13 @@ def main():
|
||||
# Ошибка — только когда добавлять нечего: язык уже в дескрипторе И страница на диске.
|
||||
# Сравнение без учёта регистра — зеркало -contains в PS, который регистр не различает.
|
||||
lang_known = lang.lower() in [x.lower() for x in langs]
|
||||
page_exists = os.path.exists(page_path) or (legacy_pending and lang.lower() == "ru")
|
||||
# Существование страницы — тоже без учёта регистра: os.path.exists на Linux различает
|
||||
# регистр, а PS -contains и Test-Path на Windows — нет, и порты разошлись бы на -Lang RU.
|
||||
page_exists = False
|
||||
if os.path.isdir(page_dir):
|
||||
want = f"{lang}.html".lower()
|
||||
page_exists = any(f.lower() == want for f in os.listdir(page_dir))
|
||||
# При миграции проверять нечего: langs там задан нами же, а страница появится переносом.
|
||||
if lang_known and page_exists and not legacy_pending:
|
||||
print(f"Страница макета на языке '{lang}' уже существует: {page_path}", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
Reference in New Issue
Block a user