From 44cd26ae77cdb8ea3e5b1dc7abc2110c2c7e16f5 Mon Sep 17 00:00:00 2001
From: Bernd Bestel <bernd@berrnd.de>
Date: Sun, 30 Sep 2018 23:22:17 +0200
Subject: [PATCH] Finish first early version of "pictures for products"
 (references #58)

---
 migrations/0040.sql            |  2 ++
 public/js/grocy.js             | 36 ++++++++++++++++++++++++++
 public/viewjs/productform.js   | 46 +++++++++++++++++++++++++++++++---
 public/viewjs/stockoverview.js | 11 ++++++++
 views/productform.blade.php    | 10 ++++++++
 views/stockoverview.blade.php  |  5 ++++
 6 files changed, 106 insertions(+), 4 deletions(-)
 create mode 100644 migrations/0040.sql

diff --git a/migrations/0040.sql b/migrations/0040.sql
new file mode 100644
index 00000000..8abf173f
--- /dev/null
+++ b/migrations/0040.sql
@@ -0,0 +1,2 @@
+ALTER TABLE products
+ADD picture_file_name TEXT;
diff --git a/public/js/grocy.js b/public/js/grocy.js
index 4aa2acc7..3b21527d 100644
--- a/public/js/grocy.js
+++ b/public/js/grocy.js
@@ -177,6 +177,42 @@ Grocy.Api.Post = function(apiFunction, jsonData, success, error)
 	xhr.send(JSON.stringify(jsonData));
 };
 
+Grocy.Api.UploadFile = function(fileInput, group, success, error)
+{
+	if (fileInput[0].files.length === 0)
+	{
+		return;
+	}
+
+	var xhr = new XMLHttpRequest();
+	var url = U('/api/files/upload/' + group + '?file_name=' + encodeURIComponent(fileInput[0].files[0].name));
+
+	xhr.onreadystatechange = function()
+	{
+		if (xhr.readyState === XMLHttpRequest.DONE)
+		{
+			if (xhr.status === 200)
+			{
+				if (success)
+				{
+					success(JSON.parse(xhr.responseText));
+				}
+			}
+			else
+			{
+				if (error)
+				{
+					error(xhr);
+				}
+			}
+		}
+	};
+
+	xhr.open('POST', url, true);
+	xhr.setRequestHeader('Content-type', 'application/octet-stream');
+	xhr.send(fileInput[0].files[0]);
+};
+
 Grocy.FrontendHelpers = { };
 Grocy.FrontendHelpers.ValidateForm = function(formId)
 {
diff --git a/public/viewjs/productform.js b/public/viewjs/productform.js
index d9a053d3..59a2ce1a 100644
--- a/public/viewjs/productform.js
+++ b/public/viewjs/productform.js
@@ -9,12 +9,34 @@
 		redirectDestination = returnTo + '?createdproduct=' + encodeURIComponent($('#name').val());
 	}
 
