WordPress.org
Get WordPress

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#41377 closed defect (bug) (fixed)

Add @access entries for $admin_header_callback and $admin_image_div_callback properties in Custom_Image_Header

Reported by: rushabh4486 Owned by: SergeyBiryukov
Priority: normal Milestone: 4.9
Component: Customize Version:
Severity: normal Keywords:
Cc: Focuses: docs

Description

Needs documents improvement in wp-admin/custom header.php

Attachments (1)

41377.patch​ (434 bytes ) - added by rushabh4486 9 years ago.
Needs documents improvement!

Download all attachments as: .zip

Change History (6)

@rushabh4486
9 years ago

Needs documents improvement!

#1 @rushabh4486
9 years ago

I have attached the patch.

#2 @SergeyBiryukov
9 years ago

  • Component Editor → Customize
  • Milestone Awaiting Review → 4.9

#3 @SergeyBiryukov
9 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 41100:

Docs: Add @access entries for $admin_header_callback and $admin_image_div_callback properties in Custom_Image_Header and Custom_Background.

Props rushabh4486.
Fixes #41377.

#4 @westonruter
9 years ago

Note that the @access is redundant in classes as of PHP 5 (where previously only var was supported). I'm supporting that such @access tags be removed from class members in the future. See more around ​https://wordpress.slack.com/archives/C02RP4WU5/p1495224872995687

The @access tag need only be used for global functions and global variables that don't have a way to specify their visibility in the language itself.

#5 @SergeyBiryukov
9 years ago

  • Summary Needs documents improvement! → Add @access entries for $admin_header_callback and $admin_image_div_callback properties in Custom_Image_Header
Note: See TracTickets for help on using tickets.