nextcloud / nextcloud/cookbook
Usage of PHP constructor property promotion
Open
Nobody has claimed this yet.
Backend
good first issue
maintenance
- Dominant language
- HTML
- Stars
- 641
- Forks
- 113
- Avg merge
- 21h 31m
- Merged PRs (30d)
- 26
Description
With PHP 8, property promotion was introduced. This should be used to simplify the code (also DRY).
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
No files or tests are named. Start by locating the PHP constructors in the cookbook code and read the linked PHP 8 constructor property promotion documentation. The work is done when applicable constructors use promotion without changing behavior and the existing test suite passes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100