Opened 8 years ago
Closed 3 years ago
#39048 closed defect (bug) (invalid)
undefined return array var being passed to wp_parse_str
Reported by: | pbearne | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | General | Keywords: | |
Focuses: | Cc: |
Description
I spotted that the array being passed by ref is not be set in all places in core
This patch fixes that
#WCUS contrib day
Attachments (3)
Change History (6)
#1
@
8 years ago
@pbearne Thanks for the patch. I'm not certain I understand what issue this is resolving. Does not setting these arrays beforehand cause a PHP warning or error? A unit test demonstrating that would be immensely helpful here.
Note that generally code refactoring should not be done just because we can.
#3
@
3 years ago
- Milestone Awaiting Review deleted
- Resolution set to invalid
- Status changed from new to closed
Hi there, thanks for the patch!
Unless I'm missing something, for variables passed by reference to there is no need to create them in advance.
Per the PHP manual:
If you assign, pass, or return an undefined variable by reference, it will get created.
This is similar to the $matches
parameter of preg_match()
, we don't generally define that in advance either.
fixed patch remove other changes