X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Fmain%2Fresources%2Ftemplates%2FimageBrowser.html;h=a8a33c9e3a57b0df84b5d58d5b0f6ed9cb49923a;hp=da8d032301cbb3a9af1aee079e1ac68e7113605b;hb=faf66247a34f64946990a985d2ea3003465969cb;hpb=629ddb006542df2b671e172d8f544815bbab639b

diff --git a/src/main/resources/templates/imageBrowser.html b/src/main/resources/templates/imageBrowser.html
index da8d032..a8a33c9 100644
--- a/src/main/resources/templates/imageBrowser.html
+++ b/src/main/resources/templates/imageBrowser.html
@@ -2,7 +2,7 @@
 
 	<div class="page-id hidden">image-browser</div>
 
-	<script language="javascript">
+	<script type="application/javascript">
 
 		/* hide all those forms. */
 		function hideAndShowBlock(blockElement, clickToShowElement, clickToHideElement) {
@@ -21,7 +21,7 @@
 		}
 
 		/* ID of the image currently being edited. */
-		var editingImageId = null;
+		let editingImageId = null;
 
 		/**
 		 * Shows the form for editing an image.
@@ -29,7 +29,7 @@
 		 * @param imageId The ID of the image to edit.
 		 */
 		function editImage(imageId) {
-			if (editingImageId != imageId) {
+			if (editingImageId !== imageId) {
 				cancelImageEditing();
 			} else {
 				return;
@@ -38,7 +38,7 @@
 			$(".show-data", getImage(imageId)).hide();
 			$(".edit-data", getImage(imageId)).show();
 			$(document).bind("click.sone", function(event) {
-				if ($(event.target).closest("#image-" + imageId).size() == 0) {
+				if ($(event.target).closest("#image-" + imageId).size() === 0) {
 					cancelImageEditing();
 				}
 			});
@@ -74,21 +74,21 @@
 		 * @param destinationId The ID of the destionation image
 		 */
 		function swapImage(sourceId, destinationId) {
-			sourceElement = getImage(sourceId);
-			destinationElement = getImage(destinationId);
-			sourceParent = sourceElement.closest(".image-row");
-			sourcePrevSibling = sourceElement.prev();
+			const sourceElement = getImage(sourceId);
+			const destinationElement = getImage(destinationId);
+			const sourceParent = sourceElement.closest(".image-row");
+			const sourcePrevSibling = sourceElement.prev();
 			sourceElement.detach();
 			destinationElement.before(sourceElement);
-			if (sourcePrevSibling.get(0) != destinationElement.get(0)) {
+			if (sourcePrevSibling.get(0) !== destinationElement.get(0)) {
 				destinationElement.detach();
 				(sourcePrevSibling.size() > 0) ? sourcePrevSibling.after(destinationElement) : sourceParent.prepend(destinationElement);
 			}
-			if ($("button[name='moveLeft']", sourceElement).hasClass("hidden") != $("button[name='moveLeft']", destinationElement).hasClass("hidden")) {
+			if ($("button[name='moveLeft']", sourceElement).hasClass("hidden") !== $("button[name='moveLeft']", destinationElement).hasClass("hidden")) {
 				$("button[name='moveLeft']", sourceElement).toggleClass("hidden");
 				$("button[name='moveLeft']", destinationElement).toggleClass("hidden");
 			}
-			if ($("button[name='moveRight']", sourceElement).hasClass("hidden") != $("button[name='moveRight']", destinationElement).hasClass("hidden")) {
+			if ($("button[name='moveRight']", sourceElement).hasClass("hidden") !== $("button[name='moveRight']", destinationElement).hasClass("hidden")) {
 				$("button[name='moveRight']", sourceElement).toggleClass("hidden");
 				$("button[name='moveRight']", destinationElement).toggleClass("hidden");
 			}
@@ -99,13 +99,13 @@
 		 */
 		function prepareImages() {
 			$(".image").each(function() {
-				imageId = $(this).closest(".image").find(".image-id").text();
+				const imageId = $(this).closest(".image").find(".image-id").text();
 				(function(element, imageId) {
 					$(".show-data", element).click(function() {
 						editImage(imageId);
 					});
 					$("button[name='moveLeft'], button[name='moveRight']", element).click(function() {
-						ajaxGet("editImage.ajax", { "formPassword": getFormPassword(), "image": imageId, "moveLeft": this.name == "moveLeft", "moveRight": this.name == "moveRight" }, function(data) {
+						ajaxGet("editImage.ajax", { "formPassword": getFormPassword(), "image": imageId, "moveLeft": this.name === "moveLeft", "moveRight": this.name === "moveRight" }, function(data) {
 							if (data && data.success) {
 								swapImage(data.sourceImageId, data.destinationImageId);
 							}
@@ -113,16 +113,23 @@
 						return false;
 					});
 					$("button[name='submit']", element).click(function() {
-						title = $(":input[name='title']:enabled", this.form).val();
-						description = $(":input[name='description']:enabled", this.form).val();
+						const title = $(":input[name='title']:enabled", this.form).val();
+						const description = $(":input[name='description']:enabled", this.form).val();
 						ajaxGet("editImage.ajax", { "formPassword": getFormPassword(), "image": imageId, "title": title, "description": description }, function(data) {
-							if (data && data.success) {
-								getImage(data.imageId).find(".image-title").text(data.title);
-								getImage(data.imageId).find(".image-description").html(data.parsedDescription);
-								getImage(data.imageId).find(":input[name='title']").attr("defaultValue", title);
-								getImage(data.imageId).find(":input[name='description']").attr("defaultValue", data.description);
+							const imageElement = getImage(data.imageId);
+							const imageTitleInput = imageElement.find(":input[name='title']");
+							const imageDescriptionInput = imageElement.find(":input[name='description']");
+                            if (data && data.success) {
+								imageElement.find(".image-title").text(data.title);
+								imageElement.find(".image-description").html(data.parsedDescription);
+								imageTitleInput.prop("defaultValue", data.title);
+								imageDescriptionInput.prop("defaultValue", data.description);
 								cancelImageEditing();
-							}
+							} else if (data && !data.success) {
+								imageTitleInput.prop("value", imageTitleInput.prop("defaultValue"));
+                                imageDescriptionInput.prop("value", imageDescriptionInput.prop("defaultValue"));
+                                cancelImageEditing();
+                            }
 						});
 						return false;
 					});
@@ -131,7 +138,7 @@
 		}
 
 		/* ID of the album currently being edited. */
-		var editingAlbumId = null;
+		let editingAlbumId = null;
 
 		/**
 		 * Shows the form for editing an album.
@@ -139,7 +146,7 @@
 		 * @param albumId The ID of the album to edit.
 		 */
 		function editAlbum(albumId) {
-			if (editingAlbumId != albumId) {
+			if (editingAlbumId !== albumId) {
 				if (editingAlbumId != null) {
 					cancelAlbumEditing();
 				}
@@ -152,7 +159,7 @@
 			$(".edit-data", getAlbum(albumId)).show();
 			console.log(getAlbum(albumId));
 			$(document).bind("click.sone", function(event) {
-				if ($(event.target).closest("#album-" + albumId).size() == 0) {
+				if ($(event.target).closest("#album-" + albumId).size() === 0) {
 					cancelAlbumEditing();
 				}
 			});
@@ -189,21 +196,21 @@
 		 * @param destinationId The ID of the destionation album
 		 */
 		function swapAlbum(sourceId, destinationId) {
-			sourceElement = getAlbum(sourceId);
-			destinationElement = getAlbum(destinationId);
-			sourceParent = sourceElement.closest(".album-row");
-			sourcePrevSibling = sourceElement.prev();
+			const sourceElement = getAlbum(sourceId);
+			const destinationElement = getAlbum(destinationId);
+			const sourceParent = sourceElement.closest(".album-row");
+			const sourcePrevSibling = sourceElement.prev();
 			sourceElement.detach();
 			destinationElement.before(sourceElement);
-			if (sourcePrevSibling.get(0) != destinationElement.get(0)) {
+			if (sourcePrevSibling.get(0) !== destinationElement.get(0)) {
 				destinationElement.detach();
 				(sourcePrevSibling.size() > 0) ? sourcePrevSibling.after(destinationElement) : sourceParent.prepend(destinationElement);
 			}
-			if ($("button[name='moveLeft']", sourceElement).hasClass("hidden") != $("button[name='moveLeft']", destinationElement).hasClass("hidden")) {
+			if ($("button[name='moveLeft']", sourceElement).hasClass("hidden") !== $("button[name='moveLeft']", destinationElement).hasClass("hidden")) {
 				$("button[name='moveLeft']", sourceElement).toggleClass("hidden");
 				$("button[name='moveLeft']", destinationElement).toggleClass("hidden");
 			}
-			if ($("button[name='moveRight']", sourceElement).hasClass("hidden") != $("button[name='moveRight']", destinationElement).hasClass("hidden")) {
+			if ($("button[name='moveRight']", sourceElement).hasClass("hidden") !== $("button[name='moveRight']", destinationElement).hasClass("hidden")) {
 				$("button[name='moveRight']", sourceElement).toggleClass("hidden");
 				$("button[name='moveRight']", destinationElement).toggleClass("hidden");
 			}
@@ -214,14 +221,14 @@
 		 */
 		function prepareAlbums() {
 			$(".album").each(function() {
-				albumId = $(this).closest(".album").find(".album-id").text();
+				const albumId = $(this).closest(".album").find(".album-id").text();
 				(function(element, albumId) {
 					$(".show-data", element).click(function() {
 						console.log("show-data");
 						editAlbum(albumId);
 					});
 					$("button[name='moveLeft'], button[name='moveRight']", element).click(function() {
-						ajaxGet("editAlbum.ajax", { "formPassword": getFormPassword(), "album": albumId, "moveLeft": this.name == "moveLeft", "moveRight": this.name == "moveRight" }, function(data) {
+						ajaxGet("editAlbum.ajax", { "formPassword": getFormPassword(), "album": albumId, "moveLeft": this.name === "moveLeft", "moveRight": this.name === "moveRight" }, function(data) {
 							if (data && data.success) {
 								swapAlbum(data.sourceAlbumId, data.destinationAlbumId);
 							}
@@ -229,16 +236,23 @@
 						return false;
 					});
 					$("button[name='submit']", element).click(function() {
-						title = $(":input[name='title']:enabled", this.form).val();
-						description = $(":input[name='description']:enabled", this.form).val();
+						const title = $(":input[name='title']:enabled", this.form).val();
+						const description = $(":input[name='description']:enabled", this.form).val();
 						ajaxGet("editAlbum.ajax", { "formPassword": getFormPassword(), "album": albumId, "title": title, "description": description }, function(data) {
-							if (data && data.success) {
-								getAlbum(data.albumId).find(".album-title").text(data.title);
-								getAlbum(data.albumId).find(".album-description").text(data.description);
-								getAlbum(data.albumId).find(":input[name='title']").attr("defaultValue", title);
-								getAlbum(data.albumId).find(":input[name='description']").attr("defaultValue", description);
-								cancelAlbumEditing();
-							}
+							if (data) {
+								const albumTitleField = getAlbum(data.albumId).find(".album-title");
+								const albumDescriptionField = getAlbum(data.albumId).find(".album-description");
+                                if (data.success) {
+                                    albumTitleField.text(data.title);
+                                    albumDescriptionField.text(data.description);
+                                    getAlbum(data.albumId).find(":input[name='title']").prop("defaultValue", title);
+                                    getAlbum(data.albumId).find(":input[name='description']").prop("defaultValue", description);
+                                } else {
+                                    albumTitleField.prop("value", albumTitleField.prop("defaultValue"));
+                                    albumDescriptionField.prop("value", albumDescriptionField.prop("defaultValue"));
+                                }
+                                cancelAlbumEditing();
+                            }
 						});
 						return false;
 					});
@@ -261,7 +275,7 @@
 		<%else>
 
 			<%if album.sone.local>
-				<script language="javascript">
+				<script type="application/javascript">
 
 					$(function() {
 						getTranslation("WebInterface.DefaultText.UploadImage.Title", function(text) {
@@ -327,7 +341,7 @@
 				<%/foreach>
 			</div>
 
-			<p id="description"><% album.description|parse sone=album.sone></p>
+			<p id="description"><% album.description|parse sone=album.sone|render></p>
 
 			<%if album.sone.local>
 				<div class="show-edit-album hidden toggle-link"><a class="small-link">» <%= Page.ImageBrowser.Album.Edit.Title|l10n|html></a></div>
@@ -339,17 +353,6 @@
 						<input type="hidden" name="formPassword" value="<%formPassword|html>" />
 						<input type="hidden" name="album" value="<%album.id|html>" />
 
-						<%if ! album.images.empty>
-							<div>
-								<label for="album-image"><%= Page.ImageBrowser.Album.Label.AlbumImage|l10n|html></label>
-								<select name="album-image">
-									<option disabled="disabled"><%= Page.ImageBrowser.Album.AlbumImage.Choose|l10n|html></option>
-									<%foreach album.images image>
-										<option value="<% image.id|html>"<%if album.albumImage.id|match value=image.id> selected="selected"<%/if>><% image.title|html></option>
-									<%/foreach>
-								</select>
-							</div>
-						<%/if>
 						<div>
 							<label for="title"><%= Page.ImageBrowser.Album.Label.Title|l10n|html></label>
 							<input type="text" name="title" value="<%album.title|html>" />
@@ -373,7 +376,7 @@
 				</div>
 			<%/if>
 
-			<%foreach album.images image|paginate pageSize=core.preferences.imagesPerPage page=page>
+			<%foreach album.images image|paginate pageSize=preferences.imagesPerPage page=page>
 				<%first>
 					<h2><%= Page.ImageBrowser.Header.Images|l10n|html></h2>
 					<%include include/pagination.html pageParameter=="page">
@@ -386,7 +389,7 @@
 					</div>
 					<div class="show-data">
 						<div class="image-title"><% image.title|html></div>
-						<div class="image-description"><% image.description|parse sone=image.sone></div>
+						<div class="image-description"><% image.description|parse sone=image.sone|render></div>
 					</div>
 					<%if album.sone.local>
 						<form class="edit-image" action="editImage.html" method="post">
@@ -470,7 +473,7 @@
 		<%else>
 
 			<%if image.sone.local>
-				<script language="javascript">
+				<script type="application/javascript">
 					$(function() {
 						getTranslation("WebInterface.DefaultText.EditImage.Title", function(text) {
 							$("#edit-image input[name='title']").each(function() {
@@ -498,7 +501,7 @@
 				<%/if>
 			</div>
 
-			<p class="parsed"><%image.description|parse sone=image.sone></p>
+			<p class="parsed"><%image.description|parse sone=image.sone|render></p>
 
 			<%if image.sone.local>
 
@@ -544,7 +547,7 @@
 	<%elseif soneRequested>
 
 		<%if sone.local>
-			<script language="javascript">
+			<script type="application/javascript">
 				$(function() {
 					getTranslation("WebInterface.DefaultText.CreateAlbum.Name", function(text) {
 						$("#create-album input[name='name']").each(function() {
@@ -596,7 +599,7 @@
 
 	<%elseif galleryRequested>
 
-		<%foreach albums album|paginate pageSize=core.preferences.imagesPerPage pageParameter=request.page pagination=albumPagination>
+		<%foreach albums album|paginate pageSize=preferences.imagesPerPage pageParameter=request.page pagination=albumPagination>
 			<%first>
 				<h2><%= Page.ImageBrowser.Header.Albums|l10n|html></h2>
 				<%include include/pagination.html pagination=albumPagination pageParameter=="page">
@@ -616,7 +619,7 @@
 				<div class="show-data">
 					<div class="album-sone"><a href="imageBrowser.html?sone=<%album.sone.id|html>"><%album.sone.niceName|html></a></div>
 					<div class="album-title"><% album.title|html> (<%= View.Sone.Stats.Images|l10n 0=album.images.size>)</div>
-					<div class="album-description"><% album.description|parse sone=album.sone></div>
+					<div class="album-description"><% album.description|parse sone=album.sone|render></div>
 				</div>
 			</div>
 			<%= false|store key==endRow>