Inline docs for wp-admin/includes/class-wp-upgrader.php

[jdgrimes]

Most of wp-admin/includes/class-wp-upgrader.php doesn't have inline docs, though hook docs have been added throughout.

[jdgrimes]

First run

[jdgrimes]

Initial patch, will probably need some work.

[DrewAPicture]

Hi J.D., sorry we didn't work our way around to your patch until just now. I'm seeing 2 out of 40 failures in applying your patch. Would you mind refreshing it?

patching file src/wp-admin/includes/class-wp-upgrader.php
Hunk #1 FAILED at 21.
Hunk #2 succeeded at 78 (offset 2 lines).
Hunk #3 FAILED at 118.
Hunk #4 succeeded at 177 (offset 4 lines).
Hunk #5 succeeded at 219 (offset 4 lines).
Hunk #6 succeeded at 268 (offset 4 lines).
Hunk #7 succeeded at 467 (offset 4 lines).
Hunk #8 succeeded at 588 (offset 4 lines).
Hunk #9 succeeded at 623 (offset 4 lines).
Hunk #10 succeeded at 655 (offset 4 lines).
Hunk #11 succeeded at 670 (offset 4 lines).
Hunk #12 succeeded at 719 (offset 4 lines).
Hunk #13 succeeded at 785 (offset 4 lines).
Hunk #14 succeeded at 914 (offset 4 lines).
Hunk #15 succeeded at 952 (offset 4 lines).
Hunk #16 succeeded at 976 (offset 4 lines).
Hunk #17 succeeded at 1002 (offset 4 lines).
Hunk #18 succeeded at 1048 (offset 4 lines).
Hunk #19 succeeded at 1080 (offset 4 lines).
Hunk #20 succeeded at 1105 (offset 4 lines).
Hunk #21 succeeded at 1171 (offset 4 lines).
Hunk #22 succeeded at 1236 (offset 4 lines).
Hunk #23 succeeded at 1301 (offset 4 lines).
Hunk #24 succeeded at 1418 (offset 5 lines).
Hunk #25 succeeded at 1458 (offset 5 lines).
Hunk #26 succeeded at 1482 (offset 5 lines).
Hunk #27 succeeded at 1512 (offset 5 lines).
Hunk #28 succeeded at 1539 (offset 5 lines).
Hunk #29 succeeded at 1575 with fuzz 1 (offset 5 lines).
Hunk #30 succeeded at 1647 with fuzz 1 (offset 36 lines).
Hunk #31 succeeded at 1662 with fuzz 2 (offset 36 lines).
Hunk #32 succeeded at 1686 (offset 43 lines).
Hunk #33 succeeded at 1808 (offset 47 lines).
Hunk #34 succeeded at 1841 (offset 47 lines).
Hunk #35 succeeded at 1881 (offset 47 lines).
Hunk #36 succeeded at 1897 (offset 47 lines).
Hunk #37 succeeded at 2061 (offset 48 lines).
Hunk #38 succeeded at 2168 (offset 48 lines).
Hunk #39 succeeded at 2203 (offset 48 lines).
Hunk #40 succeeded at 2290 (offset 48 lines).
2 out of 40 hunks FAILED

[jdgrimes]

Refreshed in 29086.2.diff​.

[DrewAPicture]

In 30758:

Add missing DocBlocks and parameter descriptions for a wide variety of properties and methods in WP_Upgrader, Plugin_Upgrader, Theme_Upgrader, Language_Pack_Upgrader, Core_Upgrader, and File_upload_Upgrader.

Props jdgrimes, DrewAPicture.

See #29086.

[DrewAPicture]

@jdgrimes: I committed all of your work up to 29086.2.diff​. There were still a few parameters and returns missing description that I'd like to get handled before we close this ticket out. Is that something you're up for?

[kraftbj]

Corrects typo in [30758] (s/plugins/themes)

[DrewAPicture]

In 33683:

Docs: Correct a typo in the summary for the Theme_Upgrader->bulk property.

Also adds a couple of missing @access tags.

Props kraftbj
See #29086.

[DrewAPicture]

In 33684:

Docs: Add missing @access tags to properties and methods in wp-admin/includes/class-wp-upgrader.

See #29086.

[DrewAPicture]

In 33685:

Docs: Fix a variety of inline documentation syntax issues and add some strategic spacing.

Fixes #29086.

[SergeyBiryukov]

In 41914:

Docs: Add @since entry for WP_Automatic_Updater::after_core_update(), introduced in [25841].

See #41017, #29086.