You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2012/02/07 02:20:01 UTC

svn commit: r1241310 - /jmeter/trunk/src/components/org/apache/jmeter/reporters/MailerModel.java

Author: sebb
Date: Tue Feb  7 01:20:00 2012
New Revision: 1241310

URL: http://svn.apache.org/viewvc?rev=1241310&view=rev
Log:
Property values are documented as being Strings (even if interpreted as int)
Although it seems to work when saved as an Integer, better to follow the docs.

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/reporters/MailerModel.java

Modified: jmeter/trunk/src/components/org/apache/jmeter/reporters/MailerModel.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/reporters/MailerModel.java?rev=1241310&r1=1241309&r2=1241310&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/reporters/MailerModel.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/reporters/MailerModel.java Tue Feb  7 01:20:00 2012
@@ -350,7 +350,7 @@ public class MailerModel extends Abstrac
         Properties props = new Properties();
 
         props.put(MAIL_SMTP_HOST, host);
-        props.put(MAIL_SMTP_PORT, smtpPort);
+        props.put(MAIL_SMTP_PORT, Integer.toString(smtpPort)); // property values are strings
         Authenticator authenticator = null;
         if(mailAuthType != MailAuthType.NONE) {
             props.put(MAIL_SMTP_AUTH, "true");