Code Review #988
closed
ref_issuesの引数に'D'がある？
100%
Description
ref_issuesのテストコードを書きたいと思っているのですが、コードで分からない部分があるので教えてください。
引数チェックで'D'をチェックしていますがwikiの説明を見るかぎりdはあってもDはないようです。このDは何を意味しているのでしょうか？
Updated by Tomohisa Kusukawa over 12 years ago
- Status changed from 新規（New） to フィードバック（Reopend）
- Assignee changed from Tomohisa Kusukawa to Haru Iida
以前のバージョンでは
wオプションで検索キーワードを指定する仕様で、
その機能指定として
S:サブジェクトを検索対象にしない
D:説明を検索対象にしない
をwの前に付けていました。
これら古い仕様のオプションで書かれた既存のWikiページを壊してしまわないように
当時のオプション指定でも動くように残してあります。
（実際に弊社にそのようなページがかなりの量あります）
Updated by Haru Iida over 12 years ago
互換性ということですね。
だとすると、wDとかwsDとかswDとかも対象になる気がするのですが、組合せはこれでよいのでしょうか？
Updated by Haru Iida over 12 years ago
- Assignee changed from Haru Iida to Tomohisa Kusukawa
Updated by Haru Iida over 12 years ago
確認なのですが、今の仕様ではsかdかwのどれか一つでよくて、swとかdwとかdwとか2文字以上を組み合わせることは無い、でよいですか？
Updated by Tomohisa Kusukawa over 12 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管理しているので
めちゃくちゃ使えると思います。
Updated by Haru Iida over 12 years ago
- Status changed from フィードバック（Reopend） to 終了（Closed）
- Assignee changed from Haru Iida to Tomohisa Kusukawa
- % Done changed from 0 to 100
Updated by Haru Iida over 12 years ago
Tomohisa Kusukawa は書きました:
回答ありがとうございました。なんとかテストが書けそうです。
p.s.
コードレビュープラグインをちゃんと使ったこと無かったのですが、
非常に便利ですね。
うちはサーバ運用がメインなのであまり関係ない思っていましたが、
設定ファイル等をSubversion管理しているので
めちゃくちゃ使えると思います。
ありがとうございます。是非使って見てください。
Updated by Tomohisa Kusukawa over 12 years ago
本来は私がテストを書くべきだと思うのですが、
不勉強で申し訳ありません。
せめて、書いて頂いたテストを
更新できるようには勉強していきたいと思っております。
いつも本当にありがとうございます。