.editorconfig

@@ -1,23 +0,0 @@
-#   Copyright 2017 Aviral Dasgupta
-#
-#   Licensed under the Apache License, Version 2.0 (the "License");
-#   you may not use this file except in compliance with the License.
-#   You may obtain a copy of the License at
-#
-#       http://www.apache.org/licenses/LICENSE-2.0
-#
-#   Unless required by applicable law or agreed to in writing, software
-#   distributed under the License is distributed on an "AS IS" BASIS,
-#   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-#   See the License for the specific language governing permissions and
-#   limitations under the License.
-
-root = true
-
-[*]
-charset=utf-8
-end_of_line = lf
-insert_final_newline = true
-indent_style = tab
-indent_size = 2
-trim_trailing_whitespace = true

composer.lock

@@ -27,6 +27,7 @@
                 "phpunit/php-code-coverage": "^7.0",
                 "phpunit/phpunit": "^8.5"
             },
+            "default-branch": true,
             "type": "library",
             "extra": {
                 "branch-alias": {

lib/Controller/Errors.php

@@ -3,7 +3,7 @@
 namespace OCA\UPschooling\Controller;
 
 use Closure;
-use OCA\UPschooling\Exceptions\TicketNotFoundException;
+use OCA\UPschooling\Service\NoteNotFound;
 use OCP\AppFramework\Http;
 use OCP\AppFramework\Http\DataResponse;
 
@@ -11,7 +11,7 @@ trait Errors {
 	protected function handleNotFound(Closure $callback): DataResponse {
 		try {
 			return new DataResponse($callback());
-		} catch (TicketNotFoundException $e) {
+		} catch (NoteNotFound $e) {
 			$message = ['message' => $e->getMessage()];
 			return new DataResponse($message, Http::STATUS_NOT_FOUND);
 		}

lib/Controller/TicketApiController.php

@@ -51,14 +51,14 @@ class TicketApiController extends ApiController
 		return new DataResponse($this->service->create($title, $content, $this->userId));
 	}
 
-
-
 	/**
 	 * @NoAdminRequired
 	 */
-	public function update(): DataResponse
+	public function update(int $id, string $title, string $content): DataResponse
 	{
-		return new DataResponse($this->service->assign(44,$this->userId));
+		return $this->handleNotFound(function () use ($id, $title, $content) {
+			return $this->service->update($id, $title, $content, $this->userId);
+		});
 	}
 
 	public function destroy(int $id): DataResponse

lib/Db/TicketMapper.php

@@ -15,22 +15,6 @@ class TicketMapper extends QBMapper
 		parent::__construct($db, 'upschooling_tickets', MatrixTicket::class);
 	}
 
-	/**
-	 * @param int $id
-	 * @return Entity|MatrixTicket
-	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException
-	 * @throws DoesNotExistException
-	 */
-	public function findTicket(int $id): MatrixTicket
-	{
-		/* @var $qb IQueryBuilder */
-		$qb = $this->db->getQueryBuilder();
-		$qb->select('*')
-			->from('upschooling_tickets')
-			->where($qb->expr()->eq('id', $qb->createNamedParameter($id, IQueryBuilder::PARAM_INT)));
-		return $this->findEntity($qb);
-	}
-
 	/**
 	 * @param int $id
 	 * @param string $userId
@@ -38,19 +22,14 @@ class TicketMapper extends QBMapper
 	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException
 	 * @throws DoesNotExistException
 	 */
-	public function findForUser(int $id, MatrixUser $matrixUser): MatrixTicket
+	public function find(int $id, MatrixUser $matrixUser): MatrixTicket
 	{
 		/* @var $qb IQueryBuilder */
 		$qb = $this->db->getQueryBuilder();
 		$qb->select('*')
 			->from('upschooling_tickets')
-			->where(
-				$qb->expr()->eq('id', $qb->createNamedParameter($id, IQueryBuilder::PARAM_INT)),
-				$qb->expr()->orX(
-					$qb->expr()->eq('matrix_assisted_user', $qb->createNamedParameter($matrixUser->getMatrixUser(), IQueryBuilder::PARAM_STR)),
-					$qb->expr()->eq('matrix_helper_user', $qb->createNamedParameter($matrixUser->getMatrixUser(), IQueryBuilder::PARAM_STR))
-				)
-			);
+			->where($qb->expr()->eq('id', $qb->createNamedParameter($id, IQueryBuilder::PARAM_INT)))
+			->andWhere($qb->expr()->in($matrixUser->getMatrixUser(), ['matrix_assisted_user', 'matrix_helper_user']));
 		return $this->findEntity($qb);
 	}
 
@@ -58,18 +37,13 @@ class TicketMapper extends QBMapper
 	 * @param string $userId
 	 * @return array
 	 */
-	public function findAllForUser(MatrixUser $matrixUser): array
+	public function findAll(MatrixUser $matrixUser): array
 	{
 		/* @var $qb IQueryBuilder */
 		$qb = $this->db->getQueryBuilder();
 		$qb->select('*')
 			->from('upschooling_tickets')
-			->where(
-				$qb->expr()->orX(
-					$qb->expr()->eq('matrix_assisted_user', $qb->createNamedParameter($matrixUser->getMatrixUser(), IQueryBuilder::PARAM_STR)),
-					$qb->expr()->eq('matrix_helper_user', $qb->createNamedParameter($matrixUser->getMatrixUser(), IQueryBuilder::PARAM_STR))
-				)
-			);
+			->where($qb->expr()->in($matrixUser->getMatrixUser(), ['matrix_assisted_user', 'matrix_helper_user']));
 		return $this->findEntities($qb);
 	}
 }

lib/Exceptions/TicketNotFoundException.php

@@ -1,14 +0,0 @@
-<?php
-
-namespace OCA\UPschooling\Exceptions;
-
-use OC\OCS\Exception;
-use OC\OCS\Result;
-
-class TicketNotFoundException extends Exception
-{
-	public function __construct($ticketId)
-	{
-		parent::__construct(new Result(null, 404, 'Ticket with id ' . $ticketId . ' not found'));
-	}
-}

lib/Service/TicketService.php

@@ -11,7 +11,6 @@ use OCA\UPschooling\Db\MatrixUser;
 use OCA\UPschooling\Db\TicketMapper;
 use OCA\UPschooling\Db\UserMapper;
 use OCA\UPschooling\Exceptions\RoomNotFoundException;
-use OCA\UPschooling\Exceptions\TicketNotFoundException;
 use OCP\AppFramework\Db\DoesNotExistException;
 use OCP\AppFramework\Db\MultipleObjectsReturnedException;
 
@@ -33,24 +32,12 @@ class TicketService {
 	}
 
 	public function findAll(string $userId): array {
-		$dbTickets = $this->ticketMapper->findAllForUser($this->getOrCreateUser($userId));
+		$dbTickets = $this->ticketMapper->findAll($this->getOrCreateUser($userId));
 		return array_map(function ($ticket) { return $this->resolveTicket($ticket); }, $dbTickets);
 	}
 
-	/**
-	 * @throws MatrixRequestException
-	 * @throws MatrixHttpLibException
-	 * @throws MultipleObjectsReturnedException
-	 * @throws \OCP\DB\Exception
-	 * @throws TicketNotFoundException
-	 * @throws MatrixException
-	 */
 	public function find($id, $userId): array {
-		try {
-			return $this->resolveTicket($this->ticketMapper->findForUser($id, $this->getOrCreateUser($userId)));
-		} catch (DoesNotExistException $e) {
-			throw new TicketNotFoundException($id);
-		}
+		return $this->resolveTicket($this->ticketMapper->find($id, $this->getOrCreateUser($userId)));
 	}
 
 	public function create($title, $content, $userId): array {
@@ -72,19 +59,6 @@ class TicketService {
 		return $this->resolveTicket($this->ticketMapper->insert($ticket));
 	}
 
-	
-
-	public function assign($id, $userId): array{
-		$matrixUser = $this->getOrCreateUser($userId);
-		$ticket = $this->ticketMapper->findTicket($id);
-		$roomID= $ticket->getMatrixRoom();
-		$this->matrix->setProperty($roomID, "upschooling.ticket", array(
-			"matrixHelperUser" => $matrixUser->getMatrixUser()
-		));
-		$ticket->setMatrixHelperUser($matrixUser->getMatrixUser());
-		return $this->resolveTicket($this->ticketMapper->update($ticket));
-	}
-
 	public function update($id, $title, $content, $userId) {
 		throw new Exception("Not implemented");
 	}
@@ -106,11 +80,7 @@ class TicketService {
 			$title = array_get($matrixTicketContent, "title", "Untitled");
 			$description = array_get($matrixTicketContent, "description", "");
 			$lastModified = $this->matrix->getLastEventDate($ticket->getMatrixRoom());
-<<<<<<< HEAD
-			$matrixHelperUser = array_get($matrixTicketContent, "matrixHelperUser", "nicht zugewiesen"); 
-=======
 			$matrixHelperUserName = array_get($matrixTicketContent, "matrixHelperUserName", "nicht zugewiesen"); 
->>>>>>> ae0ffa811f99d5ee49597ff70a01aed705377b0c
 			$expirationDate = array_get($matrixTicketContent, "expiration date", "");
 			return array(
 				'ticketId' => $ticketId,
@@ -118,11 +88,7 @@ class TicketService {
 				'lastModified' => $lastModified,
 				'title' => $title,
 				'description' => $description,
-<<<<<<< HEAD
-				'matrixHelperUser' => $matrixHelperUser,
-=======
 				'matrixHelferName' => $matrixHelperUserName,
->>>>>>> ae0ffa811f99d5ee49597ff70a01aed705377b0c
 				'expirationDate' => $expirationDate 
 			);
 		} catch (MatrixException | RoomNotFoundException $e) {

src/App.vue

@@ -61,16 +61,14 @@ export default {
 					this.tickets.push(response.data)
 				}
 			}).catch(console.error)
-			axios.get(
-				'api/v1/tickets/1',
+			axios.post(
+				'api/v1/tickets',
+				{ title: 'Zweites Ticket', content: 'Zweites Beispiel' },
 				{ headers: { 'Content-Type': 'application/json', Accept: 'application/json' } },
-			).catch(console.error)
-			axios.put(
-				'api/v1/tickets/1',
-				{},
-				{ headers: { 'Content-Type': 'application/json', Accept: 'application/json' } }
 			).then((response) => {
-				console.error('added a helper to ticket 2')
+				if (this.ticketsFetched === false) {
+					this.tickets.push(response.data)
+				}
 			}).catch(console.error)
 		},
 		fetchTickets() {