From 81ae89697f53be871b3e097c777543c44855cc27 Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sat, 21 Aug 2021 19:32:36 +0200
Subject: [PATCH 01/11] Add Ticket and a HeaderlessKeyValueTable

---
 src/Ticket.vue                             | 23 +++++++++++++++
 src/components/HeaderlessKeyValueTable.vue | 34 ++++++++++++++++++++++
 2 files changed, 57 insertions(+)
 create mode 100644 src/Ticket.vue
 create mode 100644 src/components/HeaderlessKeyValueTable.vue

diff --git a/src/Ticket.vue b/src/Ticket.vue
new file mode 100644
index 0000000..cf55961
--- /dev/null
+++ b/src/Ticket.vue
@@ -0,0 +1,23 @@
+<template>
+	<div class="single-ticket">
+		<h2>Ticket "ding"</h2>
+		<HeaderlessKeyValueTable :data-rows="{Name: 'Bernd', Status: 'Offen', Ablaufdatum: 'gestern'}" />
+		<br>
+		<label for="descripton">Beschreibung</label>
+		<textarea id="descripton" v-model="description" />
+	</div>
+</template>
+
+<script>
+import HeaderlessKeyValueTable from './components/HeaderlessKeyValueTable'
+export default {
+	name: 'Ticket',
+	components: { HeaderlessKeyValueTable },
+}
+</script>
+
+<style scoped>
+textarea {
+	width: 100%;
+}
+</style>
diff --git a/src/components/HeaderlessKeyValueTable.vue b/src/components/HeaderlessKeyValueTable.vue
new file mode 100644
index 0000000..3a7d888
--- /dev/null
+++ b/src/components/HeaderlessKeyValueTable.vue
@@ -0,0 +1,34 @@
+<template>
+	<table>
+		<tr v-for="(value, key, i) in dataRows" :key="i">
+			<td>{{ key }}</td>
+			<td>{{ value }}</td>
+		</tr>
+	</table>
+</template>
+
+<script>
+export default {
+	name: 'HeaderlessKeyValueTable',
+	props: {
+		dataRows: {
+			type: Object,
+			default: () => ({}),
+		},
+	},
+}
+</script>
+
+<style scoped>
+	table {
+		width: 100%;
+	}
+
+	td {
+		border-bottom: solid #000 1px;
+	}
+
+	tr:nth-child(2n) {
+		background: #f0f0f0;
+	}
+</style>

From 3e9484484d6a163a23da36f75672f079927acddf Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sat, 21 Aug 2021 19:38:46 +0200
Subject: [PATCH 02/11] Add save button

---
 src/Ticket.vue | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/Ticket.vue b/src/Ticket.vue
index cf55961..7617172 100644
--- a/src/Ticket.vue
+++ b/src/Ticket.vue
@@ -5,6 +5,9 @@
 		<br>
 		<label for="descripton">Beschreibung</label>
 		<textarea id="descripton" v-model="description" />
+		<br>
+		<button>Speichern</button>
+		<hr>
 	</div>
 </template>
 

From 2538c745a7a1a814bfb39f27c557d0a51a8113f2 Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sat, 21 Aug 2021 19:58:03 +0200
Subject: [PATCH 03/11] Add save event listener to save button

---
 src/Ticket.vue | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/src/Ticket.vue b/src/Ticket.vue
index 7617172..4c7c9d5 100644
--- a/src/Ticket.vue
+++ b/src/Ticket.vue
@@ -3,10 +3,12 @@
 		<h2>Ticket "ding"</h2>
 		<HeaderlessKeyValueTable :data-rows="{Name: 'Bernd', Status: 'Offen', Ablaufdatum: 'gestern'}" />
 		<br>
-		<label for="descripton">Beschreibung</label>
-		<textarea id="descripton" v-model="description" />
+		<label for="description">Beschreibung</label>
+		<textarea id="description" v-model.trim.lazy="description" />
 		<br>
-		<button>Speichern</button>
+		<button @click="save">
+			Speichern
+		</button>
 		<hr>
 	</div>
 </template>
@@ -16,6 +18,14 @@ import HeaderlessKeyValueTable from './components/HeaderlessKeyValueTable'
 export default {
 	name: 'Ticket',
 	components: { HeaderlessKeyValueTable },
+	data() {
+		return {
+			description: '',
+		}
+	},
+	methods: {
+		save() {},
+	},
 }
 </script>
 

