Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

名前の記述を統一する #991

Open
seasoftjapan opened this issue Sep 1, 2024 · 3 comments · May be fixed by #1143
Open

名前の記述を統一する #991

seasoftjapan opened this issue Sep 1, 2024 · 3 comments · May be fixed by #1143
Assignees
Milestone

Comments

@seasoftjapan
Copy link
Contributor

#982 (comment) でスペースに関する指摘があった。

敬称も含めて確認したい。

arrCustomer を渡して、整形する PHP 関数・Smarty 修飾子も候補にはなりそう。少数だと、逆に見通しが悪くなるデメリットも大きそうなので、どの程度対象箇所があるか調査した上で判断か。

@seasoftjapan
Copy link
Contributor Author

seasoftjapan commented Jan 10, 2025

調査結果 (姓名に関して)

区切り文字 該当数
全SP 3
半SP 23
SPなし 12

※ grep「name01.*name02」で抽出された箇所を一回の目視によるカウント。
※ テストコードは除外。
※ モバイル端末テンプレートは除外。

半角スペース区切りに統一で進める。

@seasoftjapan
Copy link
Contributor Author

敬称についての調査結果

敬称 該当数
「様」のみ 3
半SP+様 10
全SP+様 5
「さん」のみ 3
なし 41
  • 今回整形対象とする見込みの箇所を対象として調査。(先の姓名の調査と概ね同じが、フリガナも含む。)

既存でなしの箇所は現状維持。それ以外は、「半SP+様」に統一を予定。

seasoftjapan added a commit that referenced this issue Jan 11, 2025
敬称はベタ書き版
seasoftjapan added a commit that referenced this issue Jan 11, 2025
敬称も関数で対応
@seasoftjapan
Copy link
Contributor Author

冒頭で触れた「arrCustomer を渡して、整形する PHP 関数・Smarty 修飾子」を実装しました。

敬称も関数で処理しようかと思いました。しかし、例外的な記述に対応できず (本体実装では1箇所のみでしたが)、テンプレートの記述量が無駄に増える箇所も多かったので、敬称はベタ書きでいこうと思います。

b0b0ddf 敬称も関数
41d49f6 敬称はベタ書き ← こちらの方向で PR 予定

もう少し確認・調整します。

@seasoftjapan seasoftjapan self-assigned this Jan 13, 2025
@seasoftjapan seasoftjapan linked a pull request Jan 13, 2025 that will close this issue
@nanasess nanasess modified the milestones: 2.18(仮), 2.x Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants