redmine-trunk-changes 2019-09-18

リビジョン 18476 - Change textfield to textarea for "Exclude attachments by name" (#19903). Pat... - Redmine

メールの添付ファイルを弾く設定の入力欄が1行で使いづらかったので、複数行のtextareaにしてます。

リビジョン 18477 - code cleanup: rubocop: fix Lint/UselessSetterCall in test/unit/issue_test.rb - Redmine

rubocopのLint/UselessSetterCallに対応しているようです。

属性を代入したときに、期待している順番通り代入されることを確認するテストで、最後に代入していたのをLint/UselessSetterCallに怒られてなおしているけど、このCop自体に若干のモヤりがあります。

redmine-trunk-changes 2019-09-15

リビジョン 18471 - Constrain sidebar width on different resolutions (#32037). Patch by Antonio ... - Redmine

サイドバーの幅を画面の解像度に合わせて増減するようにしています。

リビジョン 18472 - code cleanup: rubocop: fix Layout/IndentAssignment in app/helpers/repositorie... - Redmine

代入のインデントを修正しています。

リビジョン 18473 - code cleanup: rubocop: fix Layout/LeadingCommentSpace in app/controllers/repo... - Redmine

コメントの # とコメントの間に空白を追加しています。

redmine-trunk-changes 2019-09-14

リビジョン 18468 - Add clear query icon next to selected query in sidebar (#31640). Patch by Ma... - Redmine

カスタムクエリをクリアするためのアイコンを、サイドバーの選択中のカスタムクエリの横に表示しています。 スマホでRedmineみたことがあまりないんだけど、スマホだとどういう見た目になるんだろう?

リビジョン 18469 - code cleanup: rubocop: fix Layout/EmptyLinesAroundMethodBody in test/function... - Redmine

メソッド末尾の意味のない空行を削除しています。

リビジョン 18470 - code cleanup: rubocop: fix Layout/IndentationConsistency in test/functional/r... - Redmine

インデントを直しています

redmine-trunk-changes 2019-09-13

リビジョン 18460 - Fix that date grouping for MySQL is not working (#32046, #13803). Patch by M... - Redmine

MySQLで日付をグループ化するSQLがうまく動いていなかったのを修正しています。

カラムを日付に変更してからタイムゾーンを変換していたのを、カラムのタイムゾーンを変えてから日付に変更するようにしています。 た、たしかに〜〜!!!

リビジョン 18461 - rubocop: disable Naming/PredicateName - Redmine

真偽値を返すようなメソッドの名前で is_ とか has_ とかついているとおこるCopを無効にしています。

リビジョン 18462 - code cleanup: rubocop: fix Style/MethodCallWithoutArgsParentheses in test/fun... - Redmine

メソッド呼び出しの不要な()を消しています。

リビジョン 18463 - code clean up test/functional/repositories_git_controller_test.rb - Redmine

外部エンコーディングがUTF8かどうか調べた結果を入れているインスタンス変数や、メソッドの名前にruby19_prefixがついてたので、消しています。R.I.P.

リビジョン 18464 - code clean up test/functional/repositories_mercurial_controller_test.rb - Redmine

外部エンコーディングがUTF8かしらべた結果を一時変数に入れていたのですが、つかっている場所が1箇所なので、直接書くようにしてローカル変数を消しています。

リビジョン 18465 - mail_body method in test/test_helper.rb raises an exception if the message is... - Redmine

テストヘルパーの中でメールオブジェクトを渡すとbodyの文字列を返すmail_bodyメソッドがあるんですが、multipartじゃないメールを渡すとエラーになっていたので、multipartかどうかで処理をわけてエラーにならないよう修正しています。

リビジョン 18466 - code cleanup: rubocop: fix Layout/IndentationConsistency in test/functional/r... - Redmine

インデントを直しています

リビジョン 18467 - Merged r18377 from trunk to 4.0-stable (#31870). - Redmine

jQuery関係のdeprecatedの修正を4.0にバックポートしています。

redmine-trunk-changes 2019-09-12

リビジョン 18452 - Fix the test name different from the actual behavior (#31994). Patch by Mari... - Redmine

テストのメソッド名が実際の動作と違う説明をしていたので、修正しています。

リビジョン 18453 - code cleanup: scm: cvs: remove unused branchPoint method from CvsRevisionHelp... - Redmine

使っていないメソッドを削除しています。

リビジョン 18454 - Update default database config for MySQL to support 4 byte characters (emoji)... - Redmine

config/database.yml のサンプルファイルで、MySQLのエンコーディングをutf8からutf8mb4に変更しています!!!

これでconfig/database.ymlの設定をコピーしてデータベースを作ったとき、絵文字が使えるエンコーディングでDBが作成されますね。

絵文字を書いても500にならなくなります! やったね!

リビジョン 18455 - Add missing fixture (#13468). - Redmine

テストにfixtureの指定が足りていないので、fixtureを追記しています。

リビジョン 18456 - code cleanup: rubocop: fix Lint/AmbiguousOperator in lib/redmine/scm/adapters... - Redmine

hg *hg_args doをhg(*hg_args) do に修正しています。*がsplatなのか乗算的な*なのか区別がつきづらいみたいな理由っぽいですね。なるほど。

リビジョン 18457 - rubocop: use Lint/HandleExceptions AllowComments true - Redmine

Rubocopのデフォルトだと例外を握りつぶすとダメで何かしらのエラーハンドリングをしないといけない感じになっています。 この変更でオプションを設定して、コメント書いてあれば例外握りつぶしても大丈夫みたいな感じのルールに変更しています。

リビジョン 18458 - code cleanup: rubocop: fix Lint/HandleExceptions in lib/redmine/scm/adapters/... - Redmine

例外を握りつぶしている箇所にエラーハンドリングで特に何もしていない旨のコメントを追記しています。

リビジョン 18459 - Add additional mail headers for issue tracker (#31910). Patch by Go MAEDA. - Redmine

メールのヘッダーにチケットのトラッカーも含めて送るようにしています。

特定のトラッカーやプライオリティが設定されている場合に、メールクライアントの方でうまく振り分けしたいみたいな動機があるようです。なるほどなー。