redmine-trunk-changes 2019-09-05

リビジョン 18428 - Broken JavaScript icon in the repository view (#32012). Patch by Antonio McD... - Redmine

リポジトリディレクトリの内容を表示している際にJavaScriptファイルの専用アイコンが表示されていなかったみたいです。

mimeタイプをもとにCSSのクラス生成するヘルパーがあってMimeTypeの返す値が変わったのが原因だったよう。

Redmine::MimeType.css_class_of('foo.gz')
# => "application-gzip"

リビジョン 18429 - Merged r18428 from trunk to 4.0-stable (#32012). - Redmine

JSのアイコン表示修正を4.0にバックポートしています。

リビジョン 18430 - Merged r18428 from trunk to 3.4-stable (#32012). - Redmine

JSのアイコン表示修正を3.4にバックポートしています。

リビジョン 18431 - Add inherit_members to projects API response (#32002). Patch by Yuichi HARADA. - Redmine

プロジェクトのAPIのレスポンスに項目を追加しています。

属性は増えたんだけどAPIのレスポンスに追加はされていない、みたいなのが他にもありそうな雰囲気

リビジョン 18432 - Merged r17761 from trunk to 4.0-stable (#32022). - Redmine

テスト実行時前に User.currentnilでクリアする修正を4.0のブランチにバックポートしています。

たまに落ちるテスト関連なのかな?

リビジョン 18433 - Merged r17761 from trunk to 3.4-stable (#32022). - Redmine

テスト実行時前に User.currentnilでクリアする修正を3.44のブランチにバックポートしています。

リビジョン 18434 - Fix that test_mail_from_format_should_be_validated randomly fails (#31154). ... - Redmine

メールの設定のテストがたまに落ちるのを修正しています。

Settings.default_language を英語にしていたけど、正しくテストするためには実際は I18n.locale の変更が必要だったよう。他のテストでI18n.localeがフランス語に設定されたあとこのテストを実行すると落ちていたようです。with_localeヘルパー便利そう。

リビジョン 18435 - Broken gzip icon in the repository view (#32024). Patch by Seiei Miyagi. - Redmine

JSのアイコン表示修正を見ていると、他にも特定のファイル形式向けのCSSが並んでいるのが見えたので、1つぐらいは壊れているだろうと思って確認したらgzipのファイルが壊れていそうだったのでパッチを作って投げました。

リビジョン 18436 - Merged r18435 from trunk to 4.0-stable (#32024). - Redmine

gzのアイコン表示修正を4.0向けにバックポートしています。

リビジョン 18437 - Merged r18435 from trunk to 3.4-stable (#32024). - Redmine

gzのアイコン表示修正を3.4向けにバックポートしています。

リビジョン 18438 - Add missing fixture to IssueStatusesControllerTest (#32023). Patch by Yuichi... - Redmine

テストに足りていないfixtureを追加しています。

リビジョン 18439 - Remove unnecessary vertical scroll bar from gantt (#32019, #27672). Patch by... - Redmine

ガントチャートの縦スクロールのバーを消しています。 チケット多すぎたときにスクロールできるようにする配慮だけど出ない人のほうが多そうですしね。

うーん、これブラウザのデフォルトがvisibleなんだけどautoにすると不要な場合はでなくなるし、autoにしてもよかったんじゃないかなあ?

overflow - CSS: カスケーディングスタイルシート | MDN