Document jQuery.isEmptyObject doesn't take symbols into account
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 35/100
- issue の種類
- ドキュメント
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- jquery
調査の方向性
まず jQuery.isEmptyObject API のドキュメントを見つけ、現在の説明を issue に記載された動作と比較します。ドキュメントで Symbol プロパティとプロトタイプチェーンのプロパティについて説明すべきかを確認し、その後、該当するドキュメントを更新して、レンダリングされたエントリが合意した動作を反映していることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Description
On version 3.6.0, I found a vulnerability，I find that the method of detecting empty objects is not perfect.
isEmptyObject: function( obj ) {
var name;
for ( name in obj ) {
return false;
}
return true;
},
The downside of writing this is that the properties of the prototype chain and the Symbol type are not taken into account，I have a better way to implement it as follows.
isEmptyObject: function( obj ) {
var keys = Object.keys(obj);
// 看浏览器是否支持 Symbol
if (typeof Symbol !== "undefined") {
keys = keys.concat(Object.getOwnPropertySymbols(obj));
}
return keys.length === 0;
}
Link to test case
This direct modification of the source code test is good, I do not have a service to run this test. I'm looking forward to hearing from you.
- 主要言語
- HTML
- スター
- 324
- フォーク
- 259
- 平均マージ
- 1日 30分
- マージ済み PR（30日）
- 1
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
jquery/api.jquery.com のほかの issue
-
Mention of ‘timestamp’ in documentation for ‘cache’ parameter of ‘ajax’ is misleading対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 62/100
jquery/api.jquery.com#1241 · コメント 2 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 68/100
jquery/api.jquery.com#1176 ·
-
The documentation of CSS methods should be further classified再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
jquery/api.jquery.com#1281 · コメント 2 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
jquery/api.jquery.com#1271 · コメント 2 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
jquery/api.jquery.com#1259 ·
jquery/api.jquery.com の issue をすべて見る
似ている issue
-
5-minutes algorithm all-levels any-language beginner-friendly community documentation easy first-timers-only good first issue hacktoberfest help wanted learning no-code-needed open-source python students
難易度 2/5 1時間未満 初心者へのやさしさ 88/100
Huerte/algo-solutions-python#253 ·
メンテナーはふだん 1 日以内に返信
-
size/s
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
sipyourdrink-ltd/bernstein#7179 ·
メンテナーはふだん 1 日以内に返信
-
area/docs
難易度 1/5 1時間未満 初心者へのやさしさ 86/100
meshery/meshery#23375 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Synthetic
難易度 1/5 1時間未満 初心者へのやさしさ 66/100
samtoth/agda-synthetic-categories#373 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 62/100
tanaabased/big-test-bucket#19952 ·
メンテナーはふだん 1 日以内に返信