![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 13 years ago
Closed 12 years ago
#26149 closed defect (bug) (fixed)
Hook Docs (13): wp-admin/includes/image-edit.php
| Reported by: | theorboman | Owned by: | kpdesign |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Media | Version: | |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: | docs |
Description
Documenting all undocumented filter hooks in wp-admin/includes/image-edit.php
Attachments (5)
Change History (15)
#2
@
13 years ago
- Milestone Awaiting Review → 3.8
Personally I rather have jpeg_quality description in /wp-includes/class-wp-image-editor.php. See #25721
@
13 years ago
Updated patch for wp-admin/includes/image-edit.php. Added periods after variable descriptions as per standards. Also note that 'admin_memory_limit' is documented in wp-includes/class-wp-image-editor-imagick.php (see https://core.trac.wordpress.org/ticket/26177)
#5
@
13 years ago
- Summary Hooks Docs: wp-admin/includes/image-edit.php → Hook Docs (13): wp-admin/includes/image-edit.php
At a glance, the docs here look fairly complete, though the spacing is a little wonky. Perhaps you were using tabs instead of spaces or something. I'll see if I can get you a more thorough review today.
#6
@
13 years ago
Here's another patch for this using spaces, not tabs for spacing out the docs. Let me know if this isn't the right way to do it!
#8
@
12 years ago
- Milestone Awaiting Review → 3.9
- Owner changed from to
26149.diff Should do it. Just needs a last review and recommendation.
#9
@
12 years ago
- Keywords commit added
26149.3.diff contains a few tweaks to 26149.diff.
This should be good to go. Recommend commit.
Patch for inline docs