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 を使うように修正しています。

redmine-trunk-changes 2019-09-09

news関連のREST APIが追加されています。

リビジョン 18440 - REST API for creating news (#13468). Patch by Takenori TAKAKI. - Redmine

newsを作成するためのREST APIを追加しています。

  • POST /project/:project_id/news.json
  • POST /project/:project_id/news.xml

が使えるようになりました。

リビジョン 18441 - REST API for retrieving news (#13468). Patch by Takenori TAKAKI. - Redmine

newsに関する情報を個別に取得するためのREST APIが追加されました。

  • GET /news/:id.json
  • GET /news/:id.xml

リビジョン 18442 - REST API for deleting news (#13468). Patch by Takenori TAKAKI. - Redmine

newsを削除するAPIが実装されました。

  • DELETE /news/:id.json

  • DELETE /news/:id.xml

リビジョン 18443 - REST API for updating news (#13468). Patch by Takenori TAKAKI. - Redmine

newsを更新するAPIが実装されました

  • PUT /news/:id.json
  • PUT /news/:id.xml

リビジョン 18444 - Inline auto complete for issue (#) in wiki-edit fields (#31989). Patch by Ma... - Redmine

#のあとに文字列を入力するとチケットの入力候補が出てきて補完できる実装されました。べんりそう。

Tribute | tributeを使ってるみたいです。

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

redmine-trunk-changes 2019-09-04

近況: ruby-jpのredmineチャンネルにコミットのRSSを流すようにしてみました。

ruby-jp.github.io

リビジョン 18424 - Update Rouge to 3.10.0 (#31556). - Redmine

Rougeのバージョンを上げています。更新が盛んですね。

リビジョン 18425 - code cleanup: rubocop: fix Layout/SpaceBeforeFirstArg in test/unit/lib/redmin... - Redmine

メソッド呼び出しで最初の引数の前のスペースが2つだったのを1つにしています。

全部直すのかな

リビジョン 18426 - add newline at end of config/settings.yml - Redmine

settings.yml の末尾に改行を追加しています。

テキストファイルの末尾に改行が必要な話、最近同じ話題がOkinawa.rbで出ていたような、最近のPOSIXの仕様だとなくても大丈夫になったっぽいです。

POSIXの仕様では「テキストファイルの末尾は改行(newline)で終わる」のが正しい、を確認してみた。 - msfukuiの日記

リビジョン 18427 - Footer is not placed at the bottom on pages with little content (#30467). Pa... - Redmine

ページの内容が少なすぎる場合フッターが一番↓ではなく途中にでちゃっていたのを、一番したに表示するように修正しています。 bodyのサイズを最低でもビューポートの高さになるように調整して↓になるようにしたみたいですね。 その他flexの効いてるdivがmainから一段上の階層のwrapper3に移ってajax-indicator/ajax-modalの収まってる階層が変わった感じなのかな...(CSSの変更、動かしてみないとよくわからない)

redmine-trunk-changes 2019-09-03

リビジョン 18421 - iTraditional Chinese translation (#31997). Patch by ChunChang Lo. - Redmine

繁体字中国語の訳を更新しています。

リビジョン 18422 - Add missing fixtures to TimelogCustomFieldsVisibilityTest (#31999). Patch by... - Redmine

作業時間のカスタムフィールドのテストで指定が足りていなかったfixtureを追加しています。

リビジョン 18423 - Code cleanup: RuboCop: Fix Performance/InefficientHashSearch - Redmine

keys.include?key? に直しています。よさそう。

https://rubocop-performance.readthedocs.io/en/stable/cops_performance/#performanceinefficienthashsearch

redmine-trunk-changes 2019-08-31

コードきれいにした日っぽい

リビジョン 18417 - code cleanup: rubocop: fix Layout/LeadingCommentSpace in lib/redmine/scm/adap... - Redmine

不要なコメントだったのかコメントをバッサリ消してますね。

リビジョン 18418 - code cleanup lib/redmine/scm/adapters/cvs_adapter.rb - Redmine

=の位置を揃えています。

リビジョン 18419 - code cleanup: rubocop: fix Layout/LeadingCommentSpace in lib/redmine/scm/adap... - Redmine

コードの末尾に書いてあったコメントをコードの行の上に移動しています。

コメントの#の後ろに空白があるかどうかチェックしてくれるCopがあるんですね。 https://rubocop.readthedocs.io/en/stable/cops_layout/#layoutleadingcommentspace