Very minor, but it does add an extra closing /div on the settings page which subsequently pops that which follows it out of my layout.

It seems to be generated around the list of reserved usernames....

<p>
<h3>Reserved Usernames</h3>
<p>The following usernames cannot be registered, since they are used as unique subdomains:</p>
</div>????????????????????????????????????????????????????????????????????????????????????????
<ul><li>michael-shanley.com</li></ul>
</p>

Also, it seems bizarre to put a heading or a list within a paragraph, so maybe that's part of the coding error.

But anyway, thank you SOOOOOOOO much for this awesome module!

Oh! -- Just in case you need these...
Drupal 6.3,
MySQL 5.x
PHP 5.2
I've got too many modules on this site to list them all, but nothing that would cause this kind of PHP error.

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Active » Patch (to be ported)

Nah, the extra <p> was there to provide spacing -- should be a BR. The div is vestigial, like the appendix. Line 70 of domain_user.admin.inc should be:

    '#value' => '<br /><br />'. t('<h3>Reserved Usernames</h3><p>The following usernames cannot be registered, since they are used as unique subdomains:</p>') . $output,

This would apply (slightly offset) to D5 as well.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

Committed to HEAD. Needs backport.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Patch (to be ported) » Fixed

Committed to 5.x.dev.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.