Bug with mailing Reply-to (with fix)

Permalink Browser Info Environment
Hi,

There is a bug with the mailing portion of the plugin when selecting Reply-to as an email field from the form. The result was that the reply to was always using the From email instead of the selected one.

To fix the issue:
models/formidable/mailing.php replace:

// Set Reply To
if ($this->reply_type == 'from') {
$this->reply_name = $this->from_name;
$this->reply_email = $this->from_email;
} elseif (intval($this->reply_type) != 0) {
if (!empty($this->elements[$this->from_type]->result)) {
$this->reply_name = '';
$this->reply_email = $this->elements[$this->from_type]->result;
}
}

with
// Set Reply To
if ($this->reply_type == 'from') {
$this->reply_name = $this->from_name;
$this->reply_email = $this->from_email;
} elseif (intval($this->reply_type) != 0) {
if (!empty($this->elements[$this->reply_type]->result)) {
$this->reply_name = '';
$this->reply_email = $this->elements[$this->reply_type]->result;
}
}


Thanks!

Type: Discussion
Status: Resolved
olacom
View Replies:
DeWebmakers replied on at Permalink Reply
DeWebmakers
Thanx! I'll change this asap.

concrete5 Environment Information

# concrete5 Version
5.6.3.3

# concrete5 Packages
Designer Content (3.1.1), Fileset attribute (1.0), Formidable (2.1.1), Internationalization (1.3), Page List+ (5.6.0.0.0.5.7), tnSpacer (1.3).

# concrete5 Overrides
blocks/autonav, blocks/formidable, blocks/page_list_plus, blocks/switch_language, blocks/team_member, languages/ar, languages/cs_CZ, languages/da_DK, languages/de_DE, languages/el_GR, languages/es_AR, languages/es_ES, languages/es_MX, languages/es_PE, languages/fa_IR, languages/fi_FI, languages/fr_CA, languages/fr_FR, languages/it_IT, languages/ja_JP, languages/nb_NO, languages/nl_NL, languages/pt_PT, languages/ro_RO, languages/ru_RU, languages/sk_SK, languages/sl_SI, languages/sv_SE, languages/tr_TR, languages/vi_VN, languages/zh_TW, single_pages/en, single_pages/fr, themes/designed_by_tonik

# concrete5 Cache Settings
Block Cache - Off
Overrides Cache - Off
Full Page Caching - Off

# Server Software
Apache/2.4.6 (CentOS) PHP/5.6.30

# Server API
apache2handler

# PHP Version
5.6.30

# PHP Extensions
apache2handler, apc, apcu, bcmath, bz2, calendar, Core, ctype, curl, date, dom, ereg, exif, fileinfo, filter, ftp, gd, gettext, gmp, hash, iconv, intl, json, libxml, mbstring, mcrypt, mhash, mongo, mysql, mysqli, mysqlnd, odbc, openssl, pcre, PDO, pdo_mysql, PDO_ODBC, pdo_pgsql, pdo_sqlite, pgsql, Phar, posix, recode, Reflection, session, shmop, SimpleXML, soap, sockets, SPL, SQLite, sqlite3, standard, sysvmsg, sysvsem, sysvshm, tidy, tokenizer, wddx, xml, xmlreader, xmlwriter, xsl, zip, zlib.

# PHP Settings
max_execution_time - 30
log_errors_max_len - 1024
max_file_uploads - 20
max_input_nesting_level - 64
max_input_time - 60
max_input_vars - 1000
memory_limit - 128M
post_max_size - 16M
sql.safe_mode - Off
upload_max_filesize - 15M
mysql.max_links - Unlimited
mysql.max_persistent - Unlimited
mysqli.max_links - Unlimited
mysqli.max_persistent - Unlimited
odbc.max_links - Unlimited
odbc.max_persistent - Unlimited
pcre.backtrack_limit - 1000000
pcre.recursion_limit - 100000
pgsql.max_links - Unlimited
pgsql.max_persistent - Unlimited
session.cache_limiter - nocache
session.gc_maxlifetime - 7200
soap.wsdl_cache_limit - 5

Browser User-Agent String

Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0

Hide Post Content

This will replace the post content with the message: "Content has been removed by an Administrator"

Hide Content

Request Refund

You may not request a refund that is not currently owned by you.