Bug（バグ） #1167
完了
ログイン画面以外の携帯からメールアドレスなどを入力するテキストボックスにistyle属性が設定されていない
Kiwa Sakai さんが14年以上前に追加.
約7年前に更新.
説明
携帯からメールアドレスなどを入力する際に日本語入力モードが全角かなになっていると非常に不便なので、テキストボックスに適切な 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)
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種類が考えられますが、どちらも比較的大きな変更になるため、安定版への導入は難しいと考えられます。
- 対象バージョン を OpenPNE 3.6beta6 にセット
- 対象バージョン を OpenPNE 3.6beta6 から OpenPNE 3.7.0 に変更
- 説明 を更新 (差分)
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を Maki Takahashi にセット
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
- 秘密の質問の設定（member/config?category=secretQuestion）のパスワード欄
パスワード欄の対応はできましたが、「秘密の質問の答え」部分も対応されてしまいます。
（FormTypeがpasswordであるため）
「秘密の質問」はopAuthMailAddressPluginで追加されている設定なので別途対応する必要があります。
- アクセスブロック設定について
opWidgetFormInputIncreasedのrender()によって、<ul><li>タグにもistyle属性が設定されてしまうようです。
Maki Takahashi は書きました:
- 秘密の質問の設定（member/config?category=secretQuestion）のパスワード欄
パスワード欄の対応はできましたが、「秘密の質問の答え」部分も対応されてしまいます。
（FormTypeがpasswordであるため）
「秘密の質問」はopAuthMailAddressPluginで追加されている設定なので別途対応する必要があります。
パスワードについては、_partsForm.php側で既に対応済みだったようです。
- ステータス を 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() の実装を改善するべきでしょう。今回対応するべきだとは思いませんが、別チケットを作成してください
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
Kousuke Ebihara は書きました:
- opWidgetFormInputIncreased::render() で ul や li にも属性値が追加されてしまうのは、もともとの実装が悪いせいですね。今回追加した属性値に限らず、フォームフィールドに対して追加することを想定したあらゆる属性値が ul や li にも反映されてしまいます。 opWidgetFormInputIncreased::render() の実装を改善するべきでしょう。今回対応するべきだとは思いませんが、別チケットを作成してください
↑ #2704 を作成いたしました。
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
- 3.6 で発生するか を Unknown (未調査) から Yes (はい) に変更
- 3.4 で発生するか を Unknown (未調査) から Yes (はい) に変更
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
他の形式にエクスポート: Atom
PDF
