バグ #1302
closed
依存関係の足りないプラグインがある
Description
- intent プラグイン
intent プラグインは uitranslator に依存しているが書かれていないため、下記エラーで落ちる1: from /home/mikutter/mikutter/core/plugin/intent/intent.rb:63:in `block in <top (required)>' /usr/local/bundle/gems/pluggaloid-1.1.1/lib/pluggaloid/plugin.rb:257:in `method_missing': undefined method `_' for #<Plugin:0x00005650b8984bf8> (NoMethodError) - twitter プラグイン
Photo プラグインに依存しているが、依存関係に書かれていないため、下記エラーで落ちる1: from /home/mikutter/mikutter/core/plugin/twitter/model/world.rb:53:in `to_hash' /home/mikutter/mikutter/core/plugin/twitter/model/user.rb:84:in `icon': undefined method `generate' for nil:NilClass (NoMethodError)
それぞれ mikutter.yml を修正すれば問題ないですが、これはそれぞれチケット分けた方がいいですかね（それがわかればブランチにコミットしてプッシュします）
Updated by cob odo about 6 years ago
twitterプラグインは、
https://dev.mikutter.hachune.net/projects/mikutter/repository/revisions/bc54fdf5cb054567316af95e53ce188a2d9d12b8/entry/core/plugin/twitter/twitter.rb#L408
にもPhotoモデルに依存したコードがありますね（添付画像の処理）。
他に
https://dev.mikutter.hachune.net/projects/mikutter/repository/revisions/bc54fdf5cb054567316af95e53ce188a2d9d12b8/entry/core/plugin/gui/tablike.rb#L81
https://dev.mikutter.hachune.net/projects/mikutter/repository/revisions/bc54fdf5cb054567316af95e53ce188a2d9d12b8/entry/core/plugin/gui/window.rb#L39
https://dev.mikutter.hachune.net/projects/mikutter/repository/revisions/bc54fdf5cb054567316af95e53ce188a2d9d12b8/entry/core/plugin/shortcutkey/shortcutkey_listview.rb#L35
の3箇所に、Plugin::Photo::Photo.[]メソッドの存在を前提にしたコードがありますね。guiプラグイン、shortcutkeyプラグインの2つとも、photoプラグインに依存していません。
Updated by cob odo about 6 years ago
photo_filterの使用箇所を調べた限りでは、必ずnilチェックをしていたようなので、こちらは依存関係に書いていなくても問題ない（無ければ無いでよいように書いてある）ように思います。
Updated by あひる 家鴨 about 6 years ago
- Status changed from 分類待ち to toshi_aの判断待ち
ひとまず自分の指摘した二箇所に関してコミットを分けてブランチにプッシュしてみました。
topic/1302-tarinai-dependencies ブランチです。
Updated by cob odo about 6 years ago
twitterプラグインに関しては、単に.generate(...)を&.generate(...)にして、nilだったらnilのままにしておくという手もあるかと思います。特に画像を扱う必要のない環境でも、プラグイン自体は同じように使えたほうが便利そうです。
Updated by あひる 家鴨 about 6 years ago
それもそうですね。
.generate(...) に関しては &.generate(...) にします。
その場合、photo は gui 系のプラグインが依存していればいいんですかね。
Updated by cob odo about 6 years ago
- GUI必要＆画像必要：通常の環境
- GUI必要＆画像不要：？
- GUI不要＆画像必要：画像DL botのような運用？
- GUI不要＆画像不要：あひる焼くな等のテキストベースbot？
2は必要性が謎ですね……あるとしたら極度に回線が細い環境でしょうか。例えば長期にわたって海上にいる漁船やタンカー、あるいは原子力潜水艦等で、衛星回線経由で見ている、とか？
guiやshortcutkeyはphotoに依存してしまってもいいような気がしますね。
Updated by toshi_a 初音 about 6 years ago
- Status changed from toshi_aの判断待ち to 実装待ち
twitterプラグインに関しては、単に.generate(...)を&.generate(...)にして、nilだったらnilのままにしておくという手もあるかと思います。特に画像を扱う必要のない環境でも、プラグイン自体は同じように使えたほうが便利そうです。
そう思います。TwitterプラグインがPhotoに依存していないのは意図的なものです。 cc640064 は必要ないと思います。
それもそうですね。
.generate(...) に関しては &.generate(...) にします。
その場合、photo は gui 系のプラグインが依存していればいいんですかね。
guiもphotoに依存する必要はありません。GUIはPhoto Modelを利用するだけで生成しないからです。
ただ、 source:core/plugin/gui/tablike.rb@2d0864a0#L81 はPhotoがないとクラッシュするように見えるので、修正が必要ですね。
shortcutkeyもphotoとは関係がないので依存すべきではないですが、現状のユースケースでは考えなくて良さそうです。
Updated by toshi_a 初音 about 6 years ago
guiプラグインの修正はコミットしました（ 3947035e ）。
twitterプラグインはPhoto Variantを使っているため Plugin::Photo::Photo.generate を使ってしまってますね。どうしたものか
Updated by toshi_a 初音 about 6 years ago
- Status changed from 実装待ち to レビュー待ち
- Assignee set to あひる 家鴨
- ブランチ set to topic/1302-tarinai-dependencies
多分動くと思うからpushしました！
Updated by あひる 家鴨 about 6 years ago
- Status changed from レビュー待ち to マージ待ち
最新のブランチであひる焼くながdockerコンテナで多分動いたと思うからマージしようぜ。
Updated by toshi_a 初音 about 6 years ago
merged.
@ahiru develop基準になっていたためcherry-pickしました。masterにpushしたぶんでうまく動いてるか確認して大丈夫そうだったらcloseしてください（雑）