[wp-trac] [WordPress Trac] #30891: Unchecked property overloading is detrimental to OOP.

WordPress Trac noreply at wordpress.org
Sat Jan 10 22:50:15 UTC 2015


#30891: Unchecked property overloading is detrimental to OOP.
--------------------------+------------------
 Reporter:  aercolino     |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  4.2
Component:  General       |     Version:  4.0
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:
--------------------------+------------------

Comment (by jdgrimes):

 Replying to [comment:12 wonderboymusic]:
 > I made them `private` in [28481], that was unnecessary... and every bit
 of this class was public previously. Do you have a use case for these
 classes outside of `_custom_header_background_just_in_time()`?

 No, I don't have a use case. I just wanted to clarify that I don't think
 they have to be made public. If the goal is to remove the magic methods
 where they aren't absolutely needed, then this makes sense.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/30891#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list