プロジェクト

全般

プロフィール

Backport(バックポート) #900

完了

Can added the profile distinguished name when blank. (プロフィール識別名を空欄でも追加できてしまう)

Masato Nagasawa さんが14年以上前に追加. 14年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-03-11
期日:
進捗率:

100%

予定工数:

説明

概要

プロフィール項目設定ページでプロフィール識別名を空欄でも実行できてしまう

再現環境

  • OpenPNE3.4.3-dev
  • OpenPNE3.5.0-dev
  • Firefox3.5.7

OpenPNE3.2.3-devでは再現せず

再現手順

プロフィール識別名を空欄の状態で追加ボタンを押す

仕様

空欄で項目追加ボタンを押した場合、エラー表示を出す


関連するチケット 2 (0件未完了2件完了)

関連している OpenPNE 3 - Bug(バグ) #839: Can added the profile distinguished name when blank. (プロフィール識別名を空欄でも追加できてしまう)Fixed(完了)Masato Nagasawa2010-03-11

操作
関連している OpenPNE 3 - Bug(バグ) #2356: 管理画面でプロフィール項目の識別名に全角文字や角括弧が使えてしまうFixed(完了)Minoru Takai2011-08-17

操作

Masato Nagasawa さんが14年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 0346d66f767f9df85ec49c46401ebceffa16bbff で適用されました。

Mutsumi Imamura さんが14年以上前に更新

  • 担当者Masato Nagasawa にセット

Mutsumi Imamura さんが14年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

全角ひらがなで識別名を登録できてしまいます。(空欄登録出来ないのは確認しました)
2系の仕様を確認すると、「※半角英数 と _ の組み合わせのみ(数値だけの入力は許容されません)」となっていますので合わせるのが良いかと思います。

Shinichi Urabe さんが14年以上前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

取り消しました。取り消した内容に問題ないか確認をお願いします

Shinichi Urabe さんが14年以上前に更新

更新履歴 e20ab343fbf1ca725b7f1e6c6af47c70d41c21f4 で適用されました。

Kousuke Ebihara さんが14年以上前に更新

  • ステータスPending Review(レビュー待ち) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

取り消されているのを確認しました

Kiwa Sakai さんが14年以上前に更新

プロフィール項目追加時、特に致命的なエラーは発生しませんでした。
取り消しに問題はないと判断します。

Shinichi Urabe さんが14年以上前に更新

  • 対象バージョンOpenPNE 3.4.3 から OpenPNE 3.4.4 に変更

Shinichi Urabe さんが14年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 aa2c673314b1d02778f244c8325bda22f6983010 で適用されました。

Mutsumi Imamura さんが14年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

Imamura Mutsumi は書きました:

全角ひらがなで識別名を登録できてしまいます。(空欄登録出来ないのは確認しました)
2系の仕様を確認すると、「※半角英数 と _ の組み合わせのみ(数値だけの入力は許容されません)」となっていますので合わせるのが良いかと思います。

全角ひらがなで登録出来ないことも確認できました。
一点気になるのは、半角数字のみで登録できるのは問題ないでしょうか?
ご確認をお願いします。

Shinichi Urabe さんが14年以上前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

26f6780b15b0280ed22bd15a5b999ca47bf35f0a

上記コミットで2系と同じにしました。
(-)ハイフンが追加できていたので、アンダースコア、英数字のみに修正しています。

Shinichi Urabe さんが14年以上前に更新

  • 担当者Masato Nagasawa から Shinichi Urabe に変更

Shinichi Urabe さんが14年以上前に更新

更新履歴 26f6780b15b0280ed22bd15a5b999ca47bf35f0a で適用されました。

Mutsumi Imamura さんが14年以上前に更新

  • 進捗率50 から 70 に変更

確認しました。問題ありません。

Rimpei Ogawa さんが14年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

動作上は問題ないですが、sfValidatorAnd を使わなくても、

/^\w*[a-z]\w*$/i

で同じことが表現できると思います。

_ のみの識別子は許可しない仕様で問題ないと思いますが、2系では許可されていたようなので一応確認してください。

Shinichi Urabe さんが14年以上前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

2系の場合、アクションで is_numeric で数値のみは除外しているのですが、
アンダースコアのみで作成できる仕様はあまり好ましいものでないとはんだんしたので、除外しています

Shinichi Urabe さんが14年以上前に更新

  • 進捗率70 から 50 に変更

更新履歴 8a6f1ef79199e36bf8826cdb5f15d82332d08f5f で適用されました。

Rimpei Ogawa さんが14年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更

ソースコードが変更されたので再テストお願いします。

Mutsumi Imamura さんが14年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率50 から 100 に変更

確認しました。問題ありません。

他の形式にエクスポート: Atom PDF