Project

General

Profile

Code Review #988

ref_issuesの引数に'D'がある?

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

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

100%

Estimated time:
Code review: /lib/wiki_extensions_ref_issue.rb@8eee8ae8d74f:line 21

Description

ref_issuesのテストコードを書きたいと思っているのですが、コードで分からない部分があるので教えてください。

引数チェックで'D'をチェックしていますがwikiの説明を見るかぎりdはあってもDはないようです。このDは何を意味しているのでしょうか?


Related issues

Related to Wiki Extensions - Defect #920: ref_issues macro failed with named queries that contains date ranges.終了(Closed)01/20/2012

Actions

History

#1

Updated by Tomohisa Kusukawa over 7 years ago

  • Status changed from 新規(New) to フィードバック(Reopend)
  • Assignee changed from Tomohisa Kusukawa to Haru Iida

以前のバージョンでは
wオプションで検索キーワードを指定する仕様で、
その機能指定として
S:サブジェクトを検索対象にしない
D:説明を検索対象にしない
をwの前に付けていました。

これら古い仕様のオプションで書かれた既存のWikiページを壊してしまわないように
当時のオプション指定でも動くように残してあります。
(実際に弊社にそのようなページがかなりの量あります)

#2

Updated by Haru Iida over 7 years ago

互換性ということですね。
だとすると、wDとかwsDとかswDとかも対象になる気がするのですが、組合せはこれでよいのでしょうか?

#3

Updated by Haru Iida over 7 years ago

  • Assignee changed from Haru Iida to Tomohisa Kusukawa
#4

Updated by Haru Iida over 7 years ago

wは最後に付かなければいけないということか。

#5

Updated by Haru Iida over 7 years ago

確認なのですが、今の仕様ではsかdかwのどれか一つでよくて、swとかdwとかdwとか2文字以上を組み合わせることは無い、でよいですか?

#6

Updated by Tomohisa Kusukawa over 7 years ago

  • Assignee changed from Tomohisa Kusukawa to Haru Iida

wは最後に付かなければいけないということか。

はい。そのとおりです。

確認なのですが、今の仕様ではsかdかwのどれか一つでよくて、swとかdwとかdwとか2文字以上を組み合わせることは無い、でよいですか?

こちらもそのとおりです。

ちなみに
-s,-d,-wをそれぞれ複数回混ぜて使えて、
それらがAND条件となります。
当初はそこまでの機能を想定しておらず、
拡張する際にオプションの記述方法がそれまでの方法だとまずかったので
悩んだ末に互換動作をさせることにして思い切って仕様変更した次第です。

p.s.
コードレビュープラグインをちゃんと使ったこと無かったのですが、
非常に便利ですね。
うちはサーバ運用がメインなのであまり関係ない思っていましたが、
設定ファイル等をSubversion管理しているので
めちゃくちゃ使えると思います。

#7

Updated by Haru Iida over 7 years ago

  • Status changed from フィードバック(Reopend) to 終了(Closed)
  • Assignee changed from Haru Iida to Tomohisa Kusukawa
  • % Done changed from 0 to 100
#8

Updated by Haru Iida over 7 years ago

Tomohisa Kusukawa は書きました:

回答ありがとうございました。なんとかテストが書けそうです。

p.s.
コードレビュープラグインをちゃんと使ったこと無かったのですが、
非常に便利ですね。
うちはサーバ運用がメインなのであまり関係ない思っていましたが、
設定ファイル等をSubversion管理しているので
めちゃくちゃ使えると思います。

ありがとうございます。是非使って見てください。

#9

Updated by Tomohisa Kusukawa over 7 years ago

本来は私がテストを書くべきだと思うのですが、
不勉強で申し訳ありません。

せめて、書いて頂いたテストを
更新できるようには勉強していきたいと思っております。

いつも本当にありがとうございます。

Also available in: Atom PDF