From 8631a08e5e09fbe5a60615d30404d4b607eea75a Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sat, 21 Aug 2021 20:04:05 +0200
Subject: [PATCH 04/11] Remove crap

---
 src/App.vue | 195 +---------------------------------------------------
 1 file changed, 3 insertions(+), 192 deletions(-)

diff --git a/src/App.vue b/src/App.vue
index dab9b22..3611918 100644
--- a/src/App.vue
+++ b/src/App.vue
@@ -1,215 +1,26 @@
 <template>
 	<div id="content" class="app-upschooling">
 		<AppNavigation>
-			<AppNavigationNew v-if="!loading"
-				:text="t('upschooling', 'New note')"
-				:disabled="false"
-				button-id="new-upschooling-button"
-				button-class="icon-add"
-				@click="newNote" />
-			<ul>
-				<AppNavigationItem v-for="note in notes"
-					:key="note.id"
-					:title="note.title ? note.title : t('upschooling', 'New note')"
-					:class="{active: currentNoteId === note.id}"
-					@click="openNote(note)">
-					<template slot="actions">
-						<ActionButton v-if="note.id === -1"
-							icon="icon-close"
-							@click="cancelNewNote(note)">
-							{{ t('upschooling', 'Cancel note creation') }}
-						</ActionButton>
-						<ActionButton v-else
-							icon="icon-delete"
-							@click="deleteNote(note)">
-							{{ t('upschooling', 'Delete note') }}
-						</ActionButton>
-					</template>
-				</AppNavigationItem>
-			</ul>
 		</AppNavigation>
 		<AppContent>
-			<div v-if="currentNote">
-				<input ref="title"
-					v-model="currentNote.title"
-					type="text"
-					:disabled="updating">
-				<textarea ref="content" v-model="currentNote.content" :disabled="updating" />
-				<input type="button"
-					class="primary"
-					:value="t('upschooling', 'Save')"
-					:disabled="updating || !savePossible"
-					@click="saveNote">
-			</div>
-			<div v-else id="emptycontent">
-				<div class="icon-file" />
-				<h2>{{ t('upschooling', 'Create a note to get started') }}</h2>
-			</div>
+			<Ticket />
 		</AppContent>
 	</div>
 </template>
 
 <script>
-import ActionButton from '@nextcloud/vue/dist/Components/ActionButton'
 import AppContent from '@nextcloud/vue/dist/Components/AppContent'
 import AppNavigation from '@nextcloud/vue/dist/Components/AppNavigation'
-import AppNavigationItem from '@nextcloud/vue/dist/Components/AppNavigationItem'
-import AppNavigationNew from '@nextcloud/vue/dist/Components/AppNavigationNew'
 
 import '@nextcloud/dialogs/styles/toast.scss'
