Skip to content

[テーマ管理] ディレクトリ名のバリデーション不備による文字化けや重複の問題を修正しました #2208

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gakigaki
Copy link
Contributor

@gakigaki gakigaki commented Jun 10, 2025

概要

テーマ管理の「ユーザ・テーマ機能」および「カスタムテーマ生成機能」において、ディレクトリ名のバリデーションルールを修正・統一しました。

変更の目的

サーバーの文字コード設定によっては、ディレクトリ名に日本語などのマルチバイト文字を使用すると文字化けが発生し、テーマが正常に参照できなくなる問題がありました。
また、同名のテーマディレクトリが存在すると意図しない上書きが発生する可能性があるため、これらの問題を未然に防ぐ目的でバリデーションを強化します。

変更内容

対象機能

  • ユーザ・テーマ機能
  • カスタムテーマ生成機能

修正内容

  • ユーザ・テーマ機能:正しく機能していなかったディレクトリ名の正規表現(regex)を修正しました。
  • カスタムテーマ生成機能:ディレクトリ名のバリデーションを追加し、さらに同名のディレクトリが存在しないかチェックする処理も追加しました。

新しいバリデーションルール

  • 使用可能な文字:半角の英数字、アンダースコア(_)、ハイフン(-)。
  • 先頭の文字:半角の英数字、またはアンダースコア(_)。
  • 使用不可能な文字:スペース、スラッシュ(/)、その他の記号。

テスト

両機能の画面にて、以下の項目を確認しました。

  • 許可されていない文字(全角文字、スペース、記号など)を入力した場合に、エラーメッセージが正しく表示されること。
  • カスタムテーマ生成機能で、既に存在するディレクトリ名を入力した場合にエラーメッセージが表示されること。
  • 許可されている形式のディレクトリ名で、正常に登録・作成できること。

レビュー完了希望日

関連Pull requests/Issues

参考

DB変更の有無

無し

チェックリスト

サーバー側の文字コード設定などでディレクトリ名が文字化けするのを回避
- 使える文字: 半角の英数字、アンダースコア(_)、ハイフン(-)
- 先頭の文字: 半角の英数字、またはアンダースコア(_)
- 使えない文字: スペース、スラッシュ(/)、その他の記号
@gakigaki gakigaki added the bug バグ・不具合連絡 label Jun 10, 2025
@gakigaki gakigaki marked this pull request as ready for review June 10, 2025 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグ・不具合連絡 mange(theme)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant