操作
Bug(バグ) #1167
完了ログイン画面以外の携帯からメールアドレスなどを入力するテキストボックスにistyle属性が設定されていない
開始日:
2009-09-24
期日:
進捗率:
100%
予定工数:
3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Unknown (未調査)
説明
携帯からメールアドレスなどを入力する際に日本語入力モードが全角かなになっていると非常に不便なので、テキストボックスに適切な istyle を設定する。
対応すべき箇所(携帯版)¶
- 友人を誘う(invite)のメールアドレス欄
- PCメールアドレス設定(member/config?category=pcAddress)のメールアドレス欄
- 携帯メールアドレス設定(member/config?category=mobileAddress)のメールアドレス欄
- アクセスブロック設定(member/config?category=accessBlock)のID入力欄
確認バージョン¶
OpenPNE 3.7.0-dev (master)
OpenPNE 3.6.1 (stable-3.6.x)
OpenPNE 3.4.17 (stable-3.4.x)
Kiwa Sakai さんが14年以上前に更新
http://redmine.openpne.jp/issues/1141#note-5 転記
修正の現状をまとめておきます。 本修正は、基本的にBackport元の#9の変更に追従しました。#9では、Symfony1.3以降で導入された、プロジェクト全体のフォームを変更できる「BaseForm」に修正を加えています。このため、Symfony1.3以前のOpenPNE3.2には導入が難しいです。また、OpenPNE3.4/3.5においても、フォームを全てBaseFormから継承させるように変更したわけではなく、BaseFormから継承していないフォームでは、変更が反映されません。これを修正するには ■sfFormから継承したフォームを新規作成し、変更を加え、他のフォームもそれから継承させる ■既存の他のフォームもBaseFormに対応させる の2種類が考えられますが、どちらも比較的大きな変更になるため、安定版への導入は難しいと考えられます。
Maki Takahashi さんが約13年前に更新
- 説明 を更新 (差分)
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を Maki Takahashi にセット
Maki Takahashi さんが約13年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 5d66cbefe990fbf2864bbb90b69cb5948a293649 で適用されました。
Maki Takahashi さんが約13年前に更新
- 秘密の質問の設定(member/config?category=secretQuestion)のパスワード欄
パスワード欄の対応はできましたが、「秘密の質問の答え」部分も対応されてしまいます。
(FormTypeがpasswordであるため)
「秘密の質問」はopAuthMailAddressPluginで追加されている設定なので別途対応する必要があります。
- アクセスブロック設定について
opWidgetFormInputIncreasedのrender()によって、<ul><li>タグにもistyle属性が設定されてしまうようです。
Maki Takahashi さんが約13年前に更新
- 説明 を更新 (差分)
Maki Takahashi は書きました:
- 秘密の質問の設定(member/config?category=secretQuestion)のパスワード欄
パスワード欄の対応はできましたが、「秘密の質問の答え」部分も対応されてしまいます。
(FormTypeがpasswordであるため)
「秘密の質問」はopAuthMailAddressPluginで追加されている設定なので別途対応する必要があります。
パスワードについては、_partsForm.php側で既に対応済みだったようです。
Kousuke Ebihara さんがほぼ13年前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
- 3.6 で発生するか を Yes から Unknown (未調査) に変更
- 3.4 で発生するか を Unknown (未調査) にセット
- lib/form/MemberConfigForm/MemberConfigAccessBlock.class.php の 38 行目に不要なスペースが混入してしまっています
- MemberConfigAccessBlockForm::appendMobileInputMode() のメソッド定義の終端の括弧の次の行から MemberConfigAccessBlockForm::saveConfig() の定義がはじまっていますが、空行を 1 つ追加してください
- opWidgetFormInputIncreased::render() で ul や li にも属性値が追加されてしまうのは、もともとの実装が悪いせいですね。今回追加した属性値に限らず、フォームフィールドに対して追加することを想定したあらゆる属性値が ul や li にも反映されてしまいます。 opWidgetFormInputIncreased::render() の実装を改善するべきでしょう。今回対応するべきだとは思いませんが、別チケットを作成してください
Maki Takahashi さんがほぼ13年前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 203078114ff5b890c3c3530ce419c4c2e8d29ebb で適用されました。
Maki Takahashi さんがほぼ13年前に更新
Kousuke Ebihara は書きました:
- opWidgetFormInputIncreased::render() で ul や li にも属性値が追加されてしまうのは、もともとの実装が悪いせいですね。今回追加した属性値に限らず、フォームフィールドに対して追加することを想定したあらゆる属性値が ul や li にも反映されてしまいます。 opWidgetFormInputIncreased::render() の実装を改善するべきでしょう。今回対応するべきだとは思いませんが、別チケットを作成してください
↑ #2704 を作成いたしました。
Kousuke Ebihara さんがほぼ13年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
Mutsumi Imamura さんがほぼ13年前に更新
- 3.6 で発生するか を Unknown (未調査) から Yes (はい) に変更
- 3.4 で発生するか を Unknown (未調査) から Yes (はい) に変更
Shouta Kashiwagi さんがほぼ13年前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
テストOKです。
操作