Backport(バックポート) #1265
完了プロフィール項目の日付やテキストの最小値を最大値より大きくして設定できてしまう
0%
説明
現象¶
プロフィール項目の登録/編集画面にある、
フォームタイプのテキストと日付にある最小値/最大値の設定で、最小値>最大値という設定ができてしまう。エラー表示もされない。
再現方法¶
例:日付の場合- 「pc_backend.php/profile/edit」を開く
- プルダウンから「自分で入力する」を選択する
- フォームタイプで「日付」を選択する
- 最小値「2010/04/07」と最大値に「2010/04/05」と入力し、追加ボタンを押す
- 設定できてしまう。
最小値、最大値をstrotime()が解釈できる値でも同じように最小値>最大値の設定が出来ます。
再現バージョン¶
- OpenPNE3.5.x
- OpenPNE3.4.x
- OpenPNE3.2.x
- OpenPNE3.0.x
修正内容¶
Shinichi Urabe さんが約14年前に更新
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
Shinichi Urabe さんが約14年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Fixing(修正待ち) に変更
- 担当者 を削除 (
Shinichi Urabe) - 対象バージョン を OpenPNE 3.4.8 から OpenPNE 3.4.9 に変更
元チケットが差し戻しで取り込むのが難しいと思うので、次バージョンで対応
Shinichi Urabe さんが13年以上前に更新
- 対象バージョン を OpenPNE 3.4.11 から OpenPNE 3.4.12 に変更
今回も対応が厳しいため、次期バージョンにのばします
Shinichi Urabe さんが13年以上前に更新
- ステータス を Pending Fixing(修正待ち) から Won't fix(対応せず) に変更
バグチケットでの対応方針が明確にはなっておらず、安定版での取り込みは難しいので対応しない方針とします
Kousuke Ebihara さんが13年以上前に更新
http://redmine.openpne.jp/issues/940#note-26 で安定版への取り込みについて述べたのでここにも転記しておきます。
h3. 3. 安定版には取り込まれるべきか このチケットのバックポートチケットである #1265 にて安定版における取り込みについて述べられていたのでこの点についても言及します。 > バグチケットでの対応方針が明確にはなっておらず、安定版での取り込みは難しいので対応しない方針とします とありますが、バックポートチケットへの対応を検討した時点で対応方針が明確になっていないからといって安定版での取り込みが難しいというのは論理が飛躍している、というより意味がわかりません(別チケット対応にするなどということならわかりますが……)。この問題は明らかにバグで、しかもこの問題によって長期間後方互換性が壊れています。安定版で対応しない理由がありません。安定版の安定性を損なわない形で変更を取り込んでください。
Kousuke Ebihara さんが13年以上前に更新
ところで、 3.2.x 側のバックポートチケット http://redmine.openpne.jp/issues/1264 でおこなわれている対応が取り込まれていないのはなぜでしょうか。 3.2.x 側へのバックポートは誤りだったのでしょうか。
Minoru Takai さんが13年以上前に更新
- ステータス を 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 向けを再オープンして、対応する予定があることを明示しておきます。
Minoru Takai さんが13年以上前に更新
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 での修正を取り込む予定です。
Mutsumi Imamura さんが13年以上前に更新
- 対象バージョン を OpenPNE 3.4.15 から OpenPNE 3.4.16 に変更
親チケットの対応がまだ完了していませんので、次バージョンへ対象バージョンを変更します。
Shinichi Urabe さんが約13年前に更新
- ステータス を New(新規) から Won't fix(対応せず) に変更
- 対象バージョン を削除 (
OpenPNE 3.4.16)
3.4ではひとまず対応を見送ります
Shinichi Urabe さんが約13年前に更新
- ステータス を Won't fix(対応せず) から Pending Fixing(修正待ち) に変更
- 対象バージョン を OpenPNE 3.4.17 にセット
Rimpei Ogawa さんが10年以上前に更新
- ステータス を Pending Fixing(修正待ち) から Won't fix(対応せず) に変更
3.4.x 向けの Backport チケットを Won't fix に変更