Fix types warnings from psalm

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

Author: CarlSchwan

apps/sharebymail/lib/ShareByMailProvider.php

@@ -334,13 +334,13 @@ protected function createMailShare(IShare $share) {
 			$share->getNote()
 		);
 
-		if ($this->mailer->validateMailAddress($share->getSharedWith())) {
+		if (!$this->mailer->validateMailAddress($share->getSharedWith())) {
 			$this->removeShareFromTable($shareId);
 			$e = new HintException('Failed to send share by mail. Got an invalid email address: ' . $share->getSharedWith(),
 				$this->l->t('Failed to send share by email. Got an invalid email address'));
-			$this->logger->error($e->getMessage(), [
-				'message' => 'Failed to send share by mail. Got an invalid email address ' . $share->getSharedWith(),
+			$this->logger->error('Failed to send share by mail. Got an invalid email address ' . $share->getSharedWith(), [
 				'app' => 'sharebymail',
+				'exception' => $e,
 			]);
 		}
 
@@ -689,7 +689,7 @@ public function getChildren(IShare $parent) {
 	 * @param \DateTime|null $expirationTime
 	 * @return int
 	 */
-	protected function addShareToDB($itemSource, $itemType, $shareWith, $sharedBy, $uidOwner, $permissions, $token, $password, $sendPasswordByTalk, $hideDownload, $label, $expirationTime, $note = '') {
+	protected function addShareToDB($itemSource, $itemType, $shareWith, $sharedBy, $uidOwner, $permissions, $token, $password, $sendPasswordByTalk, $hideDownload, $label, $expirationTime, $note = ''): int {
 		$qb = $this->dbConnection->getQueryBuilder();
 		$qb->insert('share')
 			->setValue('share_type', $qb->createNamedParameter(IShare::TYPE_EMAIL))
@@ -784,7 +784,7 @@ public function delete(IShare $share) {
 		} catch (\Exception $e) {
 		}
 
-		$this->removeShareFromTable($share->getId());
+		$this->removeShareFromTable((int)$share->getId());
 	}
 
 	/**
@@ -980,9 +980,9 @@ public function getShareByToken($token) {
 	/**
 	 * remove share from table
 	 *
-	 * @param string $shareId
+	 * @param int $shareId
 	 */
-	protected function removeShareFromTable($shareId) {
+	protected function removeShareFromTable(int $shareId): void {
 		$qb = $this->dbConnection->getQueryBuilder();
 		$qb->delete('share')
 			->where($qb->expr()->eq('id', $qb->createNamedParameter($shareId)));

apps/sharebymail/tests/ShareByMailProviderTest.php

@@ -465,6 +465,7 @@ public function testCreateMailShare() {
 
 		$instance = $this->getInstance(['generateToken', 'addShareToDB', 'sendMailNotification']);
 
+		$instance->expects($this->once())->method('getSharedWith')->willReturn(['valid@valid.com']);
 		$instance->expects($this->once())->method('generateToken')->willReturn('token');
 		$instance->expects($this->once())->method('addShareToDB')->willReturn(42);
 		$instance->expects($this->once())->method('sendMailNotification');