opencats / OpenCATS

Applicant Tracking System (maintained code base)
http://www.opencats.org
Other
511 stars 243 forks source link

Career portal applicants receive 'candidate ownership change' email in error #343

Open RussH opened 6 years ago

RussH commented 6 years ago

reported in the forums; http://forums.opencats.org/viewtopic.php?f=4&t=977 Issue: Even when no emails are checked to send, the applicant still gets an email message:

From: email@ofadmin.com Subject:CATS Notification: Candidate Ownership Change Body: email@ofapplicant.com Powered by CATS (Free ATS)

Workaround: commented out the following in lib/Candidates.php

if (!empty($emailAddress))
        {
            /* Send e-mail notification. */
            //FIXME: Make subject configurable.
            // $mailer = new Mailer($this->_siteID);
            // $mailerStatus = $mailer->sendToOne(
            //    array($emailAddress, ''),
            //    'CATS Notification: Candidate Ownership Change',
            //    $email,
            //    true
            //);
        }
evank28 commented 5 years ago

@RussH Why is this labelled as NotAnIssue? seems like a very easy fix. Can I submit a PR?

RussH commented 5 years ago

HI @evank28 this is flagged as not an issue as it's only been reported once, may be down to user misconfiguration and there's a simple workaround posted above. If this is reported by multiple users then a PR to change default behaviour would be accepted