[wp-trac] [WordPress Trac] #15706: Allow wildcarded domains in multisite limited email domains

WordPress Trac wp-trac at lists.automattic.com
Mon Dec 6 21:35:03 UTC 2010


#15706: Allow wildcarded domains in multisite limited email domains
-------------------------+--------------------------------------------------
 Reporter:  djcp         |       Owner:                     
     Type:  enhancement  |      Status:  new                
 Priority:  normal       |   Milestone:  Future Release     
Component:  Multisite    |     Version:                     
 Severity:  normal       |    Keywords:  has-patch 3.2-early
-------------------------+--------------------------------------------------
Changes (by nacin):

  * keywords:  has-patch => has-patch 3.2-early


Comment:

 Cool. (Not a problem, also. I killed the attachment.)

 Patch looks great, at a glance. All I see are coding standards
 improvements to be addressed. Marking for 3.2-early.

  - `foreach (` and `if (` rather than `foreach(` and `if(`. Same goes for
 opening braces, space before.
  - `substr( $limited_domain, 0, 2 ) == '*.'` should probably be `0 ===
 strpos( $limited_domain, '*.' )`.
  - The direct domain match can be made into an elseif, instead of the
 nested if.
  - Once a match is found, you should `break;` out of the foreach; no need
 to keep checking.
  - No need for the braces around the one-line if at the bottom.
  - `$matched_domain == false` should be `! $matched_domain`.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/15706#comment:6>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list