Merge pull request #267 from tobiasKaminsky/improvements

Whitespaces / Sort
tobiasKaminsky · Sep 23, 2024 · d5d420a · d5d420a
2 parents 1496332 + 5fb15e0
commit d5d420a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/lib/Controller/GroceryListController.php b/lib/Controller/GroceryListController.php
@@ -255,8 +255,8 @@ public function addItem(string $name, string $quantity, int $category, int $list
 	public function updateItem(int $id, string $name, string $quantity, int $category)
 	{
 		$item = $this->itemMapper->find($id);
-		$item->setName($name);
-		$item->setQuantity($quantity);
+		$item->setName(trim($name));
+		$item->setQuantity(trim($quantity));
 		$item->setCategory($category);
 
 		return new DataResponse($this->itemMapper->update($item));

diff --git a/src/views/GroceryList.vue b/src/views/GroceryList.vue
@@ -86,7 +86,7 @@
 		<div>
 			<div v-if="hideCategory">
 				<ul class="list-item">
-					<ListItem v-for="item in allItems"
+					<ListItem v-for="item in sortedItems"
 						:key="item.id"
 						:item="item"
 						@edit="() => editItem(item)"
@@ -197,6 +197,11 @@ export default {
 		allItems() {
 			return this.items
 		},
+    sortedItems() {
+      return this.items.sort((a, b) => {
+        return a.name.toLowerCase().localeCompare(b.name.toLowerCase())
+      })
+    },
 		filteredCategories() {
 			if (this.categories == null) {
 				return