redmine-trunk-changes 2019-09-11

リビジョン 18445 - Fix that Redmine::ApiTest::NewsTest fails when config.time_zone is set (#1346... - Redmine

config.time_zone が設定されていると落ちるテストを直しています。

UTCの時刻をiso8601の書式で文字列化した場合にマッチする正規表現でテキストを探していたのを、実際にiso8601メソッドを読んで文字列化した値を探すように直しています。

time_zone を設定したbuildを1つ生やすといいのかな、という気持ちになりますね。

リビジョン 18446 - Don't use ^= operator to target the wrapper elements (#30467). Patch by Mari... - Redmine

CSSのセレクターでwrapperから始まる要素に対してスタイルが指定されていたのを、きちんと完全一致するセレクタに直しています。 wrapperは一般的すぎるので使っているテーマや導入しているプラグインによっては意図しない要素にスタイルが聞いてしまう可能性がありそうですね。

リビジョン 18447 - code cleanup: rubocop: fix Layout/SpaceInsideReferenceBrackets in lib/redmine... - Redmine

[] で参照するときにブラケットの内側にスペースが空いていたのを、消しています。

リビジョン 18448 - code cleanup: rubocop: fix Layout/SpaceInsideReferenceBrackets in test/unit/l... - Redmine

これも上記と同じくスペースを削除しています。こういう感じで合わせるスタイルとこのCopの相性わるそう

a[ 0]
a[-1]

リビジョン 18449 - Issue auto complete should return last 10 issues (#31994). Patch by Marius B... - Redmine

チケットのオートコンプリートで検索する文字列がない場合に絞り込みがうまく効いてなかったのを効くようにしてるのかな?

あと最大10件しか取得しないようにしています。

リビジョン 18450 - Remove 'app/views/auto_completes/issues.html.erb' which is not used (#31994).... - Redmine

使っていない不要なテンプレートを削除しています。

リビジョン 18451 - Code cleanup: RuboCop: Fix Performance/RedundantMatch and Rails/DynamicFindBy... - Redmine

matchしているけど返り値のMatchDataを使っていないので、=~ でマッチして $1 を使うように修正しています。