[Domain Mapping] DM: incorrect path to wp-config shown in sunrise notification

Issue: incorrect path to wp-config shown in sunrise notification

to replicate:

1) place wp-config.php and index.php in webroot directory, /

2) place WP files in a subdirectory of webroot, /wp

3) per codex, edit /wp-config.php ABSPATH to '/wp/' [and index.php /wp/wp-blog-header.php]

4) setup network

5) do NOT add sunrise define, do NOT copy sunrise file

6) network activate Domain Mapping plugin, navigate to …/wp-admin/network/settings.php?page=domainmapping_options

7) observer notification with incorrect path:

"In your /home/username/public_html/wp/wp-config.php file please uncomment or add (if not available) the following code:"

[this is third try to submit this ticket, this time with no code… if this works I'll add a comment with further details ~max]

  • wp.network
    • The Bug Hunter

    ok, cool beans :slight_smile:–> obviously, that worked and here is the rest of the ticket:

    Why is this happening: well, seems like domain-mappingclassesDomainmapRenderNetworkOptions.php L76

    `__( ‘Please copy the sunrise.php from your plugin folder %s into %s.<br/>In your %s file please uncomment (or add, if not available), the following code: %s’, ‘domainmap’ ),

    ‘<b>’.DOMAINMAP_ABSPATH.’/sunrise.php</b>’,

    ‘<b>’ . WP_CONTENT_DIR . ‘/sunrise.php</b>’,

    ‘<b>’ . ABSPATH . ‘wp-config.php</b>’,

    ‘<code>define( ‘SUNRISE’, ‘on’ );</code>’`

    Possible solution/over-to-you: its kind of a quick-fix, but it reads ok to me…

    `__( ‘Please copy the sunrise.php from your plugin folder %s into %s.<br/>In your wp-config.php file please uncomment (or add, if not available), the following code: %s (Your wp-config.php file is likely located at %s)’, ‘domainmap’ ),

    ‘<b>’.DOMAINMAP_ABSPATH.’/sunrise.php</b>’,

    ‘<b>’ . WP_CONTENT_DIR . ‘/sunrise.php</b>’,

    ‘<code>define( ‘SUNRISE’, ‘on’ );</code>’,

    ‘<b>’ . ABSPATH . ‘wp-config.php</b>’`

    Obviously, this is a small thing; still, its a bit of polish and might save someone from getting confused…

    attached: screenshot, showing original on Left, and my edits on Right

    Cheers, Max

    (ps. if you notice that there is no /wp/ shown in screenshot URL, see: https://core.trac.wordpress.org/ticket/19796#comment:78)

  • Ash
    • WordPress Hacker

    Hello wp.network

    Thank you for your details explanation. It makes sense.

    I have sent a note to the developer for his opinion, he will check the issue and definitely add a solution. I just can’t mark it as a bug right now w/o developer’s opinion though, but we will update you.

    Have a nice day!

    Cheers,

    Ash

Thank NAME, for their help.

Let NAME know exactly why they deserved these points.

Gift a custom amount of points.