Bug 144207

Summary: ChangeEmailAction has hard coded text
Product: Red Hat Network Reporter: Mike McCune <mmccune>
Component: RHN/R&DAssignee: Ken Ganong <kganong>
Status: CLOSED CURRENTRELEASE QA Contact: Vlady Zlatkin <vzlatkin>
Severity: medium Docs Contact:
Priority: medium    
Version: RHN DevelCC: fmoquete, jesusr, mspevack, vzlatkin
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: RHN 4.0.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-08-31 20:55:20 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 151514    

Description Mike McCune 2005-01-05 01:54:18 UTC
This should be localized:

  
    /**
     * Sends out a verification email to the given address.
     * @param addr The email address to which the email should be sent.
     */
    private void sendVerificationEmail(String addr) {
        Mail mailer = new SmtpMail();
        mailer.setRecipients(new String[] { addr });
        mailer.setSubject("Email Verification");
        mailer.setBody("This is a test");
        mailer.send();
    }

Comment 1 Mike McCune 2005-03-02 19:24:37 UTC
*** Bug 145785 has been marked as a duplicate of this bug. ***

Comment 2 Ken Ganong 2005-05-31 21:11:57 UTC
There were much larger issues here.  We needed the correct email text for one,
including generating a url for verification.  Also, the current code did not
take into account the differences for satellite in the perl code.

Comment 3 Jesus M. Rodriguez 2005-06-10 18:33:38 UTC
Mass moving ON_DEV to ON_QA

Comment 4 Beth Nackashi 2005-07-05 20:02:20 UTC
This doesn't look like it should be ON_QA; at the very least there should be
some sort of test plan.

Comment 5 Ken Ganong 2005-07-05 20:17:23 UTC
TEST PLAN
Login as an org admin on hosted.
Go to users->click a user->confirm email address.
change the email address and then click send verification.
A verification email should be sent.

confirm that this email has a more interesting body than "this is a test",
account information, and a hashed url for verification.  Confirm that the url
has the correct host name and is either http or https accordingly.

On satellite there should be no "confirm" for email address... simply "change".
No email should be sent in a satellite setting.

Comment 6 Vlady Zlatkin 2005-07-05 21:39:31 UTC
looks good

Comment 7 Vlady Zlatkin 2005-07-05 21:44:14 UTC
its coming from dev-null@rhn.redhat.com, is that intentional?

Comment 8 Jesus M. Rodriguez 2005-07-05 21:57:42 UTC
That is the default value as defined by the web.default_mail_from configuration
variable.  If you would like it to be from someone else, override the above 
entry in /etc/rhn/rhn.conf.  

This is the current value in /etc/rhn/default/rhn_web.conf:

web.default_mail_from = Red Hat Network <dev-null@rhn.redhat.com>

Comment 9 Jesus M. Rodriguez 2005-07-05 21:58:13 UTC
Moving back to ON_QA.