mirror of
https://github.com/Nikolay-Shirokov/cc-1c-skills.git
synced 2026-08-20 10:09:42 +03:00
fix(skd-edit): паритет портов по тире в сообщениях
Три сообщения py-порта писали `--` там, где ps1 пишет `—`: rename-parameter с равными именами, reorder-parameters с пустым списком и дедуп SelectedItemAuto. Расхождение роняло кейс add-selection-auto-dedup на py-прогоне, из-за чего кейс был ослаблен до куска строки без тире — и перестал сторожить сам текст. Порты сведены по ps1-эталону, кейс сверяет полную строку. Сообщение про отсутствие изменений не тронуто: оно одинаково в обоих портах, и на него опирается отдельный кейс. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
fddaca37e7
commit
863c7ebbff
@@ -1,4 +1,4 @@
|
|||||||
# skd-edit v1.36 — Atomic 1C DCS editor (+esc_xml/esc_xml_text: разное экранирование атрибута и текста)
|
# skd-edit v1.37 — Atomic 1C DCS editor (+esc_xml/esc_xml_text: разное экранирование атрибута и текста)
|
||||||
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
|
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
|
||||||
# NB: парный .py собирает выражения автодат вне f-string ради совместимости с python 3.9 (PEP 701).
|
# NB: парный .py собирает выражения автодат вне f-string ради совместимости с python 3.9 (PEP 701).
|
||||||
param(
|
param(
|
||||||
|
|||||||
@@ -1,4 +1,4 @@
|
|||||||
# skd-edit v1.36 — Atomic 1C DCS editor (Python port) (+esc_xml/esc_xml_text: разное экранирование атрибута и текста)
|
# skd-edit v1.37 — Atomic 1C DCS editor (Python port) (+esc_xml/esc_xml_text: разное экранирование атрибута и текста)
|
||||||
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
|
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
|
||||||
import argparse
|
import argparse
|
||||||
import json
|
import json
|
||||||
@@ -2437,7 +2437,7 @@ elif operation == "rename-parameter":
|
|||||||
new_name = m_rn.group(2).strip()
|
new_name = m_rn.group(2).strip()
|
||||||
|
|
||||||
if old_name == new_name:
|
if old_name == new_name:
|
||||||
print('[WARN] rename-parameter: old and new names are equal -- skipped')
|
print('[WARN] rename-parameter: old and new names are equal — skipped')
|
||||||
continue
|
continue
|
||||||
|
|
||||||
# 1. Rename <parameter><name>OldName</name>
|
# 1. Rename <parameter><name>OldName</name>
|
||||||
@@ -2494,7 +2494,7 @@ elif operation == "reorder-parameters":
|
|||||||
for val in values:
|
for val in values:
|
||||||
order = [s.strip() for s in val.split(",") if s.strip()]
|
order = [s.strip() for s in val.split(",") if s.strip()]
|
||||||
if not order:
|
if not order:
|
||||||
print('[WARN] reorder-parameters: empty list -- skipped')
|
print('[WARN] reorder-parameters: empty list — skipped')
|
||||||
continue
|
continue
|
||||||
|
|
||||||
all_params = []
|
all_params = []
|
||||||
@@ -2647,7 +2647,7 @@ elif operation == "add-selection":
|
|||||||
break
|
break
|
||||||
if is_dup:
|
if is_dup:
|
||||||
target = f'group "{group_name}"' if group_name else f'variant "{var_name}"'
|
target = f'group "{group_name}"' if group_name else f'variant "{var_name}"'
|
||||||
print(f'[WARN] SelectedItemAuto already exists in {target} -- skipped')
|
print(f'[WARN] SelectedItemAuto already exists in {target} — skipped')
|
||||||
continue
|
continue
|
||||||
|
|
||||||
sel_indent = get_container_child_indent(selection)
|
sel_indent = get_container_child_indent(selection)
|
||||||
|
|||||||
@@ -26,6 +26,6 @@
|
|||||||
"value": "Auto ;; Поле1 ;; Поле2"
|
"value": "Auto ;; Поле1 ;; Поле2"
|
||||||
},
|
},
|
||||||
"expect": {
|
"expect": {
|
||||||
"stdoutContains": "[WARN] SelectedItemAuto already exists in variant \"Основной\""
|
"stdoutContains": "[WARN] SelectedItemAuto already exists in variant \"Основной\" — skipped"
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user