[wp-trac] [WordPress Trac] #22486: PHP warning when creating a new site as part of a network

WordPress Trac noreply at wordpress.org
Sat Nov 17 06:32:01 UTC 2012


#22486: PHP warning when creating a new site as part of a network
--------------------------+--------------------
 Reporter:  jkudish       |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  high          |   Milestone:  3.5
Component:  Multisite     |     Version:  trunk
 Severity:  major         |  Resolution:
 Keywords:  has-patch     |
--------------------------+--------------------
Description changed by jkudish:

Old description:

> I get the following error when I create a new site as part of a network
> running an up to date trunk.
>
> {{{
> Notice: Undefined variable: s in .../wp/wp-admin/network/site-new.php on
> line 88
> }}}
>
> My setup is a bit particular but I am fairly positive it has nothing to
> do with the bug. Here it goes anyway:
>
> * locally hosted, using MAMP Pro, mapped to a .dev domain using my hosts
> file
> * subdomain setup for multisite
> * WordPress is in a subfolder on the domain, in a folder named 'wp'
> * The wp-content folder has been moved to '/content'
>
> The double quotes are causing the `$` to be executed as variables by the
> PHP parser instead of being interpreted as strings to replace by
> `sprintf`
> Patch attached that fixes the issue.

New description:

 I get the following error when I create a new site as part of a network
 running an up-to-date checkout of trunk.

 {{{
 Notice: Undefined variable: s in .../wp/wp-admin/network/site-new.php on
 line 88
 }}}

 My setup is a bit particular but I am fairly positive it has nothing to do
 with the bug. Here it goes anyway:

 * locally hosted, using MAMP Pro, mapped to a .dev domain using my hosts
 file
 * subdomain setup for multisite
 * WordPress is in a subfolder on the domain, in a folder named 'wp'
 * The wp-content folder has been moved to '/content'

 The double quotes are causing the `$` to be executed as variables by the
 PHP parser instead of being interpreted as strings to replace by `sprintf`
 Patch attached that fixes the issue.

--

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


More information about the wp-trac mailing list