Ticket #2105 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

募集期間未指定でのイベント作成時、特定環境下で「このページにはアクセスできません」というエラーメッセージが表示される

Reported by: tsukamoto Assigned to: ebihara
Priority: minor Milestone: OpenPNE2.10.5
Component: 指定しない Version: 2.8.x & 2.10.x & 2.11.x
Keywords: OpenPNE2.8.10 OpenPNE2.11.5 Cc:

Description

以下の環境下で、募集期間を指定せずにイベントを作成しようとすると、確認画面で作成ボタン押下時に「このページにはアクセスすることができません。」という表示になり、イベントが作成されません。

  • MySQL 5.x使用
  • my.cnfでsql-modeがSTRICT_TRANS_TABLESなど

[原因]
pc_do_c_event_add_insert_c_commu_topicで、募集期間未指定時に募集期間フィールドに空文字列を指定してることによります。空文字列などの不正値が、上記のsql-modeでなければゼロ値（dateでは"0000-00-00"）変換されますが、上記モードなどではDBエラーになります。
その後、トピック作成時にエラーが発生してfalseを返したものを、トピックIDとみなして処理続行しリダイレクトをかけるので、トピックID 0にアクセスしようとして冒頭のエラーが表示されます。

[提案]
募集期間未指定時は、不正値であるから文字列ではなく、明示的にゼロ値である"0000-00-00"を指定するほうがよいともいます。
2.10.4.2に対するパッチを添付します。

Attachments

openpne_2.10.4.2_diff.txt (0.7 kB) - added by tsukamoto on 03/19/08 07:27:16.
Diff for 2.10.4.2 (trunk)

Change History

03/19/08 07:27:16 changed by tsukamoto

  • attachment openpne_2.10.4.2_diff.txt added.

Diff for 2.10.4.2 (trunk)

03/19/08 19:39:02 changed by ebihara

  • summary changed from 募集期間未指定でのイベント作成 to 募集期間未指定でのイベント作成時、特定環境下で「このページにはアクセスできません」というエラーメッセージが表示される.
  • milestone set to OpenPNE2.11.5.

パッチの提供ありがとうございます。

内容を確認して 2.11.5 に入れたいと思います。

03/19/08 22:03:28 changed by ebihara

  • owner changed from nobody to ebihara.
  • status changed from new to assigned.

内容を確認しました。特段問題ないように思えます。

一応念のために、このパッチと同様の変更を、以下の「datetime型もしくはdate型カラムで、db_now() 以外の値が入ると思われる（空文字列が入るかもしれない）箇所」に対してもおこないます。

  • c_commu_topic.open_date
  • c_commu_topic.invite_period
  • c_login_reject.expired_at
  • c_rss_cache.cache_date
  • c_schedule.start_date
  • c_schedule.start_time
  • c_schedule.end_date
  • c_schedule.end_time

03/19/08 22:12:49 changed by ebihara

いただいたパッチについては、以下のリビジョンで取り込みました。

追加対応をtrunkにおいておこないます。

03/19/08 22:23:51 changed by ebihara

現状、いただいたパッチは以下が不足しているので、先に対応をおこなってしまいます。

  • PC版イベント追加のPostgreSQL対応
  • PC版イベント編集
  • ktai版イベント追加
  • ktai版イベント編集

03/20/08 00:00:30 changed by ebihara

以下のリビジョンで不足していた項目に対応しました（trunk）。

  • PC版イベント追加のPostgreSQL対応, 携帯版イベント追加 : r5884
  • PC版イベント編集, 携帯版イベント編集 : r5885

03/20/08 00:18:04 changed by ebihara

  • keywords changed from 2.11要望 to 2.11要望 確認中.

以下のリビジョンで安定版にも取り込みました。

ご確認ください。

03/24/08 22:15:30 changed by ogawa

  • keywords changed from 2.11要望 確認中 to OpenPNE2.8.10 OpenPNE2.11.5 確認中.
  • version set to 2.8.x & 2.10.x & 2.11.x.
  • type changed from enhancement to defect.
  • milestone changed from OpenPNE2.11.5 to OpenPNE2.10.5.

03/24/08 22:18:08 changed by ogawa

  • keywords changed from OpenPNE2.8.10 OpenPNE2.11.5 確認中 to OpenPNE2.8.10 OpenPNE2.11.5 テスト待ち.

03/24/08 23:03:26 changed by kiwa

#2018に同じ内容のチケットがあがっていたので、こちらでまとめます。

03/25/08 21:36:39 changed by kiwa

  • keywords changed from OpenPNE2.8.10 OpenPNE2.11.5 テスト待ち to OpenPNE2.8.10 OpenPNE2.11.5.
  • status changed from assigned to closed.
  • resolution set to fixed.

テスト完了しました。問題なさそうです。