+	var jsonData = $('#product-form').serializeJSON();
+	if ($("#product-picture")[0].files.length > 0)
+	{
+		jsonData.picture_file_name = $("#product-picture")[0].files[0].name;
+	}
+
 	if (Grocy.EditMode === 'create')
 	{
-		Grocy.Api.Post('add-object/products', $('#product-form').serializeJSON(),
+		Grocy.Api.Post('add-object/products', jsonData,
 			function(result)
 			{
-				window.location.href = redirectDestination;
+				if (jsonData.hasOwnProperty("picture_file_name"))
+				{
+					Grocy.Api.UploadFile($("#product-picture"), 'productpictures',
+						function(result)
+						{
+							window.location.href = redirectDestination;
+						},
+						function(xhr)
+						{
+							Grocy.FrontendHelpers.ShowGenericError('Error while saving, probably this item already exists', xhr.response)
+						}
+					);
+				}
+				else
+				{
+					window.location.href = redirectDestination;
+				}
 			},
 			function(xhr)
 			{
@@ -24,10 +46,26 @@
 	}
 	else
 	{
-		Grocy.Api.Post('edit-object/products/' + Grocy.EditObjectId, $('#product-form').serializeJSON(),
+		Grocy.Api.Post('edit-object/products/' + Grocy.EditObjectId, jsonData,
 			function(result)
 			{
-				window.location.href = redirectDestination;
+				if (jsonData.hasOwnProperty("picture_file_name"))
+				{
+					Grocy.Api.UploadFile($("#product-picture"), 'productpictures',
+						function(result)
+						{
+							window.location.href = redirectDestination;
+						},
+						function(xhr)
+						{
+							Grocy.FrontendHelpers.ShowGenericError('Error while saving, probably this item already exists', xhr.response)
+						}
+					);
+				}
+				else
+				{
+					window.location.href = redirectDestination;
+				}
 			},
 			function(xhr)
 			{
diff --git a/public/viewjs/stockoverview.js b/public/viewjs/stockoverview.js
index ddc3f9df..06c1772e 100644
--- a/public/viewjs/stockoverview.js
+++ b/public/viewjs/stockoverview.js
@@ -141,6 +141,17 @@ $(document).on('click', '.product-consume-button', function(e)
 	);
 });
 
+$(document).on("click", ".show-product-picture-button", function(e)
+{
+	var pictureUrl = $(e.currentTarget).attr("data-picture-url");
+	var productName = $(e.currentTarget).attr("data-product-name");
+
+	bootbox.alert({
+		title: L("Image of product #1", productName),
+		message: "<img src='" + pictureUrl + "' class='img-fluid'>"
+	});
+});
+
 function RefreshStatistics()
 {
 	Grocy.Api.Get('stock/get-current-stock',
diff --git a/views/productform.blade.php b/views/productform.blade.php
index fceec874..c062ca43 100644
--- a/views/productform.blade.php
+++ b/views/productform.blade.php
@@ -108,6 +108,16 @@
 				'additionalHtmlElements' => '<p id="qu-conversion-info" class="form-text text-muted small d-none"></p>'
 			))
 
+			<div class="form-group">
+				<label for="product-picture">{{ $L('Product picture') }}</label>
+				<input type="file" class="form-control-file" id="product-picture" accept="image/*">
+				
+				@if(!empty($product->picture_file_name))
+					<label class="mt-2">{{ $L('Current picture') }}</label>
+					<img src="{{ $U('/api/files/get/productpictures?file_name=' . $product->picture_file_name) }}" class="img-fluid">
+				@endif
+			</div>
+
 			<button id="save-product-button" class="btn btn-success">{{ $L('Save') }}</button>
 		</form>
 	</div>
diff --git a/views/stockoverview.blade.php b/views/stockoverview.blade.php
index 362ffcea..d2b57afa 100644
--- a/views/stockoverview.blade.php
+++ b/views/stockoverview.blade.php
@@ -74,6 +74,11 @@
 							data-consume-amount="{{ $currentStockEntry->amount }}">
 							<i class="fas fa-utensils"></i> {{ $L('All') }}
 						</a>
+						<a id="product-{{ $currentStockEntry->product_id }}-show-product-picture-button" class="btn btn-info btn-sm show-product-picture-button @if(empty(FindObjectInArrayByPropertyValue($products, 'id', $currentStockEntry->product_id)->picture_file_name)) disabled @endif" href="#"
+							data-picture-url="{{ $U('/api/files/get/productpictures?file_name=' . FindObjectInArrayByPropertyValue($products, 'id', $currentStockEntry->product_id)->picture_file_name) }}"
+							data-product-name="{{ FindObjectInArrayByPropertyValue($products, 'id', $currentStockEntry->product_id)->name }}">
+							<i class="fas fa-image"></i>
+						</a>
 					</td>
 					<td>
 						{{ FindObjectInArrayByPropertyValue($products, 'id', $currentStockEntry->product_id)->name }}