プロジェクト

全般

プロフィール

Backport(バックポート) #1265

プロフィール項目の日付やテキストの最小値を最大値より大きくして設定できてしまう

Mutsumi Imamura13年以上前に追加. ほぼ10年前に更新.

ステータス:
Won't fix(対応せず)
優先度:
Low(低め)
担当者:
-
対象バージョン:
開始日:
2010-04-05
期日:
進捗率:

0%


説明

現象

プロフィール項目の登録/編集画面にある、
フォームタイプのテキストと日付にある最小値/最大値の設定で、最小値>最大値という設定ができてしまう。エラー表示もされない。

再現方法

例:日付の場合
  1. 「pc_backend.php/profile/edit」を開く
  2. プルダウンから「自分で入力する」を選択する
  3. フォームタイプで「日付」を選択する
  4. 最小値「2010/04/07」と最大値に「2010/04/05」と入力し、追加ボタンを押す
  5. 設定できてしまう。

最小値、最大値をstrotime()が解釈できる値でも同じように最小値>最大値の設定が出来ます。

再現バージョン

  • OpenPNE3.5.x
  • OpenPNE3.4.x
  • OpenPNE3.2.x
  • OpenPNE3.0.x

修正内容


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #940: プロフィール項目の日付やテキストの最小値を最大値より大きくして設定できてしまう Pending Fixing(修正待ち) 2010-04-05
関連している OpenPNE 3 - Backport(バックポート) #1706: ProfileForm で日付型のプロフィール項目の最大値・最小値の入力欄に now などの strtotime() が解釈できる文字列を入力すると、そのプロフィール項目を保存した時点の日付が DB に保存されてしまう Fixed(完了) 2010-09-17

履歴

#1 Mutsumi Imamura13年以上前に更新

  • 対象バージョンOpenPNE 3.4.6 から OpenPNE 3.4.7 に変更

#2 Shinichi Urabe13年以上前に更新

  • 対象バージョンOpenPNE 3.4.7 から OpenPNE 3.4.8 に変更

#3 Shinichi Urabe13年以上前に更新

  • 優先度Normal(通常) から High(高め) に変更

#4 Shinichi Urabe13年以上前に更新

  • 担当者Shinichi Urabe にセット

#5 Shinichi Urabe13年以上前に更新

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

#6 Shinichi Urabe13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Fixing(修正待ち) に変更
  • 担当者 を削除 (Shinichi Urabe)
  • 対象バージョンOpenPNE 3.4.8 から OpenPNE 3.4.9 に変更

元チケットが差し戻しで取り込むのが難しいと思うので、次バージョンで対応

#7 Shinichi Urabe13年以上前に更新

  • 対象バージョンOpenPNE 3.4.9 から OpenPNE 3.4.10 に変更

#8 Shinichi Urabe約13年前に更新

  • 対象バージョンOpenPNE 3.4.10 から OpenPNE 3.4.11 に変更

次期バージョンでのリリースにします

#9 Shinichi Urabe約13年前に更新

  • 対象バージョンOpenPNE 3.4.11 から OpenPNE 3.4.12 に変更

今回も対応が厳しいため、次期バージョンにのばします

#10 Shinichi Urabeほぼ13年前に更新

  • 対象バージョンOpenPNE 3.4.12 から OpenPNE 3.4.13 に変更

#11 Shinichi Urabeほぼ13年前に更新

  • ステータスPending Fixing(修正待ち) から Won't fix(対応せず) に変更

バグチケットでの対応方針が明確にはなっておらず、安定版での取り込みは難しいので対応しない方針とします

#12 Minoru Takaiほぼ13年前に更新

バグチケットでの対応方針が明確にはなっておらず、安定版での取り込みは難しいので対応しない方針とします

