add missing conditional statement in send-all method also comment the sendmail functionality in welcome mail

Author: TSI-ashishsharma

lib/Command/SendAllMail.php

@@ -52,20 +52,15 @@ public function __construct(
 	}
 
 	protected function configure() {
-		$this
-						->setName('monthly_status_email:send-all')
-						->setDescription('Send the notification mail to all users');
+		$this->setName('monthly_status_email:send-all')
+			->setDescription('Send the notification mail to all users');
 		parent::configure();
 	}
 
 	protected function execute(InputInterface $input, OutputInterface $output) {
 		$trackedNotifications = $this->service->findAll();
 		foreach ($trackedNotifications as $trackedNotification) {
-			try {
-				$ret = $this->mailSender->sendMonthlyMailTo($trackedNotification);
-			} catch (\Exception $e) {
-				$output->writeln('Failure sending email to ' . $trackedNotification->getUserId());
-			}
+			$ret = $this->mailSender->sendMonthlyMailTo($trackedNotification);
 			if ($ret) {
 				$output->writeln('Email sent to ' .  $trackedNotification->getUserId());
 			} else {

lib/Service/MailSender.php

@@ -250,7 +250,7 @@ public function sendStatusEmailActivation(IUser $user, NotificationTracker $trac
 		}
 
 		$this->provider->writeWelcomeMail($emailTemplate, $user->getDisplayName());
-		$this->sendEmail($emailTemplate, $user, $message, $trackedNotification);
+		//$this->sendEmail($emailTemplate, $user, $message, $trackedNotification);
 	}
 
 	private function sendEmail(IEMailTemplate $template, IUser $user, IMessage $message, ?NotificationTracker $trackedNotification = null): void {