Project

General

Profile

Backport(バックポート) #900

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

Added by Masato Nagasawa over 12 years ago. Updated over 12 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-03-11
Due date:
% Done:

100%


Description

概要

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

再現環境

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

OpenPNE3.2.3-devでは再現せず

再現手順

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

仕様

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


Related issues

Related to OpenPNE 3 - Bug(バグ) #839: Can added the profile distinguished name when blank. (プロフィール識別名を空欄でも追加できてしまう) Fixed(完了) 2010-03-11
Related to OpenPNE 3 - Bug(バグ) #2356: 管理画面でプロフィール項目の識別名に全角文字や角括弧が使えてしまう Fixed(完了) 2011-08-17

Associated revisions

Revision 0346d66f (diff)
Added by Masato Nagasawa over 12 years ago

fixed added the profile distinguished name is invalid when blank. (fixes #900, BP from #839)

Revision e20ab343 (diff)
Added by Shinichi Urabe over 12 years ago

Revert "fixed added the profile distinguished name is invalid when blank. (fixes #900, BP from #839)"

This reverts commit 0346d66f767f9df85ec49c46401ebceffa16bbff.

Revision aa2c6733 (diff)
Added by Shinichi Urabe over 12 years ago

fixed added the profile distinguished name is invalid when blank. (fixes #900, BP from #839)

Revision 26f6780b (diff)
Added by Shinichi Urabe over 12 years ago

fixed added the profile distinguished name is invalid when only "number" or "_". (fixes #900, BP from #839)

Revision 8a6f1ef7 (diff)
Added by Shinichi Urabe over 12 years ago

(fixes #900, BP from #839) Change the regular expression.

History

#1 Updated by Masato Nagasawa over 12 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#2 Updated by Mutsumi Imamura over 12 years ago

  • Assignee set to Masato Nagasawa

#3 Updated by Mutsumi Imamura over 12 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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

#4 Updated by Shinichi Urabe over 12 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

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

#5 Updated by Shinichi Urabe over 12 years ago

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

#6 Updated by Kousuke Ebihara over 12 years ago

  • Status changed from Pending Review(レビュー待ち) to Accepted(着手)
  • % Done changed from 50 to 0

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

#7 Updated by Kiwa Sakai over 12 years ago

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

#8 Updated by Shinichi Urabe over 12 years ago

  • Target version changed from OpenPNE 3.4.3 to OpenPNE 3.4.4

#9 Updated by Shinichi Urabe over 12 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#10 Updated by Mutsumi Imamura over 12 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

Imamura Mutsumi は書きました:

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

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

#11 Updated by Shinichi Urabe over 12 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

26f6780b15b0280ed22bd15a5b999ca47bf35f0a

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

#12 Updated by Shinichi Urabe over 12 years ago

  • Assignee changed from Masato Nagasawa to Shinichi Urabe

#13 Updated by Shinichi Urabe over 12 years ago

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

#14 Updated by Mutsumi Imamura over 12 years ago

  • % Done changed from 50 to 70

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

#15 Updated by Rimpei Ogawa over 12 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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

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

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

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

#16 Updated by Shinichi Urabe over 12 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

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

#17 Updated by Shinichi Urabe over 12 years ago

  • % Done changed from 70 to 50

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

#18 Updated by Rimpei Ogawa over 12 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)

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

#19 Updated by Mutsumi Imamura over 12 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 50 to 100

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

Also available in: Atom PDF