feat: mail provider backend

Signed-off-by: SebastianKrupinski <[email protected]>

lib/private/Mail/Provider/Manager.php

@@ -125,15 +125,15 @@ public function providers(): array {
 	 *
 	 * @return IProvider|null
 	 */
-	public function findProviderById(string $id): IProvider | null {
+	public function findProviderById(string $providerId): IProvider | null {
 
 		// evaluate if we already have a cached collection of providers
 		if (!is_array($this->providersCollection)) {
 			$this->providers();
 		}
 
-		if (isset($this->providersCollection[$id])) {
-			return $this->providersCollection[$id];
+		if (isset($this->providersCollection[$providerId])) {
+			return $this->providersCollection[$providerId];
 		}
 		// return null if provider was not found
 		return null;

lib/public/Mail/Provider/IMessage.php

@@ -63,7 +63,7 @@ public function setReplyTo(IAddress $value): self;
 	 *
 	 * @since 30.0.0
 	 *
-	 * @param IAddress|null                     sender's mail address object
+	 * @param IAddress|null                     sender's reply to mail address object
 	 */
 	public function getReplyTo(): IAddress | null;
 

lib/public/Mail/Provider/Message.php

@@ -23,7 +23,7 @@ class Message implements \OCP\Mail\Provider\IMessage {
 	 *
 	 * @since 30.0.0
 	 *
-	 * @param array|null $data          message data array
+	 * @param array $data						message data array
 	 */
 	public function __construct(
 		protected array $data = [],
@@ -39,7 +39,7 @@ public function __construct(
 	 */
 	public function id(): string {
 		// return id of message
-		return (isset($this->data['id'])) ? $this->data['id'] : null;
+		return (isset($this->data['id'])) ? $this->data['id'] : '';
 	}
 
 	/**
@@ -91,7 +91,7 @@ public function setReplyTo(IAddress $value): self {
 	 *
 	 * @since 30.0.0
 	 *
-	 * @param IAddress|null                     sender's mail address object
+	 * @param IAddress|null                     sender's reply to mail address object
 	 */
 	public function getReplyTo(): IAddress | null {
 		// evaluate if data store field exists and return value(s) or null otherwise
@@ -284,9 +284,9 @@ public function getBodyHtml(): string | null {
 	 *
 	 * @since 30.0.0
 	 *
-	 * @param string $value         plain text body of message
+	 * @param string $value         			plain text body of message
 	 *
-	 * @return self                 return this object for command chaining
+	 * @return self                 			return this object for command chaining
 	 */
 	public function setBodyPlain(string $value): self {
 		// create or update field in data store with value
@@ -300,7 +300,7 @@ public function setBodyPlain(string $value): self {
 	 *
 	 * @since 30.0.0
 	 *
-	 * @param string|null                       plain text body of this message or null if one is not set
+	 * @param string|null						plain text body of this message or null if one is not set
 	 */
 	public function getBodyPlain(): string | null {
 		// evaluate if data store field exists and return value(s) or null otherwise
@@ -312,9 +312,9 @@ public function getBodyPlain(): string | null {
 	 *
 	 * @since 30.0.0
 	 *
-	 * @param IAttachment ...$value                     collection of or one or more mail attachment objects
+	 * @param IAttachment ...$value				collection of or one or more mail attachment objects
 	 *
-	 * @return self                                     return this object for command chaining
+	 * @return self                             return this object for command chaining
 	 */
 	public function setAttachments(IAttachment ...$value): self {
 		// create or update field in data store with value
@@ -328,7 +328,7 @@ public function setAttachments(IAttachment ...$value): self {
 	 *
 	 * @since 30.0.0
 	 *
-	 * @return array<int,IAttachment>		            collection of all mail attachment objects
+	 * @return array<int,IAttachment>		    collection of all mail attachment objects
 	 */
 	public function getAttachments(): array {
 		// evaluate if data store field exists and return value(s) or null otherwise