-import { generateUrl } from '@nextcloud/router'
-import { showError, showSuccess } from '@nextcloud/dialogs'
-import axios from '@nextcloud/axios'
+import Ticket from './Ticket'
 
 export default {
 	name: 'App',
 	components: {
-		ActionButton,
+		Ticket,
 		AppContent,
 		AppNavigation,
-		AppNavigationItem,
-		AppNavigationNew,
-	},
-	data() {
-		return {
-			notes: [],
-			currentNoteId: null,
-			updating: false,
-			loading: true,
-		}
-	},
-	computed: {
-		/**
-		 * Return the currently selected note object
-		 * @returns {Object|null}
-		 */
-		currentNote() {
-			if (this.currentNoteId === null) {
-				return null
-			}
-			return this.notes.find((note) => note.id === this.currentNoteId)
-		},
-
-		/**
-		 * Returns true if a note is selected and its title is not empty
-		 * @returns {Boolean}
-		 */
-		savePossible() {
-			return this.currentNote && this.currentNote.title !== ''
-		},
-	},
-	/**
-	 * Fetch list of notes when the component is loaded
-	 */
-	async mounted() {
-		try {
-			const response = await axios.get(generateUrl('/apps/upschooling/notes'))
-			this.notes = response.data
-		} catch (e) {
-			console.error(e)
-			showError(t('upschooling', 'Could not fetch notes'))
-		}
-		this.loading = false
-	},
-
-	methods: {
-		/**
-		 * Create a new note and focus the note content field automatically
-		 * @param {Object} note Note object
-		 */
-		openNote(note) {
-			if (this.updating) {
-				return
-			}
-			this.currentNoteId = note.id
-			this.$nextTick(() => {
-				this.$refs.content.focus()
-			})
-		},
-		/**
-		 * Action tiggered when clicking the save button
-		 * create a new note or save
-		 */
-		saveNote() {
-			if (this.currentNoteId === -1) {
-				this.createNote(this.currentNote)
-			} else {
-				this.updateNote(this.currentNote)
-			}
-		},
-		/**
-		 * Create a new note and focus the note content field automatically
-		 * The note is not yet saved, therefore an id of -1 is used until it
-		 * has been persisted in the backend
-		 */
-		newNote() {
-			if (this.currentNoteId !== -1) {
-				this.currentNoteId = -1
-				this.notes.push({
-					id: -1,
-					title: '',
-					content: '',
-				})
-				this.$nextTick(() => {
-					this.$refs.title.focus()
-				})
-			}
-		},
-		/**
-		 * Abort creating a new note
-		 */
-		cancelNewNote() {
-			this.notes.splice(this.notes.findIndex((note) => note.id === -1), 1)
-			this.currentNoteId = null
-		},
-		/**
-		 * Create a new note by sending the information to the server
-		 * @param {Object} note Note object
-		 */
-		async createNote(note) {
-			this.updating = true
-			try {
-				const response = await axios.post(generateUrl('/apps/upschooling/notes'), note)
-				const index = this.notes.findIndex((match) => match.id === this.currentNoteId)
-				this.$set(this.notes, index, response.data)
-				this.currentNoteId = response.data.id
-			} catch (e) {
-				console.error(e)
-				showError(t('upschooling', 'Could not create the note'))
-			}
-			this.updating = false
-		},
-		/**
-		 * Update an existing note on the server
-		 * @param {Object} note Note object
-		 */
-		async updateNote(note) {
-			this.updating = true
-			try {
-				await axios.put(generateUrl(`/apps/upschooling/notes/${note.id}`), note)
-			} catch (e) {
-				console.error(e)
-				showError(t('upschooling', 'Could not update the note'))
-			}
-			this.updating = false
-		},
-		/**
-		 * Delete a note, remove it from the frontend and show a hint
-		 * @param {Object} note Note object
-		 */
-		async deleteNote(note) {
-			try {
-				await axios.delete(generateUrl(`/apps/upschooling/notes/${note.id}`))
-				this.notes.splice(this.notes.indexOf(note), 1)
-				if (this.currentNoteId === note.id) {
-					this.currentNoteId = null
-				}
-				showSuccess(t('upschooling', 'Note deleted'))
-			} catch (e) {
-				console.error(e)
-				showError(t('upschooling', 'Could not delete the note'))
-			}
-		},
 	},
 }
 </script>

From 3440797555e26c5f8a0aab1d324213b4b7a61e44 Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sun, 22 Aug 2021 16:23:54 +0200
Subject: [PATCH 05/11] add header prop and handling of that

---
 src/components/HeaderlessKeyValueTable.vue | 34 ------------
 src/components/KeyValueTable.vue           | 63 ++++++++++++++++++++++
 2 files changed, 63 insertions(+), 34 deletions(-)
 delete mode 100644 src/components/HeaderlessKeyValueTable.vue
 create mode 100644 src/components/KeyValueTable.vue

diff --git a/src/components/HeaderlessKeyValueTable.vue b/src/components/HeaderlessKeyValueTable.vue
deleted file mode 100644
index 3a7d888..0000000
--- a/src/components/HeaderlessKeyValueTable.vue
+++ /dev/null
@@ -1,34 +0,0 @@
-<template>
-	<table>
-		<tr v-for="(value, key, i) in dataRows" :key="i">
-			<td>{{ key }}</td>
-			<td>{{ value }}</td>
-		</tr>
-	</table>
-</template>
-
-<script>
-export default {
-	name: 'HeaderlessKeyValueTable',
-	props: {
-		dataRows: {
-			type: Object,
-			default: () => ({}),
-		},
-	},
-}
-</script>
-
-<style scoped>
-	table {
-		width: 100%;
-	}
-
-	td {
-		border-bottom: solid #000 1px;
-	}
-
-	tr:nth-child(2n) {
-		background: #f0f0f0;
-	}
-</style>
diff --git a/src/components/KeyValueTable.vue b/src/components/KeyValueTable.vue
new file mode 100644
index 0000000..8b90399
--- /dev/null
+++ b/src/components/KeyValueTable.vue
@@ -0,0 +1,63 @@
+<template>
+	<table>
+		<thead>
+			<tr v-for="(value, key, i) in getHeaders" :key="i">
+				<th>{{ key }}</th>
+				<th>{{ value }}</th>
+			</tr>
+		</thead>
+		<tbody>
+			<tr v-for="(value, key, i) in dataRows" :key="i+1">
+				<td>{{ key }}</td>
+				<td>{{ value }}</td>
+			</tr>
+		</tbody>
+	</table>
+</template>
+
+<script>
+export default {
+	name: 'KeyValueTable',
+	props: {
+		dataRows: {
+			type: Object,
+			default: () => ({}),
+		},
+		header: {
+			type: Object,
+			default: () => ({}),
+		},
+	},
+	computed: {
+		getHeaders() {
+			const key = Object.keys(this.header)[0]
+			const value = this.header[key]
+			if (key && value) return { [key]: value }
+
+			return {}
+		},
+	},
+}
+</script>
+
+<style scoped>
+	table {
+		width: 100%;
+	}
+
+	td, th {
+		border-bottom: solid #000 1px;
+	}
+
+	th {
+		font-weight: bold;
+	}
+
+	thead {
+		background: #f0f0f0;
+	}
+
+	tbody tr:nth-child(2n) {
+		background: #f0f0f0;
+	}
+</style>

