プロジェクト

全般

プロフィール

Bug(バグ) #1167

ログイン画面以外の携帯からメールアドレスなどを入力するテキストボックスにistyle属性が設定されていない

Kiwa Sakaiほぼ14年前に追加. 6年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
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)


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #9: 携帯からメールアドレスなどを入力するテキストボックスにistyle属性が設定されていない Fixed(完了) 2009-09-24
関連している opAuthMailAddressPlugin - Bug(バグ) #2580: 「秘密の質問の設定」において、携帯から「秘密の質問への答え」を入力する際にistyle属性が設定されている New(新規) 2009-09-24
関連している OpenPNE 3 - Backport(バックポート) #2581: ログイン画面以外の携帯からメールアドレスなどを入力するテキストボックスにistyle属性が設定されていない Fixed(完了) 2009-09-24
関連している OpenPNE 3 - Backport(バックポート) #2582: ログイン画面以外の携帯からメールアドレスなどを入力するテキストボックスにistyle属性が設定されていない Fixed(完了) 2009-09-24 2012-01-12
関連している OpenPNE 3 - Backport(バックポート) #3104: ログイン画面以外の携帯からメールアドレスなどを入力するテキストボックスにistyle属性が設定されていない Fixed(完了) 2009-09-24

関係しているリビジョン

リビジョン 5d66cbef (差分)
Maki Takahashi12年以上前に追加

(fixes #1167) added "istyle" attribute to form widgets for mobile

リビジョン 20307811 (差分)
Maki Takahashi12年以上前に追加

(fixes #1167) fixed for coding standard

履歴

#1 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種類が考えられますが、どちらも比較的大きな変更になるため、安定版への導入は難しいと考えられます。

#2 Rimpei Ogawa13年以上前に更新

  • 3.6 で発生するかYes にセット

#3 Kousuke Ebihara13年以上前に更新

  • 対象バージョンOpenPNE 3.6beta6 にセット

#4 Kousuke Ebihara13年以上前に更新

  • 対象バージョンOpenPNE 3.6beta6 から OpenPNE 3.7.0 に変更

#5 Maki Takahashi12年以上前に更新

  • 説明 を更新 (diff)
  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Maki Takahashi にセット

#6 Maki Takahashi12年以上前に更新

  • 説明 を更新 (diff)

#7 Maki Takahashi12年以上前に更新

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

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

#8 Maki Takahashi12年以上前に更新

  • 秘密の質問の設定(member/config?category=secretQuestion)のパスワード欄
    パスワード欄の対応はできましたが、「秘密の質問の答え」部分も対応されてしまいます。
    (FormTypeがpasswordであるため)
    「秘密の質問」はopAuthMailAddressPluginで追加されている設定なので別途対応する必要があります。
  • アクセスブロック設定について
    opWidgetFormInputIncreasedのrender()によって、<ul><li>タグにもistyle属性が設定されてしまうようです。

#9 Maki Takahashi12年以上前に更新

  • 説明 を更新 (diff)

Maki Takahashi は書きました:

  • 秘密の質問の設定(member/config?category=secretQuestion)のパスワード欄
    パスワード欄の対応はできましたが、「秘密の質問の答え」部分も対応されてしまいます。
    (FormTypeがpasswordであるため)
    「秘密の質問」はopAuthMailAddressPluginで追加されている設定なので別途対応する必要があります。

パスワードについては、_partsForm.php側で既に対応済みだったようです。

#10 Kousuke Ebihara12年以上前に更新

  • ステータス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() の実装を改善するべきでしょう。今回対応するべきだとは思いませんが、別チケットを作成してください

#11 Maki Takahashi12年以上前に更新

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

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

#12 Maki Takahashi12年以上前に更新

Kousuke Ebihara は書きました:

  • opWidgetFormInputIncreased::render() で ul や li にも属性値が追加されてしまうのは、もともとの実装が悪いせいですね。今回追加した属性値に限らず、フォームフィールドに対して追加することを想定したあらゆる属性値が ul や li にも反映されてしまいます。 opWidgetFormInputIncreased::render() の実装を改善するべきでしょう。今回対応するべきだとは思いませんが、別チケットを作成してください

#2704 を作成いたしました。

#13 Kousuke Ebihara12年以上前に更新

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

#14 Mutsumi Imamura12年以上前に更新

  • 3.6 で発生するかUnknown (未調査) から Yes (はい) に変更
  • 3.4 で発生するかUnknown (未調査) から Yes (はい) に変更

#15 Shouta Kashiwagi約12年前に更新

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

テストOKです。

#16 Chiharu Nakajima6年以上前に更新

3.6対応済み( #2581 )
3.8発生せず

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