Project

General

Profile

Defect #920

ref_issues macro failed with named queries that contains date ranges.

Added by Haru Iida almost 8 years ago. Updated over 7 years ago.

Status:
終了(Closed)
Priority:
通常(Normal)
Target version:
Start date:
01/20/2012
Due date:
% Done:

100%

Estimated time:

Description

クエリーに日付が含まれるとエラーになるそうです。
修正方法も提示されてるので対処をお願いできますか?

http://www.redmine.org/boards/3/topics/7130?r=28613#message-28613


Related issues

Related to Wiki Extensions - Code Review #988: ref_issuesの引数に'D'がある?終了(Closed)03/31/2012

Actions

Associated revisions

Revision 1650148a (diff)
Added by Tomohisa Kusukawa over 7 years ago

fixes #920 ref_issues macro failed with named queries that contains date ranges.

History

#1

Updated by Tomohisa Kusukawa almost 8 years ago

  • Status changed from 新規(New) to 担当(Assigned)

ご連絡ありがとうございます。
対応致します。

#2

Updated by Tomohisa Kusukawa over 7 years ago

  • Status changed from 担当(Assigned) to 解決(Resolved)
  • Assignee changed from Tomohisa Kusukawa to Haru Iida

https://bitbucket.org/kusu/redmine_wiki_extensions
にて修正を行いましたので、取り込みをお願いいたします。

#3

Updated by Haru Iida over 7 years ago

  • Assignee changed from Haru Iida to Tomohisa Kusukawa

修正ありがとうございます。

pull requestを送ってもらえますか?

#4

Updated by Tomohisa Kusukawa over 7 years ago

  • Assignee changed from Tomohisa Kusukawa to Haru Iida

https://bitbucket.org/haru_iida/redmine_wiki_extensions/pull-request/4/redmine130-ref_issues
にてプルリクエストしました。
よろしくお願いします。

#5

Updated by Haru Iida over 7 years ago

  • Assignee changed from Haru Iida to Tomohisa Kusukawa

もはや何をやっているのかさっぱりわかりませんが、楠川さんを信じて取り込みました。:)

#6

Updated by Tomohisa Kusukawa over 7 years ago

  • % Done changed from 0 to 100

更新履歴 commit:8eee8ae8d74f で適用されました。

#7

Updated by Tomohisa Kusukawa over 7 years ago

ありがとうございます。

今回修正した部分はshinagawa.redmineでもちょっと話題になった
いわゆるモンキーパッチの部分で、
Redmineが1.3.0でそのメソッドに修正が入ったので追従させなければならなかったものです。

ref_issuesでは題名や説明の文字列検索条件としてor抽出ができる仕様にしているのですが、
sql_for_fieldメソッドには元々or抽出の機能が無く、
or検索機能を追加したメソッドに差し替えています。

差し替える対象は、
ref_issues用に生成したqueryオブジェクトのメソッド(特異メソッド?)だけなので、
Redmine本体の振る舞いを壊したりはしませんが、
今回のように、メソッドがリファクタリングされたりすると
すぐに不整合を起こしてしまいます。

今回の修正はRedmine本体のsql_for_fieldをコピーしてきて、
必要な機能追加を前回同様に再度行っただけです。

良いやり方ではないのですが、
うまい方法が思いつかずにおります。

#8

Updated by Haru Iida over 7 years ago

  • Status changed from 解決(Resolved) to 終了(Closed)

Also available in: Atom PDF