WordPress.org

Ready to get started?Download WordPress

Forums

WordPress MU Domain Mapping
contributing source? (2 posts)

  1. yivi
    Member
    Posted 9 months ago #

    Line 373 is:

    } elseif( $_GET[ 'action' ] == 'delete' ) {

    But should be:

    } elseif( isset( $_GET[ 'action' ] ) && $_GET[ 'action' ] == 'delete' ) {

    And in line 447 you have:

    $domains[] = array( 'domain' => $orig_url[ 'host' ], 'path' => $orig_url['path'], 'active' => 0 );

    Which could be rewritten as this to avoid Notices when path is empty (e.g, the site is hosted at root):

    $domains[] = array( 'domain' => $orig_url[ 'host' ], 'path' => isset($orig_url[ 'path' ])?$orig_url['path'] : '', 'active' => 0 );

    Also, in line 462 you have

    $url = "{$protocol}{$details[ 'domain' ]}{$deatils['path']}";

    Which can trigger a E_NOTICE as well, same as before, and I replaced with:

    $path = isset($details['path']) ? $details['path'] : "";
    $url = "{$protocol}{$details[ 'domain' ]}$path";

    I couldn't find where to contribute code, the SVN tree seems to be read only, or am I failing to understand something?

    Regards!

    I.-

    (Otherwise you get a notice when entering "Domain Mapping" from one of the Sites (as opposed to when you enter from Network Admin).

    Thanks and regards!

    http://wordpress.org/plugins/wordpress-mu-domain-mapping/

  2. Ron Rennick
    MultiSite Guru
    Plugin Author

    Posted 9 months ago #

    the SVN tree seems to be read only

    That's correct. There isn't any way for you to directly update plugin code in the WP.org repo unless you are owner of the plugin in question or if the owner of the plugin has given you update access.

Reply

You must log in to post.

About this Plugin

About this Topic

Tags

No tags yet.