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.current
をnilでクリアする修正を4.0のブランチにバックポートしています。
たまに落ちるテスト関連なのかな?
リビジョン 18433 - Merged r17761 from trunk to 3.4-stable (#32022). - Redmine
テスト実行時前に User.current
をnilでクリアする修正を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にしてもよかったんじゃないかなあ?
redmine-trunk-changes 2019-09-04
近況: ruby-jpのredmineチャンネルにコミットのRSSを流すようにしてみました。
リビジョン 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?
に直しています。よさそう。
redmine-trunk-changes 2019-09-02
リビジョン 18420 - Update raphael.js to 2.3.0 (#31436). - Redmine
raphael.jsのバージョンを2.1.0から2.3.0に上げています。
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