ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 12 years ago

#2105 closed defect (fixed)

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

Reported by: tsukamoto Owned by: 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 (1)

openpne_2.10.4.2_diff.txt (691 bytes) - added by tsukamoto 12 years ago.
Diff for 2.10.4.2 (trunk)

Download all attachments as: .zip

Change History (11)

Changed 12 years ago by tsukamoto

Attachment: openpne_2.10.4.2_diff.txt added

Diff for 2.10.4.2 (trunk)

comment:1 Changed 12 years ago by ebihara

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

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

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

comment:2 Changed 12 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

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

一応念のために、このパッチと同様の変更を、以下の「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

comment:3 Changed 12 years ago by ebihara

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

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

comment:4 Changed 12 years ago by ebihara

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

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

comment:5 Changed 12 years ago by ebihara

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

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

comment:6 Changed 12 years ago by ebihara

Keywords: 確認中 added

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

ご確認ください。

comment:7 Changed 12 years ago by ogawa

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 added; 2.11要望 removed
Milestone: OpenPNE2.11.5OpenPNE2.10.5
Type: enhancementdefect
Version: 2.8.x & 2.10.x & 2.11.x

comment:8 Changed 12 years ago by ogawa

Keywords: テスト待ち added; 確認中 removed

comment:9 Changed 12 years ago by kiwa

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

comment:10 Changed 12 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

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

Note: See TracTickets for help on using tickets.