From b1a43c36c54cfa0b17a5b3f9d79d34dfe4c792ad Mon Sep 17 00:00:00 2001 From: RyuJongJin <266630912+RyuJongJin@users.noreply.github.com> Date: Tue, 20 Oct 2026 16:46:57 +0900 Subject: [PATCH] =?UTF-8?q?WIP=20=EC=98=B5=EC=85=98=20=EC=B9=B8=20?= =?UTF-8?q?=EB=B3=B4=EC=B6=A9=20(=EA=B4=80=EB=A6=AC=EC=9E=90=20=EC=A7=80?= =?UTF-8?q?=EC=8B=9C=2010/20,=2028=20=EC=A7=80=EC=A0=81):=20--cc=20=C2=B7?= =?UTF-8?q?=20--zc=20=C2=B7=20--zl=20=C2=B7=20--zt=20=EB=93=B1=20=EA=B0=92?= =?UTF-8?q?=20=EB=B0=9B=EB=8A=94=20=EB=B3=84=EC=B9=AD=20(=EC=98=A4?= =?UTF-8?q?=ED=83=90)=20=C2=B7=20-=E2=80=93delete=20=EC=B2=98=EB=9F=BC=20?= =?UTF-8?q?=ED=95=98=EC=9D=B4=ED=94=88=EA=B3=BC=20=EB=8C=80=EC=8B=9C?= =?UTF-8?q?=EA=B0=80=20=EC=84=9E=EC=9D=B8=20=EA=B2=83=EB=8F=84=20=EA=B3=A0?= =?UTF-8?q?=EC=B9=A8=20=C2=B7=20=EC=95=88=EB=82=B4=20=EB=81=9D=20=EB=AC=B8?= =?UTF-8?q?=EA=B5=AC=EB=A5=BC=20=EB=AC=B8=EC=A0=9C=20=EC=A2=85=EB=A5=98?= =?UTF-8?q?=EC=97=90=20=EB=A7=9E=EA=B2=8C=20(=EB=94=B0=EC=98=B4=ED=91=9C?= =?UTF-8?q?=20=EB=8B=AB=EA=B8=B0=20=C2=B7=20-=20=EB=A1=9C=20=EC=8B=9C?= =?UTF-8?q?=EC=9E=91=20=C2=B7=20=EA=B0=92=20=EC=A0=81=EA=B8=B0=20=C2=B7=20?= =?UTF-8?q?=EA=B3=A7=EC=9D=80=20=EA=B8=80=EC=9E=90=EB=A1=9C)=20=C2=B7=20?= =?UTF-8?q?=EC=98=B5=EC=85=98=20=EB=82=B1=EB=A7=90=EC=9D=80=20=EC=A4=84?= =?UTF-8?q?=EB=B0=94=EA=BF=88=20=EC=97=86=EC=9D=B4=20=C2=B7=20=EC=BA=A1?= =?UTF-8?q?=EC=B2=98=20=EC=8B=9C=ED=97=98=20windows=5Foption=5Fproblem=5Fs?= =?UTF-8?q?hot=5Ftest=20(=EA=B2=80=ED=86=A0=20=EB=8C=80=EA=B8=B0)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-Authored-By: Claude Opus 5.5 --- assets/l10n/en.json | 7 +- assets/l10n/ja.json | 7 +- assets/l10n/zh-Hans.json | 7 +- .../windows_option_problem_shot_test.dart | 79 +++++++++++++++++++ lib/app/live_sync.dart | 4 +- lib/app/transfer_job.dart | 4 +- lib/core/sync_tools.dart | 50 ++++++++++-- lib/ui/copy_sync_settings.dart | 6 +- lib/ui/explorer_page.dart | 5 +- lib/ui/monitor_page.dart | 2 +- lib/ui/sync_preview_view.dart | 17 +++- test/option_paste_test.dart | 19 ++++- 12 files changed, 182 insertions(+), 25 deletions(-) create mode 100644 integration_test/windows_option_problem_shot_test.dart diff --git a/assets/l10n/en.json b/assets/l10n/en.json index c886a29..1e4db6f 100644 --- a/assets/l10n/en.json +++ b/assets/l10n/en.json @@ -657,6 +657,7 @@ "감지한 언어: {0} (자동 감지)": "Detected language: {0} (auto-detect)", "감지한 언어: {0} - 틀렸으면 다른 언어로 다시 만드세요": "Detected language: {0} - if it is wrong, make them again with another language", "감지한 원어가 만들 언어와 다를 때만 내려받기 {0}": "Downloaded only if the detected language differs from the subtitle language {0}", + "값을 적어 주세요 (예: --exclude=*.tmp)": "Add a value (e.g. --exclude=*.tmp)", "강하게 (0.4)": "Strong (0.4)", "강할수록 많이 다듬지만 원본과 달라질 수 있습니다": "Stronger refines more but may drift from the original", "같습니다 ✔": "Match ✔", @@ -751,6 +752,7 @@ "고치기": "Edit", "고친 옵션을 저장하지 않았습니다": "The edited options aren't saved", "고화질": "High quality", + "곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요": "Retype with straight quotes and hyphens (-)", "골라서 시작": "Choose and start", "공용 저장소 (Download) 를 쓸 수 없어 되돌릴 수 없습니다.": "Cannot roll back: shared storage (Download) is not available.", "공용 폴더에 받은 파일 (동영상 · 그림 등)": "Files downloaded to shared folders (videos, images, etc.)", @@ -1131,6 +1133,7 @@ "등록됨: \"JJ_MKVMaker 로 재생\" · \"JJ_MKVMaker 로 자막 만들기\" (Windows 11: 더 많은 옵션 표시)": "Registered: \"Play with JJ_MKVMaker\" · \"Make subtitles with JJ_MKVMaker\" (Windows 11: Show more options)", "따르지 않는 것: Tor 화면 (이름도 Tor 안에서 찾음), IP 주소, 같은 집 안 이름 (점 없는 NAS · PC 이름, *.local, Tailscale 의 *.ts.net 등은 기기 DNS)": "Not covered: the Tor screen (names are looked up inside Tor), IP addresses, home-network names (NAS / PC names without a dot, *.local, Tailscale *.ts.net etc. use the device DNS)", "따옴표가 닫히지 않았습니다: {0}": "Unclosed quote: {0}", + "따옴표를 닫아 주세요": "Close the quote", "또는 다른 기기의 카메라로 QR 을 찍으세요 (코드가 들어 있습니다)": "Or scan the QR code with the other device's camera (it includes the code)", "라오어": "Lao", "라이선스 원문 보기": "View full license text", @@ -1892,8 +1895,8 @@ "옮길 항목을 고르세요 (누르거나 오른쪽 동그라미로 표시).": "Choose items to move (tap, or mark with the circle on the right).", "옮길까요?": "move?", "옵션": "Options", - "옵션을 알아볼 수 없습니다: {0}": "Unrecognized option: {0}", - "옵션을 알아볼 수 없습니다: {0} - 곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요": "Unrecognized option: {0} - retype it with straight quotes and hyphens (-)", + "옵션은 - 나 -- 로 시작합니다": "Options start with - or --", + "옵션을 알아볼 수 없습니다: {0} - {1}": "Unrecognized option: {0} - {1}", "옵션을 저장했습니다": "Options saved", "완료": "Done", "완료 시 자동 동영상 추가": "Auto-add videos when done", diff --git a/assets/l10n/ja.json b/assets/l10n/ja.json index 7c674ed..f24585c 100644 --- a/assets/l10n/ja.json +++ b/assets/l10n/ja.json @@ -657,6 +657,7 @@ "감지한 언어: {0} (자동 감지)": "検出した言語: {0} (自動検出)", "감지한 언어: {0} - 틀렸으면 다른 언어로 다시 만드세요": "検出した言語: {0} - 違っていれば別の言語で作り直してください", "감지한 원어가 만들 언어와 다를 때만 내려받기 {0}": "検出した言語が作成する言語と異なる場合のみダウンロード {0}", + "값을 적어 주세요 (예: --exclude=*.tmp)": "値を入力してください (例: --exclude=*.tmp)", "강하게 (0.4)": "強め (0.4)", "강할수록 많이 다듬지만 원본과 달라질 수 있습니다": "強いほど大きく整えますが、元と違ってくることがあります", "같습니다 ✔": "一致します ✔", @@ -751,6 +752,7 @@ "고치기": "編集", "고친 옵션을 저장하지 않았습니다": "編集したオプションが保存されていません", "고화질": "高画質", + "곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요": "半角の引用符とハイフン (-) で入力し直してください", "골라서 시작": "選んで開始", "공용 저장소 (Download) 를 쓸 수 없어 되돌릴 수 없습니다.": "共有ストレージ (Download) を使えないため戻せません。", "공용 폴더에 받은 파일 (동영상 · 그림 등)": "共有フォルダーにダウンロードしたファイル (動画・画像など)", @@ -1131,6 +1133,7 @@ "등록됨: \"JJ_MKVMaker 로 재생\" · \"JJ_MKVMaker 로 자막 만들기\" (Windows 11: 더 많은 옵션 표시)": "登録済み:「JJ_MKVMaker で再生」・「JJ_MKVMaker で字幕を作成」（Windows 11: その他のオプションを表示）", "따르지 않는 것: Tor 화면 (이름도 Tor 안에서 찾음), IP 주소, 같은 집 안 이름 (점 없는 NAS · PC 이름, *.local, Tailscale 의 *.ts.net 등은 기기 DNS)": "対象外: Tor 画面 (名前も Tor 内で解決)、IP アドレス、家庭内ネットワークの名前 (ドットのない NAS・PC 名、*.local、Tailscale の *.ts.net などは端末の DNS)", "따옴표가 닫히지 않았습니다: {0}": "引用符が閉じていません: {0}", + "따옴표를 닫아 주세요": "引用符を閉じてください", "또는 다른 기기의 카메라로 QR 을 찍으세요 (코드가 들어 있습니다)": "または、もう一方の端末のカメラで QR コードを読み取ってください (コード入り)", "라오어": "ラオ語", "라이선스 원문 보기": "ライセンス原文を見る", @@ -1892,8 +1895,8 @@ "옮길 항목을 고르세요 (누르거나 오른쪽 동그라미로 표시).": "移動する項目を選んでください (タップするか右の丸で印を付ける)。", "옮길까요?": "移動しますか?", "옵션": "オプション", - "옵션을 알아볼 수 없습니다: {0}": "オプションを認識できません: {0}", - "옵션을 알아볼 수 없습니다: {0} - 곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요": "オプションを認識できません: {0} - 半角の引用符とハイフン (-) で入力し直してください", + "옵션은 - 나 -- 로 시작합니다": "オプションは - か -- で始まります", + "옵션을 알아볼 수 없습니다: {0} - {1}": "オプションを認識できません: {0} - {1}", "옵션을 저장했습니다": "オプションを保存しました", "완료": "完了", "완료 시 자동 동영상 추가": "完了時に動画を自動追加", diff --git a/assets/l10n/zh-Hans.json b/assets/l10n/zh-Hans.json index 663d1e7..183a5e3 100644 --- a/assets/l10n/zh-Hans.json +++ b/assets/l10n/zh-Hans.json @@ -657,6 +657,7 @@ "감지한 언어: {0} (자동 감지)": "检测到的语言：{0}（自动检测）", "감지한 언어: {0} - 틀렸으면 다른 언어로 다시 만드세요": "检测到的语言：{0} - 如有误，请用其他语言重新生成", "감지한 원어가 만들 언어와 다를 때만 내려받기 {0}": "仅当检测到的语言与要生成的语言不同时下载 {0}", + "값을 적어 주세요 (예: --exclude=*.tmp)": "请填写值 (例: --exclude=*.tmp)", "강하게 (0.4)": "强 (0.4)", "강할수록 많이 다듬지만 원본과 달라질 수 있습니다": "越强优化越多，但可能与原图不同", "같습니다 ✔": "相同 ✔", @@ -751,6 +752,7 @@ "고치기": "编辑", "고친 옵션을 저장하지 않았습니다": "修改的选项尚未保存", "고화질": "高画质", + "곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요": "请用直引号和连字符 (-) 重新输入", "골라서 시작": "选择后开始", "공용 저장소 (Download) 를 쓸 수 없어 되돌릴 수 없습니다.": "无法使用公共存储 (Download)，因此无法回退。", "공용 폴더에 받은 파일 (동영상 · 그림 등)": "下载到公共文件夹的文件 (视频、图片等)", @@ -1131,6 +1133,7 @@ "등록됨: \"JJ_MKVMaker 로 재생\" · \"JJ_MKVMaker 로 자막 만들기\" (Windows 11: 더 많은 옵션 표시)": "已注册: “用 JJ_MKVMaker 播放” · “用 JJ_MKVMaker 生成字幕”（Windows 11: 显示更多选项）", "따르지 않는 것: Tor 화면 (이름도 Tor 안에서 찾음), IP 주소, 같은 집 안 이름 (점 없는 NAS · PC 이름, *.local, Tailscale 의 *.ts.net 등은 기기 DNS)": "不适用: Tor 画面 (名称也在 Tor 内解析)、IP 地址、家庭网络内的名称 (不带点的 NAS · PC 名称、*.local、Tailscale 的 *.ts.net 等使用设备 DNS)", "따옴표가 닫히지 않았습니다: {0}": "引号未闭合: {0}", + "따옴표를 닫아 주세요": "请闭合引号", "또는 다른 기기의 카메라로 QR 을 찍으세요 (코드가 들어 있습니다)": "或用另一台设备的相机扫描二维码 (已包含代码)", "라오어": "老挝语", "라이선스 원문 보기": "查看许可证原文", @@ -1892,8 +1895,8 @@ "옮길 항목을 고르세요 (누르거나 오른쪽 동그라미로 표시).": "请选择要移动的项目 (点按或用右侧圆圈标记)。", "옮길까요?": "移动吗?", "옵션": "选项", - "옵션을 알아볼 수 없습니다: {0}": "无法识别的选项: {0}", - "옵션을 알아볼 수 없습니다: {0} - 곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요": "无法识别的选项: {0} - 请用直引号和连字符 (-) 重新输入", + "옵션은 - 나 -- 로 시작합니다": "选项以 - 或 -- 开头", + "옵션을 알아볼 수 없습니다: {0} - {1}": "无法识别的选项: {0} - {1}", "옵션을 저장했습니다": "已保存选项", "완료": "完成", "완료 시 자동 동영상 추가": "完成后自动添加视频", diff --git a/integration_test/windows_option_problem_shot_test.dart b/integration_test/windows_option_problem_shot_test.dart new file mode 100644 index 0000000..c50aa13 --- /dev/null +++ b/integration_test/windows_option_problem_shot_test.dart @@ -0,0 +1,79 @@ +import 'dart:io'; +import 'dart:ui' as ui; + +import 'package:flutter/material.dart'; +import 'package:flutter/rendering.dart'; +import 'package:flutter_test/flutter_test.dart'; +import 'package:integration_test/integration_test.dart'; +import 'package:jj_mkvmaker/app/i18n_controller.dart'; +import 'package:jj_mkvmaker/core/sync_tools.dart'; +import 'package:jj_mkvmaker/ui/sync_preview_view.dart'; +import 'package:jj_mkvmaker/ui/theme.dart'; +import 'package:path/path.dart' as p; + +/// 관리자 지시 (10/20, 28 지적): 알아볼 수 없는 옵션 안내 - 끝 문구가 문제 종류에 맞는지, 412 에서 넘치지 않는지. +/// 확인 창의 빨간 글 (OptionProblemNote) 과 설정 칸 아래 글 (errorText, 3줄까지) 을 크기 412 · 960, 언어 ko · en · ja 로 찍는다. +/// 실행: flutter test integration_test\windows_option_problem_shot_test.dart -d windows (JJ_SHOT_DIR 에 PNG) +void main() { + IntegrationTestWidgetsFlutterBinding.ensureInitialized(); + final out = Platform.environment['JJ_SHOT_DIR'] ?? p.join(Directory.systemTemp.path, 'jj_shots_optproblem'); + const cases = [ + '-a --delete --exclude="a.tmp --exclude=keep.txt', // 닫히지 않은 따옴표 + '-a delete', // 경로로 읽힘 + '-a --delete --exclude', // 값 없음 + '-a ---delete «x»', // 이상한 글자 + '-av delete --exclude', // 여러 종류 + ]; + for (final lang in ['ko', 'en', 'ja']) { + for (final (w, hh) in [(412, 900), (960, 600)]) { + testWidgets('옵션 안내 $lang ${w}x$hh', (t) async { + t.view.physicalSize = Size(w.toDouble(), hh.toDouble()); + t.view.devicePixelRatio = 1; + addTearDown(t.view.reset); + await t.runAsync(() => i18n.apply(lang, save: false)); + addTearDown(() => i18n.apply('ko', save: false)); + final shotKey = GlobalKey(); + await t.pumpWidget(RepaintBoundary( + key: shotKey, + child: MaterialApp( + theme: buildTheme(), + debugShowCheckedModeBanner: false, + home: Scaffold( + body: ListView(padding: const EdgeInsets.all(16), children: [ + for (final o in cases) ...[ + Text(o, style: const TextStyle(fontFamily: 'monospace', fontSize: 12)), + OptionProblemNote(optionProblems(o)), + const SizedBox(height: 8), + ], + // 설정 칸 (copy_sync_settings 의 rsync 옵션 칸과 같은 폭 · 글 줄 수) + SizedBox( + width: 380, + child: TextField( + controller: TextEditingController(text: cases.last), + decoration: InputDecoration( + isDense: true, + border: const OutlineInputBorder(), + errorText: optionProblemText(optionProblems(cases.last)), + errorMaxLines: 3, + ), + ), + ), + ]), + ), + ), + )); + for (var i = 0; i < 4; i++) { + await t.runAsync(() => Future.delayed(const Duration(milliseconds: 60))); + await t.pump(const Duration(milliseconds: 60)); + } + expect(t.takeException(), isNull, reason: '넘침'); + expect(find.byType(OptionProblemNote), findsNWidgets(cases.length)); + final b = shotKey.currentContext!.findRenderObject()! as RenderRepaintBoundary; + final img = await t.runAsync(() => b.toImage()); + final bytes = await t.runAsync(() => img!.toByteData(format: ui.ImageByteFormat.png)); + Directory(out).createSync(recursive: true); + File(p.join(out, 'optproblem_${lang}_$w.png')).writeAsBytesSync(bytes!.buffer.asUint8List()); + }); + } + } +} diff --git a/lib/app/live_sync.dart b/lib/app/live_sync.dart index f64ee3c..7aa9b02 100644 --- a/lib/app/live_sync.dart +++ b/lib/app/live_sync.dart @@ -511,13 +511,13 @@ class LiveSync extends ChangeNotifier { } /// 관리자 지시 (10/20, 28 확인): 고쳐 읽어도 알아볼 수 없는 rsync 옵션 (있으면 맞추지 않는다 - 카드에 빨갛게) - List unreadableOf(LiveSyncPair x) => unreadableOptions(_optionsOf(x)); + List unreadableOf(LiveSyncPair x) => optionProblems(_optionsOf(x)); Future syncNow(LiveSyncPair x) async { final k = keyOf(x); final bad = unreadableOf(x); final nest = bad.isNotEmpty - ? trf('옵션을 알아볼 수 없습니다: {0}', [bad.join(' · ')]) + ? optionProblemMessage(bad) : nestingProblem(x, allowInnerToOuter: c.settings.allowInnerToOuter); if (nest != null) { status[k] = (DateTime.now(), nest); diff --git a/lib/app/transfer_job.dart b/lib/app/transfer_job.dart index 2d82eab..b44de91 100644 --- a/lib/app/transfer_job.dart +++ b/lib/app/transfer_job.dart @@ -270,8 +270,8 @@ class TransferJob extends ChangeNotifier { try { // 관리자 지시 (10/20, 28 확인): 고쳐 읽어도 알아볼 수 없는 옵션이 남으면 실행하지 않는다 (rsync 가 원본 경로로 읽어 옵션이 몰래 빠짐) if (method == CopyMethod.rsync) { - final bad = unreadableOptions(options); - if (bad.isNotEmpty) throw StateError(trf('옵션을 알아볼 수 없습니다: {0}', [bad.join(' · ')])); + final bad = optionProblems(options); + if (bad.isNotEmpty) throw StateError(optionProblemMessage(bad)); } for (var i = 0; i < sources.length; i++) { final s = sources[i]; diff --git a/lib/core/sync_tools.dart b/lib/core/sync_tools.dart index 3751c43..0b61dcf 100644 --- a/lib/core/sync_tools.dart +++ b/lib/core/sync_tools.dart @@ -157,7 +157,8 @@ List _normalizeWords(List<_Word> raw) { value = ''; continue; } - if (!w.quoted && t.isNotEmpty && _dashLike.contains(t[0])) { + // 28 확인 (10/20): 곧은 하이픈과 섞인 것 (-–delete) 도 맨 앞 둘 중 하나라도 대시면 같은 규칙으로 + if (!w.quoted && t.isNotEmpty && (_dashLike.contains(t[0]) || t.length > 1 && t[0] == '-' && _dashLike.contains(t[1]))) { final n = _dashAt(t, 1) ? 2 : 1; final rest = t.substring(n); final name = rest.split('=').first; @@ -190,15 +191,51 @@ const _rsyncLongWithValue = { '--block-size', '--checksum-choice', '--compress-choice', '--compress-level', '--skip-compress', '--info', '--debug', '--sockopts', '--port', '--address', '--outbuf', '--remote-option', '--max-alloc', '--stop-after', '--stop-at', '--write-batch', '--only-write-batch', '--read-batch', '--protocol', '--checksum-seed', '--copy-as', '--early-input', + // 28 확인 (10/20): rsync --help 의 짧은 별칭 (aka --cc · --zc · --zl · --zt) 과 값 받는 옵션 - 빠져 있어 값이 "경로로 읽힙니다" 로 막혔다 + '--cc', '--zc', '--zl', '--zt', '--compress-threads', '--confine-root', }; /// 관리자 지시 (10/20, 28 · 15 확인): [splitOptions] 로 고쳐 읽은 뒤에도 rsync 가 다르게 읽을 것 - 있으면 확인 창 · 실시간 동기화에 /// 빨갛게 알리고 실행을 막는다: 닫히지 않은 따옴표, 옵션 자리의 - 로 시작하지 않는 낱말 (rsync 는 원본 경로로 읽어 옵션이 몰래 빠짐), /// 하이픈 셋 이상, 끝에 값 없이 남은 값 받는 옵션. 옵션의 값 자리 (--exclude 한글.txt 의 한글.txt, -f 뒤) 는 세지 않는다 -List unreadableOptions(String options) { +List unreadableOptions(String options) => [for (final x in optionProblems(options)) x.text]; + +/// 옵션 칸의 문제 종류 - 안내의 끝 문구 (고칠 방법) 를 고른다 (관리자 지시 10/20, 28 지적) +enum OptionProblemKind { unclosed, path, noValue, odd } + +/// [unreadableOptions] 의 한 줄과 그 종류 +class OptionProblem { + final OptionProblemKind kind; + final String text; + + /// [text] 안의 옵션 낱말 (화면이 줄바꿈 없이 묶는다) + final String token; + const OptionProblem(this.kind, this.text, [this.token = '']); + + @override + String toString() => text; +} + +/// 문제 종류에 맞는 고칠 방법 (여러 종류면 · 로 잇는다) +String optionProblemHint(Iterable problems) { + final kinds = {for (final x in problems) x.kind}; + return [ + if (kinds.contains(OptionProblemKind.unclosed)) tr('따옴표를 닫아 주세요'), + if (kinds.contains(OptionProblemKind.path)) tr('옵션은 - 나 -- 로 시작합니다'), + if (kinds.contains(OptionProblemKind.noValue)) tr('값을 적어 주세요 (예: --exclude=*.tmp)'), + if (kinds.contains(OptionProblemKind.odd)) tr('곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요'), + ].join(' · '); +} + +/// "옵션을 알아볼 수 없습니다: … - 고칠 방법" (작업 기록 · 실시간 동기화 카드 · 설정 칸) +String optionProblemMessage(List problems) => + trf('옵션을 알아볼 수 없습니다: {0} - {1}', [problems.join(' · '), optionProblemHint(problems)]); + +/// [unreadableOptions] 의 몸통 (종류와 함께) +List optionProblems(String options) { final parsed = _parseOptions(options); // 15 확인 F1: 닫히지 않은 따옴표 - 뒤의 옵션이 다르게 읽힐 수 있다 - final bad = [for (final u in parsed.unclosed) trf('따옴표가 닫히지 않았습니다: {0}', [u])]; + final bad = [for (final u in parsed.unclosed) OptionProblem(OptionProblemKind.unclosed, trf('따옴표가 닫히지 않았습니다: {0}', [u]), u)]; var value = false; var last = ''; for (final t in parsed.words) { @@ -208,7 +245,7 @@ List unreadableOptions(String options) { } if (t.startsWith('---')) { // 관리자 판단 (10/20, 28 확인): 하이픈이 셋 이상 (---delete) 은 rsync 가 알아보지 못한다 - bad.add(t); + bad.add(OptionProblem(OptionProblemKind.odd, t, t)); } else if (t.startsWith('--')) { value = !t.contains('=') && _rsyncLongWithValue.contains(t); last = t; @@ -220,11 +257,12 @@ List unreadableOptions(String options) { } else { // 관리자 판단 (10/20, 28 확인): 옵션 칸에는 경로가 들어갈 자리가 없다 (원본 · 대상은 앱이 넣음) - 값 자리가 아닌데 // - 로 시작하지 않는 낱말 (-a delete 의 delete, 혼자 쓴 -) 은 rsync 가 원본 경로로 읽어 옵션이 몰래 빠진다 - bad.add(trf('경로로 읽힙니다: {0}', [t])); + // ASCII 밖 글자로 시작하면 (« … 등) 붙여 넣은 이상한 글자 - 안내는 곧은 글자로 다시 적기 + bad.add(OptionProblem(t.codeUnitAt(0) > 0x7F ? OptionProblemKind.odd : OptionProblemKind.path, trf('경로로 읽힙니다: {0}', [t]), t)); } } // 값이 필요한 옵션이 끝에 값 없이 (--exclude) - rsync 가 원본 경로를 그 값으로 먹는다 - if (value) bad.add(trf('{0} 의 값이 없습니다', [last])); + if (value) bad.add(OptionProblem(OptionProblemKind.noValue, trf('{0} 의 값이 없습니다', [last]), last)); return bad; } diff --git a/lib/ui/copy_sync_settings.dart b/lib/ui/copy_sync_settings.dart index a04406a..f7342de 100644 --- a/lib/ui/copy_sync_settings.dart +++ b/lib/ui/copy_sync_settings.dart @@ -19,7 +19,7 @@ import 'rsync_setup.dart'; import 'schedule_editor.dart'; import 'theme.dart'; import 'setting_tile.dart'; -import 'sync_preview_view.dart' show OptionTextFormatter; +import 'sync_preview_view.dart' show OptionTextFormatter, optionProblemText; /// 환경 설정의 복사 · 동기화 부분. /// - 파일 탐색기 ([rsync] false): 파일 / 폴더 복사 방법 (현재 방식 · robocopy), robocopy 옵션, 속도 제한 @@ -79,7 +79,7 @@ class _CopySyncSettingsState extends State { Widget _options(String title, String hint, String value, String def, void Function(AppSettings, String) set, {bool rsync = false}) { final ctl = _optionText.putIfAbsent(title, () => TextEditingController(text: value)); // 관리자 지시 (10/20, 28 확인): 고쳐 읽어도 알아볼 수 없는 rsync 옵션은 칸 아래에 빨갛게 (실행 · 실시간 동기화는 막힘) - final bad = rsync ? unreadableOptions(ctl.text) : const []; + final bad = rsync ? optionProblems(ctl.text) : const []; return SettingTile( title: Text(title), subtitle: Text(hint), @@ -93,7 +93,7 @@ class _CopySyncSettingsState extends State { decoration: InputDecoration( isDense: true, border: const OutlineInputBorder(), - errorText: bad.isEmpty ? null : trf('옵션을 알아볼 수 없습니다: {0}', [bad.join(' · ')]), + errorText: bad.isEmpty ? null : optionProblemText(bad), errorMaxLines: 3, suffixIcon: IconButton( tooltip: trf('기본값으로 ({0})', [def]), diff --git a/lib/ui/explorer_page.dart b/lib/ui/explorer_page.dart index 9f75c83..ad4924b 100644 --- a/lib/ui/explorer_page.dart +++ b/lib/ui/explorer_page.dart @@ -3419,7 +3419,10 @@ class _ExplorerPageState extends State { // 관리자 지시 (10/19): 원격 (앱이 맞춤) 도 같은 거르기를 따르므로 미리 볼 수 없는 규칙도 똑같이 알린다 List filterLimits() => {for (final t in tasks) ...RsyncFilter.parse(opts(t)).unsupported}.toList(); // 관리자 지시 (10/20, 28 확인): 고쳐 읽어도 알아볼 수 없는 옵션이 남으면 [실행] 을 막는다 - List badOptions() => {for (final t in tasks) ...unreadableOptions(opts(t))}.toList(); + List badOptions() { + final seen = {}; + return [for (final t in tasks) ...optionProblems(opts(t)).where((x) => seen.add(x.text))]; + } // 93: 지우는 실행 (--delete 등. 100: WebDAV 도 따른다) 은 비교가 끝나 지울 목록이 보인 뒤에만 bool deletes() => tasks.any((t) => optionsDelete(opts(t))); // 105: 원본이 대상 안인데 지우기가 들어가면 막는다 - 창 안에서 옵션이 바뀔 때 (원본 파일 지우기) 와 실행 직전에도 본다 diff --git a/lib/ui/monitor_page.dart b/lib/ui/monitor_page.dart index cc35e4c..f48901f 100644 --- a/lib/ui/monitor_page.dart +++ b/lib/ui/monitor_page.dart @@ -663,7 +663,7 @@ class _CopyCardState extends State<_CopyCard> { style: TextStyle(fontSize: 12, color: t.move ? Colors.orangeAccent : JjColors.textDim), ), // 관리자 지시 (10/20, 28 확인): 고쳐 읽어도 알아볼 수 없는 옵션 (실행은 작업이 막는다) - if (_method == 'rsync' && unreadableOptions(_options.text).isNotEmpty) OptionProblemNote(unreadableOptions(_options.text)), + if (_method == 'rsync' && optionProblems(_options.text).isNotEmpty) OptionProblemNote(optionProblems(_options.text)), if (!t.move && optionsRemoveSource(_options.text)) Text(tr('복사 작업이라 칸의 원본 지우기 옵션 (--remove-source-files · /MOV) 은 쓰지 않습니다. 옮기려면 Rsync 화면에서 [원본 파일 지우기] 를 켜세요'), style: const TextStyle(fontSize: 12, color: Colors.orangeAccent)), diff --git a/lib/ui/sync_preview_view.dart b/lib/ui/sync_preview_view.dart index f9bca8a..b622115 100644 --- a/lib/ui/sync_preview_view.dart +++ b/lib/ui/sync_preview_view.dart @@ -228,7 +228,7 @@ Future confirmDeletingRun(BuildContext context, CopyTask t, {bool includeM // 177: rsync 로 맞출 때만 옵션의 거르기 (--exclude 등) 를 비교에도 (앱이 직접 · robocopy 는 이 규칙을 쓰지 않음) // 관리자 지시 (10/19): 원격 (앱이 맞춤) 도 rsync 를 고른 작업이면 같은 거르기를 따른다 (현재 방식 · robocopy 는 이 규칙을 쓰지 않음) final filter = t.method == 'rsync' ? RsyncFilter.parse(t.options) : RsyncFilter.none; - final bad = t.method == 'rsync' ? unreadableOptions(t.options) : const []; + final bad = t.method == 'rsync' ? optionProblems(t.options) : const []; final ok = await showDialog( context: context, builder: (ctx) => StatefulBuilder( @@ -335,15 +335,26 @@ class OptionTextFormatter extends TextInputFormatter { } /// 관리자 지시 (10/20, 28 확인): 고쳐 읽어도 알아볼 수 없는 옵션 - 빨갛게 알리고 [실행] 을 막는다 (rsync 가 원본 경로로 읽어 옵션이 몰래 빠짐) +/// 알아볼 수 없는 옵션 안내 글 (확인 창 · 설정 칸): 옵션 낱말은 줄바꿈 없이 묶고 (28 지적: "--" 와 "exclude" 로 갈림), +/// 고칠 방법의 - · -- 뒤에서도 줄을 바꾸지 않는다 (나머지 글은 보통처럼 줄을 바꾼다) +String optionProblemText(List bad) { + final wj = String.fromCharCode(0x2060); + return trf('옵션을 알아볼 수 없습니다: {0} - {1}', [ + bad.map((x) => x.token.isEmpty ? x.text : x.text.replaceFirst(x.token, unbreakableOption(x.token))).join(' · '), + optionProblemHint(bad).replaceAll('-', '-$wj'), + ]); +} + class OptionProblemNote extends StatelessWidget { - final List bad; + final List bad; const OptionProblemNote(this.bad, {super.key}); @override Widget build(BuildContext context) => Padding( padding: const EdgeInsets.only(bottom: 4), child: Text( - trf('옵션을 알아볼 수 없습니다: {0} - 곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요', [bad.map(unbreakableOption).join(' · ')]), + // 28 지적 (10/20): 끝 문구는 문제 종류에 맞게 (따옴표 닫기 · - 로 시작 · 값 적기 · 곧은 글자로) + optionProblemText(bad), style: const TextStyle(fontSize: 12, color: Colors.redAccent, fontWeight: FontWeight.w600)), ); } diff --git a/test/option_paste_test.dart b/test/option_paste_test.dart index 3d51bcd..00271cf 100644 --- a/test/option_paste_test.dart +++ b/test/option_paste_test.dart @@ -105,11 +105,28 @@ void main() { expect(unreadableOptions('-a --exclude *.log --delete'), isEmpty, reason: '값이 있음'); expect(unreadableOptions('-a --exclude=*.log'), isEmpty); expect(unreadableOptions(''), isEmpty); + // 28 확인 (10/20): rsync 의 짧은 별칭 (--cc · --zc · --zl) 도 값을 받는다 (오탐이던 것) + expect(unreadableOptions('-a --cc md5 --zc zlib --zl 3 --zt 2'), isEmpty); + // 곧은 하이픈과 대시가 섞인 것 (-\u2013delete) + expect(splitOptions('-a -\u2013delete \u2013-del'), ['-a', '--delete', '--del']); + expect(unreadableOptions('-a -\u2013delete'), isEmpty); + }); + + test('28 지적 (10/20): 안내 끝 문구는 문제 종류에 맞게', () { + String hint(String o) => optionProblemHint(optionProblems(o)); + expect(hint('-a --exclude="a.tmp'), '따옴표를 닫아 주세요'); + expect(hint('-a delete'), '옵션은 - 나 -- 로 시작합니다'); + expect(hint('-a --exclude'), '값을 적어 주세요 (예: --exclude=*.tmp)'); + expect(hint('-a ---delete'), '곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요'); + expect(hint('-a \u00ABx\u00BB'), '곧은 따옴표와 하이픈 (-) 으로 다시 적어 주세요'); + expect(hint('-a delete --exclude'), '옵션은 - 나 -- 로 시작합니다 · 값을 적어 주세요 (예: --exclude=*.tmp)'); + expect(optionProblemMessage(optionProblems('-a delete')), '옵션을 알아볼 수 없습니다: 경로로 읽힙니다: delete - 옵션은 - 나 -- 로 시작합니다'); }); testWidgets('알아볼 수 없는 옵션 알림 (빨간 글, 옵션 글은 줄바꿈 없이)', (tester) async { - await tester.pumpWidget(const MaterialApp(home: Scaffold(body: OptionProblemNote(['\u00ABdelete\u00BB'])))); + await tester.pumpWidget(MaterialApp(home: Scaffold(body: OptionProblemNote(optionProblems('-a \u00ABdelete\u00BB'))))); expect(find.textContaining('옵션을 알아볼 수 없습니다'), findsOneWidget); + expect(find.textContaining('곧은 따옴표와 하이픈'), findsOneWidget); }); // 28 확인 (10/20): 막기 전 실제 rsync 는 이런 낱말을 원본 경로로 먹는다 - 그래서 [unreadableOptions] 로 막는다