태그가 값과 사유를 사실대로 말하게 한다
세 가지가 <b>사실과 다른 말</b>을 하고 있었다.
## ① 로그인 사용자 태그 다섯 갈래가 피커에서만 비어 보였다
피커가 TagPreviewCatalog.FromUser 의 <b>5-인자 위임판</b>을 부르고 있었다.
그 판은 면허·전문의·직종·부서·부서전화를 빈 문자열로 채운다.
FromUser(tag, id, name, mobile, office)
→ FromUser(tag, id, name, mobile, office, "", "", "", "")
그래서 의사면허번호·전문의번호·직종·부서·부서전화 태그를 피커에서 고르면
"이 계정에는 값이 비어 있습니다"가 뜨는데, 정작 종이에는 실제 값이 찍힌다.
사용자는 그 태그를 못 쓰는 것으로 판단하고 다른 것을 찾게 된다.
이 값들은 로그인 세션(HisUser 15컬럼)이 이미 들고 있다 — 조회를 더하지 않고 본판으로 넘긴다.
## ② 액션 태그 149종에 데이터 태그용 사유가 나갔다
액션 태그는 <b>값이 아니라 동작</b>이다(더블클릭·버튼이 실행한다).
그런데 피커가 데이터 태그와 같은 값 칸을 그려서
"환자 정보가 있어야 값이 나옵니다" 같은 문구가 149종 전부에 붙었다.
오른쪽 설명 칸이 통째로 거짓말을 하고 있었다.
행 종류를 받아 액션이면 "값이 아니라 동작입니다 — 더블클릭·버튼에서 실행됩니다" 한 줄만 둔다.
## ③ 오타 난 태그가 아무 표시 없이 커밋됐다
인라인으로 직접 치는 것이 이 행의 <b>주 경로</b>인데 거기에 검증이 하나도 없었다.
'PAT_이룸' 처럼 한 글자만 틀려도 조용히 저장되고, 그 사실은 미리보기에서 <b>빈칸</b>으로만 드러난다 —
빈칸의 원인이 오타인지 이 환자에게 자료가 없어서인지 가릴 방법이 없다.
글꼴 행과 <b>같은 규칙</b>으로 ⚠ 를 붙인다. 막지는 않는다 —
사이트마다 카탈로그에 없는 커스텀 태그를 쓰기 때문이다. 값은 한 글자도 건드리지 않는다.
## 곁가지 — 사유가 접속에 묶여 있었다
DB 가드가 값 계산보다 <b>위</b>에 있어서, 접속이 없으면 '값을 만들 수 없습니다' 사유까지
통째로 감춰졌다. 그 사유는 정적 판정이라 접속과 무관하다 —
정작 가장 필요한 상황(접속 안 된 단말에서 서식을 훑을 때) 안 보였다.
가드를 값 계산 직전으로 내렸다.
## 판정
- 단위 시험 388 · edit-smoke 390건 전건 통과 (태그 6건 추가)
- --dialog-shots 69장 전건 통과
- --db-render P062 md5 8d683835f5d81e7bb41c79071d6bf954 불변
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Msystech
co-authored by
Claude Fable 5
Claude Fable 5
parent
73095c1415
commit
e162d0fb6b
@@ -622,6 +622,16 @@
|
||||
<Button DockPanel.Dock="Right" Width="28" Height="26" Padding="0" Margin="4,0,0,0"
|
||||
Command="{Binding BrowseCommand}" ToolTip="전체 목록에서 선택"
|
||||
Content="{Binding Source=list, Converter={StaticResource IconName}, ConverterParameter=13}"/>
|
||||
<!--
|
||||
카탈로그에 없는 이름 — 글꼴 행과 같은 규칙이다. 값은 건드리지 않고 표식만 낸다.
|
||||
인라인으로 직접 치는 것이 이 행의 주 경로인데 검증이 하나도 없어서,
|
||||
한 글자 틀려도 조용히 커밋되고 미리보기에서 <b>빈칸</b>으로만 드러났다 —
|
||||
빈칸의 원인이 오타인지 자료가 없어서인지 가릴 수 없었다.
|
||||
-->
|
||||
<TextBlock DockPanel.Dock="Right" Text="⚠" FontSize="13" VerticalAlignment="Center"
|
||||
Margin="4,0,0,0" Foreground="{DynamicResource B.Warning}"
|
||||
ToolTip="{Binding UnknownTagHint}"
|
||||
Visibility="{Binding IsUnknownTag, Converter={StaticResource BoolToVisibility}}"/>
|
||||
<Grid>
|
||||
<TextBox x:Name="TagInput" Style="{StaticResource RowInput}"
|
||||
bh:InspectorFieldBehavior.CommitOnEnter="True"
|
||||
|
||||
@@ -64,6 +64,9 @@ public partial class TagPickerDialogView : Window
|
||||
/// </summary>
|
||||
private readonly IReadOnlyList<string> preferred;
|
||||
|
||||
/// <summary>액션 태그 목록인가 — 값 칸의 뜻이 데이터 태그와 완전히 다르다</summary>
|
||||
private readonly bool isActionKind;
|
||||
|
||||
private const string FilterAll = "*";
|
||||
private const string FilterPreferred = "!";
|
||||
private const string FilterUsed = "+";
|
||||
@@ -81,11 +84,16 @@ public partial class TagPickerDialogView : Window
|
||||
#endregion
|
||||
|
||||
#region Constructors
|
||||
/// <param name="isActionTag">
|
||||
/// 액션 태그 목록인가 — 액션은 <b>값이 아니라 동작</b>이라 값·사유 칸의 뜻이 완전히 다르다.
|
||||
/// 데이터 태그용 문구를 그대로 쓰면 149종 전부에서 오른쪽 설명이 거짓이 된다.
|
||||
/// </param>
|
||||
public TagPickerDialogView(string title, IReadOnlyList<string> tags, string? currentValue,
|
||||
IReadOnlyList<string>? preferredTags = null)
|
||||
IReadOnlyList<string>? preferredTags = null, bool isActionTag = false)
|
||||
{
|
||||
allTags = tags;
|
||||
preferred = preferredTags ?? Array.Empty<string>();
|
||||
isActionKind = isActionTag;
|
||||
previewStore = new SheetMe.Data.Stores.TagPreviewStore(
|
||||
Services.ConfigService.Current.ConnectionString);
|
||||
InitializeComponent();
|
||||
@@ -295,13 +303,21 @@ public partial class TagPickerDialogView : Window
|
||||
/// </summary>
|
||||
private void ShowTagValue(string tag)
|
||||
{
|
||||
if (previewStore is null || !previewStore.CanUseDb)
|
||||
{
|
||||
InfoValueBlock.Visibility = Visibility.Collapsed;
|
||||
return;
|
||||
}
|
||||
InfoValueBlock.Visibility = Visibility.Visible;
|
||||
|
||||
// 액션 태그는 <b>값이 아니다</b>. 더블클릭·버튼이 실행하는 동작이라
|
||||
// "환자 정보가 있어야 값이 나옵니다" 같은 데이터 태그용 사유를 붙이면
|
||||
// 149종 전부에서 이 칸이 통째로 거짓말을 한다.
|
||||
if (isActionKind)
|
||||
{
|
||||
InfoValue.Text = "값이 아니라 동작입니다";
|
||||
InfoValueNote.Text = "더블클릭·버튼에서 실행됩니다 — 종이에 찍히는 값이 아닙니다.";
|
||||
return;
|
||||
}
|
||||
|
||||
// 값을 <b>만들 수 없는 이유</b>는 DB 없이도 안다(정적 판정이다).
|
||||
// 전에는 DB 가드가 이 위에 있어서, 접속이 없으면 사유까지 통째로 감춰졌다 —
|
||||
// 정작 그 사유가 가장 필요한 상황(접속 안 된 단말에서 서식을 훑을 때)에 안 보였다.
|
||||
if (!TagPreviewCatalog.CanResolve(tag))
|
||||
{
|
||||
InfoValue.Text = "값을 만들 수 없습니다";
|
||||
@@ -309,12 +325,29 @@ public partial class TagPickerDialogView : Window
|
||||
return;
|
||||
}
|
||||
|
||||
// 여기서부터는 실제 값이 필요하다 — 그때만 DB 를 따진다.
|
||||
// (진단 스크린샷은 DB 없이 이 창을 띄우므로 이 가드가 없으면 늘 "값을 읽지 못했습니다"가 찍힌다)
|
||||
if (previewStore is null || !previewStore.CanUseDb)
|
||||
{
|
||||
InfoValue.Text = "값은 접속된 단말에서 확인됩니다";
|
||||
InfoValueNote.Text = TagPreviewCatalog.SourceText(tag);
|
||||
return;
|
||||
}
|
||||
|
||||
previewContext ??= previewStore.Read(UserSession.Current.UidCod, UserSession.Current.UidNam);
|
||||
var context = previewContext.Value;
|
||||
var value = TagPreviewCatalog.KindOf(tag) == TagPreviewKind.Clock
|
||||
? TagPreviewCatalog.FromClock(tag, context.Minute12)
|
||||
// <b>본판(10-인자)으로 부른다.</b> 5-인자 위임판은 면허·전문의·직종·부서·부서전화를
|
||||
// 빈 문자열로 채운다 — 그래서 의사면허번호·전문의번호·직종·부서·부서전화 계열이
|
||||
// 피커에서는 "이 계정에는 값이 비어 있습니다"로 보이는데 종이에는 실제 값이 찍혔다.
|
||||
// 값이 나오는 사용자 태그 중 다섯 갈래가 <b>피커에서만</b> 거짓말을 하고 있었다.
|
||||
// 이 값들은 로그인 세션이 이미 들고 있다(HisUser 15컬럼) — 조회를 더 하지 않는다.
|
||||
: TagPreviewCatalog.FromUser(tag, context.UidCod, context.UidNam,
|
||||
context.MobilePhone, context.OfficePhone);
|
||||
context.MobilePhone, context.OfficePhone,
|
||||
UserSession.Current.LicenseNumber, UserSession.Current.SpecialistNumber,
|
||||
UserSession.Current.JobName, UserSession.Current.DepNam,
|
||||
UserSession.Current.DepPhone);
|
||||
|
||||
if (value.Length > 0)
|
||||
{
|
||||
|
||||
Reference in New Issue
Block a user