Remove pass-by-reference in admin-notices-ms.php
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 55/100
Research direction
Start by inspecting admin-notices-ms.php around lines 85 and 86, then search that file for other pass-by-reference usage. Remove the deprecated usage and verify that the admin notices still work under PHP 7 compatibility requirements.
Written by the indexing model from the issue text.
Description
As discussed in https://wordpress.org/support/topic/php-7-compatibility-229/, pass-by-reference is deprecated.
Affects lines 85, 86 and possibly elsewhere:
https://github.com/littlebizzy/force-https/blob/1.2.0/admin-notices-ms.php#L85
- Dominant language
- PHP
- Stars
- 26
- Forks
- 8
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
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.
More from littlebizzy/force-https
-
Difficulty 3/5 1-2 days Newbie friendliness 38/100
littlebizzy/force-https#11 · 3 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 30/100
littlebizzy/force-https#10 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 25/100
-
Disable nag notices Open
Difficulty 2/5 1-3 hours Newbie friendliness 52/100
littlebizzy/force-https#4 · 2 comments ·
All issues in littlebizzy/force-https
Similar issues
-
status/awaiting_triage
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
az-digital/az_quickstart#6019 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
grokability/snipe-it#19688 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100