Project

General

Profile

Actions

Bug #55922

closed

Bug with multiple e-mail adresses in scheduler task

Added by Online Now! GmbH about 10 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Linkvalidator
Target version:
Start date:
2014-02-12
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
no-brainer
Is Regression:
No
Sprint Focus:

Description

There is a small bug in the scheduler task for the linkvalidator. You can enter multiple receipients seperated with comma but sending to them does not work. The mail object gets the original value from the input field with the commas as parameter and not the validated array with the adresses.

Source: typo3/sysext/linkvalidator/classes/tasks/class.tx_linkvalidator_tasks_validator.php ( Line: 458 )

if (is_array($validEmailList) && !empty($validEmailList)) {
    $mail->setTo($this->email);
} else {
    $sendEmail = FALSE;
}

This should be changed to:

if (is_array($validEmailList) && !empty($validEmailList)) {
    $mail->setTo($validEmailList);
} else {
    $sendEmail = FALSE;
}
Actions #1

Updated by Markus Klein about 10 years ago

  • Status changed from New to Accepted
  • Target version set to next-patchlevel
  • Complexity changed from easy to no-brainer

Present in 6.2 as well

Actions #2

Updated by Gerrit Code Review about 10 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27580

Actions #3

Updated by Markus Klein about 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #4

Updated by Online Now! GmbH almost 10 years ago

Is there a reason why this issue was only fixed for the TYPO3 6.2 branch and set to resolved? I just noticed that this bug is still present in the latest 4.5 release ( 4.5.35 ).

Actions #5

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF