Save old emails?

[Boofo]

Like this?

PHP Code:

 if (strpos($this->fetch_field('field6'), $this->fetch_field('email') !== false)
 if ($this->existing['usergroupid'] != 3 AND $this->existing['email'] != $this->fetch_field('email')) 
 { 
     $this->setfields['field6'] = true; 
     $this->userfield['field6'] = ($this->fetch_field('field6')  ? $this->fetch_field('field6') . ', ' . $this->fetch_field('email') : $this->fetch_field('email'));
 }
} 


[Andreas]

Then you it would only execute if the email is already in the list.
Any there is a ) and a { missing.

Edited my previous Post.

[Boofo]

Ahhh, ok, I see what I need to do now.

Like this:

PHP Code:

 if ($this->existing['usergroupid'] != 3 AND $this->existing['email'] != $this->fetch_field('email'))
{
if (strpos($this->fetch_field('field6'), $this->fetch_field('email')) === false)
{
     $this->setfields['field6'] = true;
     $this->userfield['field6'] = ($this->fetch_field('field6') ? $this->fetch_field('field6') . ', ' . $this->fetch_field('email') : $this->fetch_field('email'));
}
} 


[Andreas]

The edited Version looks correct (if there are no missing brackets or whatever)

But you can shorten it

PHP Code:

if ($this->existing['usergroupid'] != 3 AND $this->existing['email'] != $this->fetch_field('email') AND strpos($this->fetch_field('field6'), $this->fetch_field('email')) === false))
{
    $this->setfields['field6'] = true;
    $this->userfield['field6'] = (($this->fetch_field('field6'))  ? $this->fetch_field('field6') . ', ' . $this->fetch_field('email') : $this->fetch_field('email'));
}