プロジェクト

全般

プロフィール

バグ #966

settingsのaboutのnameが使われていない

あひる 家鴨10ヶ月前に追加. 9ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
開始日:
2017-01-09
プラグイン名:
core/plugin/settings/basic_settings.rb

説明

settingsabout のインスタンス生成時のオプションに :name を指定していますが、この値は使われておらずアプリケーション名( "mikutter.rb" )が常に渡されてしまっています。( before.png を参照してください )
正しくは、aboutdialog.rb#L35 のように program_name を指定する必要があるようです。
program_name を指定した結果 after.png のように ENVIRONMENT::NAME で指定している "mikutter" が使用されているのを確認しました。
また、 :name を指定してアプリケーション名が表示されていることに関しては、 Gtk::AboutDialog の仕様のようです。(AboutDialog#program_name

修正パッチを添付します。( about.patch

after.png (77.2 KB) after.png 修正後 あひる 家鴨, 2017-01-09 16:22
before.png (80.3 KB) before.png 修正前 あひる 家鴨, 2017-01-09 16:22
about.patch (1.65 KB) about.patch あひる 家鴨, 2017-01-09 16:29

関係しているリビジョン

リビジョン 90344503 (差分)
toshi_a 初音9ヶ月前に追加

setting DSLのaboutメソッドのname属性が使われていない refs #966

コメントの通り、アプリケーション名として使われるようにした

履歴

#1 あひる 家鴨10ヶ月前に更新

  • 説明 を更新 (diff)

#2 あひる 家鴨9ヶ月前に更新

  • ステータス新規 から パッチ適用待 に変更

#3 toshi_a 初音9ヶ月前に更新

  • ステータスパッチ適用待 から レビュー待ち に変更
  • 担当者あひる 家鴨 にセット

このパッチはaboutの仕様を変更してますが、それだと今までとは違う形式でパラメータを渡さなければならなくなるため、非互換な変更となってしまいます。
name属性に与えられた値をアプリケーション名として利用するようにしましたが、どうでしょう。

#4 あひる 家鴨9ヶ月前に更新

  • ステータスレビュー待ち から 解決 に変更

確認しました
いいと思います。
確かに互換性を破壊してしまう変更でした。

#5 あひる 家鴨9ヶ月前に更新

  • 担当者あひる 家鴨 から toshi_a 初音 に変更

#6 toshi_a 初音9ヶ月前に更新

  • ステータス解決 から 終了 に変更

他の形式にエクスポート: Atom PDF