JSON: Replace `value.hasOwnProperty` with `Object.prototype.hasOwnProperty.call`
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- JavaScript
- Stars
- 48
- Forks
- 14
- Avg merge
- 4h 14m
- Merged PRs (30d)
- 1
Description
Replace value.hasOwnProperty with Object.prototype.hasOwnProperty.call in the JSON module
This makes it harder for someone to use hasOwnProperty for something other than it's intended purpose.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Open src/Gren/Kernel/Json.js at line 234 and inspect the existing hasOwnProperty usage in the JSON module. Replace that call as described in the issue, then verify the JSON module's existing tests or checks pass and that the direct property call is no longer used there.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100