Save old emails?

[Boofo]

I want to update a hack that I used on vb3.0 that would save the old email address if a user other than usergroup 3 (awaiting email confirmation) changed it. The old hck used 2 queries in the profile, 1 to pull the email and one to update he profile field. I used a profile field to show the old emails but to admins only in the profile.

I'm not sure where to add it (the hook location) now and can I do it without 2 queries?

[Andreas]

I think you can do it without any Query (untested)

userdata_presave

PHP Code:

if ($this->existing['usergroupid'] != 3 AND $this->existing['email] != $this->fetch_field('email'))
{
    $this->setfields["fieldxx"] = true;
    $this->userfield["fieldxx"] = $this->fetch_field('email'); 
} 


[Boofo]

Ok, I'll test this and get back to you. Thank you, sir.

How would I go about adding a comma and a space between each email like the currently active users does?

[Marco van Herwaarden]

Change:

PHP Code:

$this->userfield["fieldxx"] = $this->fetch_field('email'); 


To:

PHP Code:

$this->userfield["fieldxx"] = ($this->userfield["fieldxx"]  ? $this->userfield["fieldxx"] . ", " . $this->fetch_field('email') : $this->userfield["fieldxx"]); 


[Andreas]

And then change

PHP Code:

$this->userfield["fieldxx"] = ($this->userfield["fieldxx"]  ? $this->userfield["fieldxx"] . ", " . $this->fetch_field('email') : $this->userfield["fieldxx"]); 


to

PHP Code:

$this->userfield["fieldxx"] = ($this->userfield["fieldxx"]  ? $this->userfield["fieldxx"] . ", " . $this->fetch_field('email') : $this->fetch_field('email')); 


to make it work correctly

[Marco van Herwaarden]

Lol, correct.

[Boofo]

LMAO @ Kirby. You guys kill me! LOL

Thank you both, sirs.

[Andreas]

4th Revision of 1 Code line (LOL):

PHP Code:

$this->userfield['fieldxx'] = ($this->fetch_field('fieldxx'))  ? $this->fetch_field('fieldxx') . ', ' . $this->fetch_field('email') : $this->fetch_field('email')); 


[Boofo]

Ok, that works now (after taking out one of these right parenthesis: ('fieldxx'))from the secoind variable) LOL

It is listing the emails like it should. But if you change the email back to what it once was, it relists that email again. Is that the way it should work?

[Andreas]

Yes. If you want to avoid that, wrap it in if (strpos($this->fetch_field('fieldxx'), $this->fetch_field('email')) === false)