Ran into a new and annoying one today. Here's steps to replicate:
1. edit a domain, set stautus to inactive
2. in domain list domain shows up as inactive
3. on domain edit domain shows up as active

The culprit is line 462 in domain.admin.inc

'#default_value' => !empty($domain['valid']) ? $domain['valid'] : 1,

which should be

'#default_value' => !empty($domain['valid']) ? $domain['valid'] : 0,

I'd roll a patch but my pc is a bit under the weather after blessing me with a blue screen...
Patch to follow as soon as I get up and running again.

CommentFileSizeAuthor
#1 572124-status.patch897 bytesagentrickard

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Active » Fixed
StatusFileSize
new 572124-status.patch897 bytes

Actually. we have to use isset() here, to ensure that the default is 1 when the form is first loaded.

Nice catch, I can't believe we never saw this before. Must've been the form changes that went through a while back.

Committed.

Status: Fixed » Closed (fixed)

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