From 347181f21a22717ccfdb3f7750cb107a3ab09ee4 Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sun, 22 Aug 2021 16:24:55 +0200
Subject: [PATCH 06/11] fix docker-run

---
 docker-run.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docker-run.sh b/docker-run.sh
index 0ef8a1e..decf9a6 100755
--- a/docker-run.sh
+++ b/docker-run.sh
@@ -8,7 +8,7 @@ docker rm -f nextcloud
 docker run -d --name=nextcloud -p 8080:80 -v "${PWD}:/var/www/html/custom_apps/upschooling" docker.io/nextcloud
 docker exec nextcloud chown -R 33 /var/www/html/custom_apps
 docker exec nextcloud chmod -R ug+rw /var/www/html/custom_apps
-docker exec --user 33 nextcloud bash -c 'cd /var/www/html/custom_apps/upschooling && make'
+docker exec --user 33 nextcloud bash -c 'cd /var/www/html/custom_apps/upschooling && make composer'
 docker exec --user 33 nextcloud php occ maintenance:install --database "sqlite" --admin-user "admin" --admin-pass "admin"
 docker exec --user 33 nextcloud php occ config:system:set --value=true --type=boolean debug
 docker exec --user 33 nextcloud php occ app:enable --force upschooling

From d88bd657ef5f3c53aeb788e82b4dbc5973d0748a Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sun, 22 Aug 2021 16:37:12 +0200
Subject: [PATCH 07/11] finish ticket page looks wise

---
 src/Ticket.vue | 33 ++++++++++++++++++++++++++++-----
 1 file changed, 28 insertions(+), 5 deletions(-)

diff --git a/src/Ticket.vue b/src/Ticket.vue
index 4c7c9d5..9afe299 100644
--- a/src/Ticket.vue
+++ b/src/Ticket.vue
@@ -1,23 +1,32 @@
 <template>
 	<div class="single-ticket">
+		<div class="header-bar">
+			<button>
+				{{ t('upschooling', 'Ticket Schließen') }}
+			</button>
+			<button>
+				{{ t('upschooling', 'Speichern') }}
+			</button>
+		</div>
 		<h2>Ticket "ding"</h2>
-		<HeaderlessKeyValueTable :data-rows="{Name: 'Bernd', Status: 'Offen', Ablaufdatum: 'gestern'}" />
+		<KeyValueTable :data-rows="{Name: 'Bernd', Status: 'Offen', Ablaufdatum: 'gestern'}" />
 		<br>
-		<label for="description">Beschreibung</label>
-		<textarea id="description" v-model.trim.lazy="description" />
+		<label for="description">{{ t('upschooling', 'Beschreibung') }}</label>
+		<textarea id="description" v-model.trim.lazy="description" rows="15" />
 		<br>
 		<button @click="save">
 			Speichern
 		</button>
 		<hr>
+		<div class="placeholder" />
 	</div>
 </template>
 
 <script>
