Scott Taylor
7de0b2b2c8
Upgrade `_WP_List_Table_Compat` to PHP5-style constructor.
...
Add `public` to methods/members of `WP_Role`.
Add `public` to methods/members of `WP_User` where appropriate. Don't set `private` where indicated until more study has occurred and tests have been written for compatibiliy with existing magic methods.
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28531
git-svn-id: http://core.svn.wordpress.org/trunk@28357 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 17:20:17 +00:00
Scott Taylor
7415ae10a3
`hackificator` doesn't like mixed single/double-quoted attributes. These were 2 lingering instances in the admin.
...
See #27881 .
Built from https://develop.svn.wordpress.org/trunk@28530
git-svn-id: http://core.svn.wordpress.org/trunk@28356 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 17:10:15 +00:00
Scott Taylor
09ecc213c8
In `WP_Filesystem_Base`, the constructor is a noop, so it shouldn't even be declared. Setting it implies that `parent::__construct()` should be called by its subclasses.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28529
git-svn-id: http://core.svn.wordpress.org/trunk@28355 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 16:46:14 +00:00
Scott Taylor
85f73cf458
Classes that have `__set()` also need `__isset()` and `__unset()`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28524
git-svn-id: http://core.svn.wordpress.org/trunk@28350 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 06:52:14 +00:00
Scott Taylor
821246b4ae
Some classes with `__get()` method also need `__set()`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28521
git-svn-id: http://core.svn.wordpress.org/trunk@28347 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 06:31:15 +00:00
Scott Taylor
dfa4de15fa
Add missing access modifiers to methods/members in `Walker` and subclasses. Add a magic `__get()` method.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28514
git-svn-id: http://core.svn.wordpress.org/trunk@28340 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 06:00:15 +00:00
Scott Taylor
2f513d3320
Fix some `hackificator` odds and ends in `wp-admin`:
...
* `wp-activate.php` and `wp-admin/themes.php` don't need the closing PHP tag
* Switch single quotes for HTML attribute values to double in a few places
* Convert `include_once file.php` syntax to `include_once( 'file.php' )`
* Add access modifiers to methods/members in: `_WP_List_Table_Compat`, `Walker_Nav_Menu_Edit`, `Walker_Nav_Menu_Checklist`, `WP_Screen`, `Walker_Category_Checklist`
* `edit_user()` doesn't need to import the `$wpdb` global
* `wp_list_widgets()` doesn't need to import the `$sidebars_widgets` global
* switch/endswitch syntax is not supported in Hack
* A `<ul>` in `wp-admin/users.php` is unclosed
See #27881 .
Built from https://develop.svn.wordpress.org/trunk@28500
git-svn-id: http://core.svn.wordpress.org/trunk@28326 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 05:04:16 +00:00
Drew Jaynes
511eabe5f3
Improve inline documentation of default arguments for `post_format_meta_box()`, `post_tags_meta_box()`, and `post_categories_meta_box()`.
...
See #28298 .
Built from https://develop.svn.wordpress.org/trunk@28499
git-svn-id: http://core.svn.wordpress.org/trunk@28325 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 03:23:13 +00:00
Scott Taylor
130771a383
`hackificator` doesn't like mixed quote styles in some generated HTML. The switch from single to double allows these files to be parsed.
...
See #27881 .
Built from https://develop.svn.wordpress.org/trunk@28497
git-svn-id: http://core.svn.wordpress.org/trunk@28323 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 01:59:15 +00:00
Scott Taylor
1383990ae2
Add access modifier (`public`) to methods and members of `WP_Upgrader` and its subclasses.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28496
git-svn-id: http://core.svn.wordpress.org/trunk@28322 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 01:29:15 +00:00
Scott Taylor
58ca85e106
Add access modifier (`public`) to methods and members of `WP_Upgrader_Skin` and its subclasses.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28495
git-svn-id: http://core.svn.wordpress.org/trunk@28321 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 01:23:15 +00:00
Scott Taylor
f090ff507d
Add access modifiers to methods and members of `WP_Users_List_Table`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28494
git-svn-id: http://core.svn.wordpress.org/trunk@28320 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 01:19:14 +00:00
Scott Taylor
d28f6344de
Add access modifiers to methods and members of list table classes:
...
* `WP_List_Table` is the base class that implements `__get()` and `__call()` for BC
* Adds unit tests to confirm that subclasses properly inherit magic methods
* Add modifiers to subclasses: `WP_Links_List_Table`, `WP_Media_List_Table`, `WP_MS_Sites_List_Table`, `WP_MS_Themes_List_Table`, `WP_MS_Users_List_Table`, `WP_Plugin_Install_List_Table`, `WP_Plugins_List_Table`, `WP_Posts_List_Table`, `WP_Terms_List_Table`, `WP_Theme_Install_List_Table`, `WP_Themes_List_Table`
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28493
git-svn-id: http://core.svn.wordpress.org/trunk@28319 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 01:17:15 +00:00
Scott Taylor
4e894f05f5
Add access modifier (`public`) to methods in `WP_Importer`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28492
git-svn-id: http://core.svn.wordpress.org/trunk@28318 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 00:19:15 +00:00
Scott Taylor
d5e35d187f
Add access modifier (`public`) to members and methods in `WP_Filesystem_SSH2`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28491
git-svn-id: http://core.svn.wordpress.org/trunk@28317 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 00:18:14 +00:00
Scott Taylor
41e1ecdfaf
Add access modifier (`public`) to members and methods in `WP_Filesystem_ftpsockets`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28490
git-svn-id: http://core.svn.wordpress.org/trunk@28316 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 00:14:14 +00:00
Scott Taylor
0dbc6b37a7
Add access modifier (`public`) to members and methods in `WP_Filesystem_FTPext`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28489
git-svn-id: http://core.svn.wordpress.org/trunk@28315 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 00:12:15 +00:00
Scott Taylor
eb750d0bd1
Add access modifier (`public`) to methods in `WP_Filesystem_Direct`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28488
git-svn-id: http://core.svn.wordpress.org/trunk@28314 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 00:09:14 +00:00
Scott Taylor
7a378bc48f
Add access modifiers to members and methods in `WP_Filesystem_Base`. Add magic `__get()` method for backwards compatibility.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28487
git-svn-id: http://core.svn.wordpress.org/trunk@28313 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-19 00:06:14 +00:00
Scott Taylor
e43e775c45
Add access modifier (`public`) to members and methods in `WP_Comments_List_Table` and `WP_Post_Comments_List_Table`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28486
git-svn-id: http://core.svn.wordpress.org/trunk@28312 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 23:59:15 +00:00
Scott Taylor
e917c14144
In `edit-link-form.php`, `hackificator` bails because there is a `</form>` with no open `<form>`. It exists, but is needlessly constructed with PHP. It always returns a `<form>`, only the `id` and `name` are different. The dynamic piece just returns the ID now.
...
See #27881 .
Built from https://develop.svn.wordpress.org/trunk@28485
git-svn-id: http://core.svn.wordpress.org/trunk@28311 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 22:07:15 +00:00
Scott Taylor
a659275338
`hackificator` bails on this file because of mixed quote styles on some HTML attributes.
...
See #27881 .
Built from https://develop.svn.wordpress.org/trunk@28484
git-svn-id: http://core.svn.wordpress.org/trunk@28310 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 21:47:14 +00:00
Scott Taylor
6a765cdd03
Remove public keyword from some JS functions. Sorry.
...
Props ocean90.
See #22234 .
Built from https://develop.svn.wordpress.org/trunk@28483
git-svn-id: http://core.svn.wordpress.org/trunk@28309 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 21:43:14 +00:00
Scott Taylor
68cfacf9ec
`hackificator` bails on this file because the `<meta>` isn't self-closing.
...
See #27881 .
Built from https://develop.svn.wordpress.org/trunk@28482
git-svn-id: http://core.svn.wordpress.org/trunk@28308 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 21:36:15 +00:00
Scott Taylor
2e912d3108
Use proper access modifiers and add a magic `__get()` method to `Custom_Background` and `Custom_Image_Header`.
...
See #27881 , #22234 .
Built from https://develop.svn.wordpress.org/trunk@28481
git-svn-id: http://core.svn.wordpress.org/trunk@28307 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 21:31:16 +00:00
Scott Taylor
b9afafffe3
`hackificator` complains if you call `include 'file.php'` without the parens, needs to be `include( 'file.php' )`
...
See #27881 .
Built from https://develop.svn.wordpress.org/trunk@28479
git-svn-id: http://core.svn.wordpress.org/trunk@28306 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 20:52:15 +00:00
Scott Taylor
112ca4e055
Because the `WP_ADMIN` constant name can be bound in multiple files, all instances should check `! defined` first. `wp-admin/admin.php` already has this check.
...
See #27881 .
Built from https://develop.svn.wordpress.org/trunk@28478
git-svn-id: http://core.svn.wordpress.org/trunk@28305 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 20:42:16 +00:00
Scott Taylor
fc23dc3551
The About page has a `<div>` that doesn't close. Also, `<hr>`s should self-close. `hackificator` complains about these things.
...
See #27881 .
Built from https://develop.svn.wordpress.org/trunk@28477
git-svn-id: http://core.svn.wordpress.org/trunk@28304 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 20:33:15 +00:00
Sergey Biryukov
8dacdb77be
Fix wp_insert_link(), broken in [28406]/[28408].
...
see #22400 .
Built from https://develop.svn.wordpress.org/trunk@28475
git-svn-id: http://core.svn.wordpress.org/trunk@28302 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 09:56:15 +00:00
Drew Jaynes
d75ca7aaaf
Add inline documentation for default arguments in `wp_insert_category()`.
...
See #28298 .
Built from https://develop.svn.wordpress.org/trunk@28474
git-svn-id: http://core.svn.wordpress.org/trunk@28301 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-18 07:28:14 +00:00
Sergey Biryukov
c453340a49
Update help text on Add Themes screen.
...
props thomasvanderbeek.
fixes #28132 .
Built from https://develop.svn.wordpress.org/trunk@28473
git-svn-id: http://core.svn.wordpress.org/trunk@28300 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-17 23:44:15 +00:00
Scott Taylor
4ce9706db1
Add inline comment after fix in [28467].
...
Props jesin.
Fixes #28292 .
Built from https://develop.svn.wordpress.org/trunk@28468
git-svn-id: http://core.svn.wordpress.org/trunk@28295 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-17 15:42:14 +00:00
Scott Taylor
bdd1ae2fc7
Restore `$soucrce` in `WP_Upgrader::install_package()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28467
git-svn-id: http://core.svn.wordpress.org/trunk@28294 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-17 15:40:15 +00:00
Scott Taylor
8eb9f2ffe4
Eliminate use of `extract()` in `request_filesystem_credentials()`.
...
The only property that doesn't need to be set to a variable is `$password`.
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28456
git-svn-id: http://core.svn.wordpress.org/trunk@28283 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-16 18:26:15 +00:00
Andrew Ozz
f184243aa3
DFW: fix closing only the wpLink or media modal when pressing Esc.
...
Built from https://develop.svn.wordpress.org/trunk@28455
git-svn-id: http://core.svn.wordpress.org/trunk@28282 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-16 18:17:14 +00:00
Scott Taylor
39de969bf2
Update inline docs for `wp_handle_upload|sideload` to reflect their non-use of `extract()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28452
git-svn-id: http://core.svn.wordpress.org/trunk@28279 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-16 17:41:16 +00:00
Scott Taylor
632eb5b17e
Eliminate use of `extract()` in `wp_handle_sideload()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28451
git-svn-id: http://core.svn.wordpress.org/trunk@28278 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-16 16:15:14 +00:00
Scott Taylor
4caf3e79f3
Eliminate use of `extract()` in `wp_handle_upload()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28450
git-svn-id: http://core.svn.wordpress.org/trunk@28277 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-16 16:10:38 +00:00
Sergey Biryukov
abc830ca9d
DFW: Properly handle Esc key when the content is focused or when switching browser tabs.
...
props avryl.
fixes #28279 .
Built from https://develop.svn.wordpress.org/trunk@28443
git-svn-id: http://core.svn.wordpress.org/trunk@28270 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-16 10:43:14 +00:00
Scott Taylor
b2d1e73fd5
Eliminate the use of `extract()` in `WP_Upgrader`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28418
git-svn-id: http://core.svn.wordpress.org/trunk@28245 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-15 05:02:14 +00:00
Scott Taylor
d74272ecf5
Eliminate one of the uses of `extract()` in `wp_handle_upload()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28417
git-svn-id: http://core.svn.wordpress.org/trunk@28244 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-15 04:31:14 +00:00
Scott Taylor
fc59804352
Eliminate one of the uses of `extract()` in `wp_handle_sideload()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28416
git-svn-id: http://core.svn.wordpress.org/trunk@28243 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-15 04:29:15 +00:00
Scott Taylor
a7a6cb5bef
Eliminate use of `extract()` in `Walker_Category_Checklist::start_el()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28411
git-svn-id: http://core.svn.wordpress.org/trunk@28238 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-15 03:14:14 +00:00
Scott Taylor
7ebab48f4c
Eliminate use of `extract()` in `wp_terms_checklist()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28410
git-svn-id: http://core.svn.wordpress.org/trunk@28237 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-15 03:09:15 +00:00
Scott Taylor
a75f8137bc
Eliminate use of `extract()` in `wp_star_rating()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28409
git-svn-id: http://core.svn.wordpress.org/trunk@28236 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-15 02:57:14 +00:00
Scott Taylor
ff665b2f38
(REALLY) Eliminate use of `extract()` in `wp_insert_link()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28408
git-svn-id: http://core.svn.wordpress.org/trunk@28235 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-15 02:55:16 +00:00
Scott Taylor
54321f4c6e
Eliminate use of `extract()` in `get_media_item()`.
...
To test, fire open the old media in the console: `tb_show('Old Media is Weird', 'media-upload.php?type=image&TB_iframe=true&post_id=0');`
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28407
git-svn-id: http://core.svn.wordpress.org/trunk@28234 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-15 02:45:15 +00:00
Scott Taylor
5f06d0f50a
Eliminate use of `extract()` in `wp_insert_link()`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28406
git-svn-id: http://core.svn.wordpress.org/trunk@28233 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-15 02:28:14 +00:00
Scott Taylor
1121efb54c
Eliminate use of `extract()` in `display_setup_form()`. Only needs to read `password` and `password_message`.
...
See #22400 .
Built from https://develop.svn.wordpress.org/trunk@28396
git-svn-id: http://core.svn.wordpress.org/trunk@28224 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-14 16:43:15 +00:00
Drew Jaynes
a23cc5634f
Properly split and document the `admin_post*` actions into the following four hooks:
...
* `admin_post_nopriv` – for logged-out requests lacking a supplied action
* `admin_post_nopriv_$action – for logged-out requests with a supplied action
* `admin_post` – for logged-in requests lacking a supplied action
* `admin_post_$action` – for logged-in requests with a supplied action
See [28349], [28350], [28351].
See #26869 .
Built from https://develop.svn.wordpress.org/trunk@28394
git-svn-id: http://core.svn.wordpress.org/trunk@28222 1a063a9b-81f0-0310-95a4-ce76da25c4cd
2014-05-13 07:24:15 +00:00