redmine-trunk-changes 2019-08-04

rubocopの設定の修正やリファクタリングがありました。

rubocop周り雑に--safe-auto-correctかけてしまってよいのでは、という気がしなくもない。 特定のCopだけかけるなら--onlyで指定できるし。

パッチなげてみようかな

リビジョン 18342 - rubocop: use Layout/TrailingWhitespace AllowInHeredoc true - Redmine

heredoc中の末尾の空白をありに設定変更しています。 これ1ファイルしかないのでそのままtodo.ymlの内容をrubocopに移動する、でもよさそうなような?(あんまりheredoc中で末尾に空白書きたくなることなさそうなので)

リビジョン 18343 - rubocop: fix Layout/BlockAlignment of test/unit/lib/redmine/scm/adapters/merc... - Redmine

ブロックを開始する行のインデントを直しています。

リビジョン 18344 - code clean up test/unit/lib/redmine/scm/adapters/mercurial_adapter_test.rb - Redmine

ブランチ名の変数がbraと省略されていたのでbranchにリネームしています。

リビジョン 18345 - rubocop: fix Layout/CommentIndentation of test/unit/repository_cvs_test.rb - Redmine

コメントのインデント位置を修正しています。

リビジョン 18346 - Previous and next month links in gantt (#31373). Patch by Mizuki ISHIKAWA. - Redmine

ガントチャートに前の月、次の月へのリンクを追加しています。 ApplicationHelperの方にリンクの生成メソッドが移動しています。 これってアプリケーション全体で使えるのかな? あとで使い方見てみよう

リビジョン 18347 - rubocop: remove app/helpers/calendars_helper.rb from Style/UnneededInterpolat... - Redmine

rubocopでExcludeしていたファイルがすでに直っていたのでtodoから消しています。

リビジョン 18348 - rubocop: fix Layout/DefEndAlignment of test/unit/repository_cvs_test.rb - Redmine

def endのendのインデントがずれてたのを直しています。 endの前のアサーションがそもそもインデントずれていますね

リビジョン 18349 - rubocop: fix Layout/IndentationConsistency of test/unit/repository_cvs_test.rb - Redmine

そのendの前のアサーションのインデントを直しています。

リビジョン 18350 - rubocop: fix Layout/DefEndAlignment of test/functional/documents_controller_t... - Redmine

endのインデントずれを直しています。

全ファイルでLayout/DefEndAlignmentが効くようになったみたいです👏

リビジョン 18351 - rubocop: fix Style/YodaCondition of lib/redmine/scm/adapters/abstract_adapter.rb - Redmine

yodaコンディション(定数的なものを左におく)を直しています

これself.いらなさそうだな