firebase / firebase/firebase-admin-java

[FR] Improvement: Change updateChildren parameter to Map<String, ?>

オープン
#917 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
needs-triage type: feature request
主要言語
Java
スター
620
フォーク
305
平均マージ
3時間 23分
マージ済み PR(30日)
1

説明

**Is your feature request related to a problem? Please describe.**
Various builders (e.g., `Map.of`) and language features (e.g., Groovy and Kotlin inline maps) create `Map` instances whose `V` bound is more strict than `Object`. Currently, the `DatabaseRef#updateChildren[Async]` interface won't accept these instances because it is defined as accepting `Map`.

**Describe the solution you'd like**
Since these method consume the map and do not modify it, they could be defined as `Map` to accept a map with any generic value type with no backwards incompatibility or other code changes.

**Describe alternatives you've considered**
Otherwise, passing in maps usually requires an explicit type declaration instead of `var` or an inline builder.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

DatabaseRef#updateChildren と updateChildrenAsync のエントリポイントを検索し、それらの宣言と関連するテストを確認します。メソッドが既存の更新動作を変更せずに Map を受け入れることを確認し、関連する Java テストスイートを実行します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
databases
issue の種類
機能追加
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。