本チケット (#1265, BP from #940) は、 #940 での対応方針が明確でないのでバックポート対応せずとしてクローズすることは賛成です。

ただし、このチケットタイトルで示されているような不具合が存在することは事実です。この件は改めて情報を整理した上で別チケットで対応する予定です。別チケットはまだ作成していませんが、この進捗については #940 を追えば分かるようにするつもりです。

#13 Kousuke Ebihara12年以上前に更新

http://redmine.openpne.jp/issues/940#note-26 で安定版への取り込みについて述べたのでここにも転記しておきます。

h3. 3. 安定版には取り込まれるべきか

このチケットのバックポートチケットである #1265 にて安定版における取り込みについて述べられていたのでこの点についても言及します。

> バグチケットでの対応方針が明確にはなっておらず、安定版での取り込みは難しいので対応しない方針とします

とありますが、バックポートチケットへの対応を検討した時点で対応方針が明確になっていないからといって安定版での取り込みが難しいというのは論理が飛躍している、というより意味がわかりません(別チケット対応にするなどということならわかりますが……)。この問題は明らかにバグで、しかもこの問題によって長期間後方互換性が壊れています。安定版で対応しない理由がありません。安定版の安定性を損なわない形で変更を取り込んでください。

#14 Kousuke Ebihara12年以上前に更新

ところで、 3.2.x 側のバックポートチケット http://redmine.openpne.jp/issues/1264 でおこなわれている対応が取り込まれていないのはなぜでしょうか。 3.2.x 側へのバックポートは誤りだったのでしょうか。

#15 Minoru Takai12年以上前に更新

  • ステータスWon't fix(対応せず) から New(新規) に変更
  • 対象バージョンOpenPNE 3.4.13 から OpenPNE 3.4.15 に変更

ところで、 3.2.x 側のバックポートチケット http://redmine.openpne.jp/issues/1264 でおこなわれている対応が取り込まれていないのはなぜでしょうか。 3.2.x 側へのバックポートは誤りだったのでしょうか。

note-14 で指摘されていることについてコメントしておきます。

本来は、 master 向けチケットで対応した内容をそっくりそのまま BP 対応したかったのですが、 master 向けチケット #940 では仕様と修正内容が煩雑になってしまったため、 master 対応および BP 対応を一時保留としました( http://redmine.openpne.jp/issues/940#note-9 あたりの時期)。

しかし、 3.2.x 向けについては、 3.2 系の対応が終了するという理由から可能な限り少しでも改善を取り込みたいとして、 #940 での修正内容を部分的に取り込み、これを #1264 の修正内容としました。

このチケットは http://redmine.openpne.jp/issues/1265#note-9 で「対応せず」としてクローズされていますが、この事実だけ見るとクローズしたことは不適切です。

  • http://redmine.openpne.jp/issues/1265#note-12 でクローズを妥当だと考えたのは、このチケットタイトル「プロフィール項目の日付やテキストの最小値を最大値より大きくして設定できてしまう」という問題については #940 で結論が導けていないため保留(*今は*対応せず)としてよいと判断したまでです。チケットをクローズしたのは適切ではなかったかもしれません。
  • #940 に絡む内容が整理された上で、このチケットで挙げられている問題や関連する問題(後方互換性の話など)に対しては 3.4.x 向けに対応すべきです。

(そもそも問題の修正方針が落ち着く前にBPチケットが作られている状況が好ましくないのですが) #940 に絡む問題のチケットについては、 3.4.x 向けを再オープンして、対応する予定があることを明示しておきます。

#16 Minoru Takai12年以上前に更新

3.2.x 向けのチケット #1264 (BP form #940) を「バックポート」ではなく「バグ」チケットとしました。3.2.7 では #1595 の問題が解決していないままの動作を(その時点での)仕様とすることにします。

ところで、 3.2.x 側のバックポートチケット http://redmine.openpne.jp/issues/1264 でおこなわれている対応が取り込まれていないのはなぜでしょうか。 3.2.x 側へのバックポートは誤りだったのでしょうか。

#1595 の問題まで含めてみると #1264 の(バックポート)対応は適切な修正内容ではありませんでした。 3.4 系では(少なくとも #1595 対応を含めた)適切な #940 での修正を取り込む予定です。

#17 Mutsumi Imamura12年以上前に更新

  • 対象バージョンOpenPNE 3.4.15 から OpenPNE 3.4.16 に変更

親チケットの対応がまだ完了していませんので、次バージョンへ対象バージョンを変更します。

#18 Shinichi Urabe12年以上前に更新

  • ステータスNew(新規) から Won't fix(対応せず) に変更
  • 対象バージョン を削除 (OpenPNE 3.4.16)

3.4ではひとまず対応を見送ります

#19 Shinichi Urabe12年以上前に更新

  • ステータスWon't fix(対応せず) から Pending Fixing(修正待ち) に変更
  • 対象バージョンOpenPNE 3.4.17 にセット

#20 Kousuke Ebihara12年以上前に更新

  • 対象バージョンOpenPNE 3.4.17 から OpenPNE 3.4.x に変更

#21 Kousuke Ebihara12年以上前に更新

  • 優先度High(高め) から Low(低め) に変更

親チケットにあわせて優先度を下げます。

#22 Rimpei Ogawaほぼ10年前に更新

  • ステータスPending Fixing(修正待ち) から Won't fix(対応せず) に変更

3.4.x 向けの Backport チケットを Won't fix に変更

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