Em um controller chamado atividades, tenho um metodo chamado lembrete.
Esse metodo, recebe um id, procura aquela atividade com base nesse id e envia um email pro moderador daquela atividade com dados da mesma.
Esse metodo esta com 'responsabilidades demais'?
Estou enviando dois metodos para já unir outra duvida. Por designer de código, qual deles está correto? O que envia separa a $client em uma variavel (reminder_a), ou o que sempre utiliza $activity para se referenciar aos clientes (reminder_b)?
Segue metodo:
/**
* Send the remember email with a resource
* @param int $id
*/
public function reminder_a($id)
{
$activity = $this->activities->getById($id);
$client = $activity->client;
$title = 'Reminder of activity ' . $activity->name;
$view = 'emails.activity.reminder';
$data = ['activity' => $activity, 'cliente' => $client];
CustomMail::sendBasicMail($view, $title, $activity->client->email, $data);
}
/**
* Send the remember email with a resource
* @param int $id
*/
public function reminder_b($id)
{
$activity = $this->activities->getById($id);
$title = 'Reminder of activity ' . $activity->name;
$view = 'emails.activity.reminder';
CustomMail::sendBasicMail($view, $title, $activity->client->email, $activity);
}