-import HeaderlessKeyValueTable from './components/HeaderlessKeyValueTable'
+import KeyValueTable from './components/KeyValueTable'
 export default {
 	name: 'Ticket',
-	components: { HeaderlessKeyValueTable },
+	components: { KeyValueTable },
 	data() {
 		return {
 			description: '',
@@ -32,5 +41,19 @@ export default {
 <style scoped>
 textarea {
 	width: 100%;
+	margin: 0;
+	resize: vertical;
+}
+
+.placeholder {
+	height: 400px;
+	width: 100%;
+	background: #f0f0f0;
+}
+
+.header-bar {
+	display: flex;
+	width: 100%;
+	flex-direction: row-reverse;
 }
 </style>

From 2252a494675b4af4131b359ba81a819a76a3008b Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sun, 22 Aug 2021 16:37:16 +0200
Subject: [PATCH 08/11] finish ticket page looks wise

---
 src/App.vue | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/App.vue b/src/App.vue
index 3611918..b0b223d 100644
--- a/src/App.vue
+++ b/src/App.vue
@@ -1,7 +1,6 @@
 <template>
 	<div id="content" class="app-upschooling">
-		<AppNavigation>
-		</AppNavigation>
+		<AppNavigation />
 		<AppContent>
 			<Ticket />
 		</AppContent>

From 5f45d0a3c827f2621a859a9225fb0c8f3713e25d Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sun, 22 Aug 2021 17:02:03 +0200
Subject: [PATCH 09/11] rename note to ticket

---
 .../{NoteApiController.php => TicketApiController.php}        | 2 +-
 lib/Controller/{NoteController.php => TicketController.php}   | 2 +-
 tests/Integration/NoteIntegrationTest.php                     | 4 ++--
 tests/Unit/Controller/NoteApiControllerTest.php               | 4 ++--
 tests/Unit/Controller/NoteControllerTest.php                  | 4 ++--
 5 files changed, 8 insertions(+), 8 deletions(-)
 rename lib/Controller/{NoteApiController.php => TicketApiController.php} (97%)
 rename lib/Controller/{NoteController.php => TicketController.php} (97%)

diff --git a/lib/Controller/NoteApiController.php b/lib/Controller/TicketApiController.php
similarity index 97%
rename from lib/Controller/NoteApiController.php
rename to lib/Controller/TicketApiController.php
index 1667bba..48c24bb 100644
--- a/lib/Controller/NoteApiController.php
+++ b/lib/Controller/TicketApiController.php
@@ -8,7 +8,7 @@ use OCP\AppFramework\ApiController;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\IRequest;
 
-class NoteApiController extends ApiController {
+class TicketApiController extends ApiController {
 	/** @var NoteService */
 	private $service;
 
diff --git a/lib/Controller/NoteController.php b/lib/Controller/TicketController.php
similarity index 97%
rename from lib/Controller/NoteController.php
rename to lib/Controller/TicketController.php
index 4c37699..b7a9066 100644
--- a/lib/Controller/NoteController.php
+++ b/lib/Controller/TicketController.php
@@ -8,7 +8,7 @@ use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\IRequest;
 
-class NoteController extends Controller {
+class TicketController extends Controller {
 	/** @var NoteService */
 	private $service;
 
diff --git a/tests/Integration/NoteIntegrationTest.php b/tests/Integration/NoteIntegrationTest.php
index 6c71c42..69a6f8b 100644
--- a/tests/Integration/NoteIntegrationTest.php
+++ b/tests/Integration/NoteIntegrationTest.php
@@ -9,7 +9,7 @@ use PHPUnit\Framework\TestCase;
 
 use OCA\UPschooling\Db\Note;
 use OCA\UPschooling\Db\NoteMapper;
-use OCA\UPschooling\Controller\NoteController;
+use OCA\UPschooling\Controller\TicketController;
 
 class NoteIntegrationTest extends TestCase {
 	private $controller;
@@ -30,7 +30,7 @@ class NoteIntegrationTest extends TestCase {
 			return $this->createMock(IRequest::class);
 		});
 
-		$this->controller = $container->query(NoteController::class);
+		$this->controller = $container->query(TicketController::class);
 		$this->mapper = $container->query(NoteMapper::class);
 	}
 
diff --git a/tests/Unit/Controller/NoteApiControllerTest.php b/tests/Unit/Controller/NoteApiControllerTest.php
index 4c13eb6..83318ce 100644
--- a/tests/Unit/Controller/NoteApiControllerTest.php
+++ b/tests/Unit/Controller/NoteApiControllerTest.php
@@ -2,11 +2,11 @@
 
 namespace OCA\UPschooling\Tests\Unit\Controller;
 
-use OCA\UPschooling\Controller\NoteApiController;
+use OCA\UPschooling\Controller\TicketApiController;
 
 class NoteApiControllerTest extends NoteControllerTest {
 	public function setUp(): void {
 		parent::setUp();
-		$this->controller = new NoteApiController($this->request, $this->service, $this->userId);
+		$this->controller = new TicketApiController($this->request, $this->service, $this->userId);
 	}
 }
diff --git a/tests/Unit/Controller/NoteControllerTest.php b/tests/Unit/Controller/NoteControllerTest.php
index e96d660..aa0603a 100644
--- a/tests/Unit/Controller/NoteControllerTest.php
+++ b/tests/Unit/Controller/NoteControllerTest.php
@@ -9,7 +9,7 @@ use OCP\IRequest;
 
 use OCA\UPschooling\Service\NoteNotFound;
 use OCA\UPschooling\Service\NoteService;
-use OCA\UPschooling\Controller\NoteController;
+use OCA\UPschooling\Controller\TicketController;
 
 class NoteControllerTest extends TestCase {
 	protected $controller;
@@ -22,7 +22,7 @@ class NoteControllerTest extends TestCase {
 		$this->service = $this->getMockBuilder(NoteService::class)
 			->disableOriginalConstructor()
 			->getMock();
-		$this->controller = new NoteController($this->request, $this->service, $this->userId);
+		$this->controller = new TicketController($this->request, $this->service, $this->userId);
 	}
 
 	public function testUpdate() {

From 0e2f6f944d8114e32f372a15f85e2bd84ff47dfb Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sun, 22 Aug 2021 17:03:19 +0200
Subject: [PATCH 10/11] rename note to ticket

---
 lib/Controller/TicketApiController.php             |  6 +++---
 lib/Controller/TicketController.php                |  6 +++---
 lib/Db/{Note.php => Ticket.php}                    |  2 +-
 lib/Db/{NoteMapper.php => TicketMapper.php}        |  8 ++++----
 lib/Service/{NoteService.php => TicketService.php} | 12 ++++++------
 tests/Integration/NoteIntegrationTest.php          | 10 +++++-----
 tests/Unit/Controller/NoteControllerTest.php       |  4 ++--
 tests/Unit/Service/NoteServiceTest.php             | 14 +++++++-------
 8 files changed, 31 insertions(+), 31 deletions(-)
 rename lib/Db/{Note.php => Ticket.php} (83%)
 rename lib/Db/{NoteMapper.php => TicketMapper.php} (85%)
 rename lib/Service/{NoteService.php => TicketService.php} (89%)

diff --git a/lib/Controller/TicketApiController.php b/lib/Controller/TicketApiController.php
index 48c24bb..d2a9d6f 100644
--- a/lib/Controller/TicketApiController.php
+++ b/lib/Controller/TicketApiController.php
@@ -3,13 +3,13 @@
 namespace OCA\UPschooling\Controller;
 
 use OCA\UPschooling\AppInfo\Application;
-use OCA\UPschooling\Service\NoteService;
+use OCA\UPschooling\Service\TicketService;
 use OCP\AppFramework\ApiController;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\IRequest;
 
 class TicketApiController extends ApiController {
-	/** @var NoteService */
+	/** @var TicketService */
 	private $service;
 
 	/** @var string */
@@ -18,7 +18,7 @@ class TicketApiController extends ApiController {
 	use Errors;
 
 	public function __construct(IRequest $request,
-								NoteService $service,
+								TicketService $service,
 								$userId) {
 		parent::__construct(Application::APP_ID, $request);
 		$this->service = $service;
diff --git a/lib/Controller/TicketController.php b/lib/Controller/TicketController.php
index b7a9066..2b3b55f 100644
--- a/lib/Controller/TicketController.php
+++ b/lib/Controller/TicketController.php
@@ -3,13 +3,13 @@
 namespace OCA\UPschooling\Controller;
 
 use OCA\UPschooling\AppInfo\Application;
-use OCA\UPschooling\Service\NoteService;
+use OCA\UPschooling\Service\TicketService;
 use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\IRequest;
 
 class TicketController extends Controller {
-	/** @var NoteService */
+	/** @var TicketService */
 	private $service;
 
 	/** @var string */
@@ -18,7 +18,7 @@ class TicketController extends Controller {
 	use Errors;
 
 	public function __construct(IRequest $request,
-								NoteService $service,
+								TicketService $service,
 								$userId) {
 		parent::__construct(Application::APP_ID, $request);
 		$this->service = $service;
diff --git a/lib/Db/Note.php b/lib/Db/Ticket.php
similarity index 83%
rename from lib/Db/Note.php
rename to lib/Db/Ticket.php
index 36e81a9..26f173d 100644
--- a/lib/Db/Note.php
+++ b/lib/Db/Ticket.php
@@ -6,7 +6,7 @@ use JsonSerializable;
 
 use OCP\AppFramework\Db\Entity;
 
-class Note extends Entity implements JsonSerializable {
+class Ticket extends Entity implements JsonSerializable {
 	protected $title;
 	protected $content;
 	protected $userId;
diff --git a/lib/Db/NoteMapper.php b/lib/Db/TicketMapper.php
similarity index 85%
rename from lib/Db/NoteMapper.php
rename to lib/Db/TicketMapper.php
index bd3ae2a..1f985b2 100644
--- a/lib/Db/NoteMapper.php
+++ b/lib/Db/TicketMapper.php
@@ -8,19 +8,19 @@ use OCP\AppFramework\Db\QBMapper;
 use OCP\DB\QueryBuilder\IQueryBuilder;
 use OCP\IDBConnection;
 
-class NoteMapper extends QBMapper {
+class TicketMapper extends QBMapper {
 	public function __construct(IDBConnection $db) {
-		parent::__construct($db, 'upschooling', Note::class);
+		parent::__construct($db, 'upschooling', Ticket::class);
 	}
 
 	/**
 	 * @param int $id
 	 * @param string $userId
-	 * @return Entity|Note
+	 * @return Entity|Ticket
 	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException
 	 * @throws DoesNotExistException
 	 */
-	public function find(int $id, string $userId): Note {
+	public function find(int $id, string $userId): Ticket {
 		/* @var $qb IQueryBuilder */
 		$qb = $this->db->getQueryBuilder();
 		$qb->select('*')
diff --git a/lib/Service/NoteService.php b/lib/Service/TicketService.php
similarity index 89%
rename from lib/Service/NoteService.php
rename to lib/Service/TicketService.php
index 8f6e167..b2966f3 100644
--- a/lib/Service/NoteService.php
+++ b/lib/Service/TicketService.php
@@ -7,15 +7,15 @@ use Exception;
 use OCP\AppFramework\Db\DoesNotExistException;
 use OCP\AppFramework\Db\MultipleObjectsReturnedException;
 
-use OCA\UPschooling\Db\Note;
-use OCA\UPschooling\Db\NoteMapper;
+use OCA\UPschooling\Db\Ticket;
+use OCA\UPschooling\Db\TicketMapper;
 
-class NoteService {
+class TicketService {
 
-	/** @var NoteMapper */
+	/** @var TicketMapper */
 	private $mapper;
 
-	public function __construct(NoteMapper $mapper) {
+	public function __construct(TicketMapper $mapper) {
 		$this->mapper = $mapper;
 	}
 
@@ -46,7 +46,7 @@ class NoteService {
 	}
 
 	public function create($title, $content, $userId) {
-		$note = new Note();
+		$note = new Ticket();
 		$note->setTitle($title);
 		$note->setContent($content);
 		$note->setUserId($userId);
diff --git a/tests/Integration/NoteIntegrationTest.php b/tests/Integration/NoteIntegrationTest.php
index 69a6f8b..1a73d07 100644
--- a/tests/Integration/NoteIntegrationTest.php
+++ b/tests/Integration/NoteIntegrationTest.php
@@ -7,8 +7,8 @@ use OCP\IRequest;
 use PHPUnit\Framework\TestCase;
 
 
-use OCA\UPschooling\Db\Note;
-use OCA\UPschooling\Db\NoteMapper;
+use OCA\UPschooling\Db\Ticket;
+use OCA\UPschooling\Db\TicketMapper;
 use OCA\UPschooling\Controller\TicketController;
 
 class NoteIntegrationTest extends TestCase {
@@ -31,12 +31,12 @@ class NoteIntegrationTest extends TestCase {
 		});
 
 		$this->controller = $container->query(TicketController::class);
-		$this->mapper = $container->query(NoteMapper::class);
+		$this->mapper = $container->query(TicketMapper::class);
 	}
 
 	public function testUpdate() {
 		// create a new note that should be updated
-		$note = new Note();
+		$note = new Ticket();
 		$note->setTitle('old_title');
 		$note->setContent('old_content');
 		$note->setUserId($this->userId);
@@ -44,7 +44,7 @@ class NoteIntegrationTest extends TestCase {
 		$id = $this->mapper->insert($note)->getId();
 
 		// fromRow does not set the fields as updated
-		$updatedNote = Note::fromRow([
+		$updatedNote = Ticket::fromRow([
 			'id' => $id,
 			'user_id' => $this->userId
 		]);
diff --git a/tests/Unit/Controller/NoteControllerTest.php b/tests/Unit/Controller/NoteControllerTest.php
index aa0603a..bf4950a 100644
--- a/tests/Unit/Controller/NoteControllerTest.php
+++ b/tests/Unit/Controller/NoteControllerTest.php
@@ -8,7 +8,7 @@ use OCP\AppFramework\Http;
 use OCP\IRequest;
 
 use OCA\UPschooling\Service\NoteNotFound;
-use OCA\UPschooling\Service\NoteService;
+use OCA\UPschooling\Service\TicketService;
 use OCA\UPschooling\Controller\TicketController;
 
 class NoteControllerTest extends TestCase {
@@ -19,7 +19,7 @@ class NoteControllerTest extends TestCase {
 
 	public function setUp(): void {
 		$this->request = $this->getMockBuilder(IRequest::class)->getMock();
-		$this->service = $this->getMockBuilder(NoteService::class)
+		$this->service = $this->getMockBuilder(TicketService::class)
 			->disableOriginalConstructor()
 			->getMock();
 		$this->controller = new TicketController($this->request, $this->service, $this->userId);
diff --git a/tests/Unit/Service/NoteServiceTest.php b/tests/Unit/Service/NoteServiceTest.php
index 416a079..51af8c5 100644
--- a/tests/Unit/Service/NoteServiceTest.php
+++ b/tests/Unit/Service/NoteServiceTest.php
@@ -7,9 +7,9 @@ use PHPUnit\Framework\TestCase;
 
 use OCP\AppFramework\Db\DoesNotExistException;
 
-use OCA\UPschooling\Db\Note;
-use OCA\UPschooling\Service\NoteService;
-use OCA\UPschooling\Db\NoteMapper;
+use OCA\UPschooling\Db\Ticket;
+use OCA\UPschooling\Service\TicketService;
+use OCA\UPschooling\Db\TicketMapper;
 
 class NoteServiceTest extends TestCase {
 	private $service;
@@ -17,15 +17,15 @@ class NoteServiceTest extends TestCase {
 	private $userId = 'john';
 
 	public function setUp(): void {
-		$this->mapper = $this->getMockBuilder(NoteMapper::class)
+		$this->mapper = $this->getMockBuilder(TicketMapper::class)
 			->disableOriginalConstructor()
 			->getMock();
-		$this->service = new NoteService($this->mapper);
+		$this->service = new TicketService($this->mapper);
 	}
 
 	public function testUpdate() {
 		// the existing note
-		$note = Note::fromRow([
+		$note = Ticket::fromRow([
 			'id' => 3,
 			'title' => 'yo',
 			'content' => 'nope'
@@ -36,7 +36,7 @@ class NoteServiceTest extends TestCase {
 			->will($this->returnValue($note));
 
 		// the note when updated
-		$updatedNote = Note::fromRow(['id' => 3]);
+		$updatedNote = Ticket::fromRow(['id' => 3]);
 		$updatedNote->setTitle('title');
 		$updatedNote->setContent('content');
 		$this->mapper->expects($this->once())

From 29322a4e9c2199f588c6b5d0b0e70b0c3c2f922e Mon Sep 17 00:00:00 2001
From: Finn <finn@hering.sh>
Date: Sun, 22 Aug 2021 17:10:23 +0200
Subject: [PATCH 11/11] Change ticket entry to contain the data needed

---
 lib/Db/Ticket.php | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/lib/Db/Ticket.php b/lib/Db/Ticket.php
index 26f173d..04c0943 100644
--- a/lib/Db/Ticket.php
+++ b/lib/Db/Ticket.php
@@ -8,14 +8,20 @@ use OCP\AppFramework\Db\Entity;
 
 class Ticket extends Entity implements JsonSerializable {
 	protected $title;
-	protected $content;
-	protected $userId;
+	protected $description;
+	protected $helperId;
+	protected $creatorId; // The ID of the person who created the Ticked. Usually the person who needs help.
+	protected $dueDate;
+	protected $status;
 
 	public function jsonSerialize(): array {
 		return [
 			'id' => $this->id,
 			'title' => $this->title,
-			'content' => $this->content
+			'description' => $this->description,
+            'helperId' => $this->helperId,
+            'dueDate' => $this->dueDate,
+            'status' => $this->status,
 		];
 	}
 }