Bug 144207 - ChangeEmailAction has hard coded text
Summary: ChangeEmailAction has hard coded text
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Network
Classification: Red Hat
Component: RHN/R&D
Version: RHN Devel
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Ken Ganong
QA Contact: Vlady Zlatkin
URL:
Whiteboard:
Keywords:
: 145785 (view as bug list)
Depends On:
Blocks: 151514
TreeView+ depends on / blocked
 
Reported: 2005-01-05 01:54 UTC by Mike McCune
Modified: 2007-04-18 17:17 UTC (History)
4 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2005-08-31 20:55:20 UTC


Attachments (Terms of Use)

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.


Note You need to log in before you can comment on or make changes to this bug.