金具✅

45391 posts 614 follows 441 followers

アイコンは @akkiesoft 氏作

金具✅ cobodo@mstdn.kanagu.info

他のパターンで、 core/plugin/gui/tablike.rb:81, core/plugin/gui/window.rb:39, core/plugin/shortcutkey/shortcutkey_listview.rb:35 の3箇所で、 Diva::Model(:photo)[icon] という、Plugin::Photo::Photoがself.[](uri)を実装していることを前提にしたコードがある（guiもshortcutkeyも明示的にはphotoに依存していない）

金具✅ cobodo@mstdn.kanagu.info

core/plugin/twitter/twitter.rb:408でもDiva::Model(:photo).generateとかやってるな

金具✅ cobodo@mstdn.kanagu.info

これ、Diva::Model(:photo).nil?なら@iconにnilを入れとけば済む話のような気がするな……

金具✅ cobodo@mstdn.kanagu.info

Plugin::Twitter::User#icon の場合、明示的にDiva::Model(:photo)と書いて、しかもnilチェックがないのが問題か。

金具✅ cobodo@mstdn.kanagu.info

photo_filterの結果をちゃんと個数チェックして、無ければ無いでどうにかするようになっていれば別に依存する必要はないけど……

金具✅ cobodo@mstdn.kanagu.info

そういう意味ではphoto_filterを使ってるプラグインは全部photoに依存させてもいいのではないか

金具✅ cobodo@mstdn.kanagu.info

callは非同期的に各イベントハンドラに通知するだけだけど、filteringは同期的、かつ返り値をその場で使うから……

金具✅ cobodo@mstdn.kanagu.info

Plugin.call は依存関係に書くほどでもないけどPlugin.filteringはもう明示しといたほうがいいのでは感ある

金具✅ cobodo@mstdn.kanagu.info

@ahiru なるほど

金具✅ cobodo@mstdn.kanagu.info

@ahiru いやWorldonはサードパーティプラグインなので

金具✅ cobodo@mstdn.kanagu.info

defimageopenerのためにopenimgプラグインに依存していて、openimgがphotoプラグインに依存しているので、間接的な依存関係はあるものの……

金具✅ cobodo@mstdn.kanagu.info

photoプラグイン、Worldonも依存してるけど書いてないなーと思った。

金具✅ cobodo@mstdn.kanagu.info

バグ #1302: 依存関係の足りないプラグインがある - mikutter - やること
https://dev.mikutter.hachune.net/issues/1302

金具✅ cobodo@mstdn.kanagu.info

FGOのオタクなので「おしゅてんみたい」と読んでしまう