#20576 closed defect (bug) (wontfix)
WP Scripts: Array key isset check
Reported by: | bigdawggi | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | General | Keywords: | has-patch |
Focuses: | Cc: |
Description
Small patch for making sure the array key is set, before doing logic against it. Other plugins' code was generating a PHP Notice here, but this cleans up WP code to not throw PHP notice if some other plugin does something poorly.
Attachments (1)
Change History (5)
#2
@
13 years ago
- Keywords reporter-feedback added
I would tend to agree. Hiding notices like these just makes it difficult for developers to debug their code. I think the notice is proper here.
Note: See
TracTickets for help on using
tickets.
If that array key doesn't exist there probably are much bigger problems than a PHP notice. Not sure if it's worth it removing the notice as it would indicate that something is misusing WP_Scripts.