ManageAssets: add thumbnails to asset table

controllers/ManageAssets.php

@@ -45,6 +45,34 @@ class ManageAssets extends HTMLController
 						},
 					],
 				],
+				'thumbnail' => [
+					'header' => ' ',
+					'is_sortable' => false,
+					'cell_class' => 'text-center',
+					'parse' => [
+						'type' => 'function',
+						'data' => function($row) {
+							$asset = Image::byRow($row);
+							$width = $height = 65;
+							if ($asset->isImage())
+							{
+								if ($asset->isPortrait())
+									$width = null;
+								else
+									$height = null;
+
+								$thumb = $asset->getThumbnailUrl($width, $height);
+							}
+							else
+								$thumb = BASEURL . '/images/nothumb.svg';
+
+							$width = isset($width) ? $width . 'px' : 'auto';
+							$height = isset($height) ? $height . 'px' : 'auto';
+
+							return sprintf('<img src="%s" style="width: %s; height: %s;">', $thumb, $width, $height);
+						},
+					],
+				],
 				'id_asset' => [
 					'value' => 'id_asset',
 					'header' => 'ID',
@@ -107,7 +135,7 @@ class ManageAssets extends HTMLController
 
 				$data = Registry::get('db')->queryAssocs('
 					SELECT a.id_asset, a.subdir, a.filename,
-						a.image_width, a.image_height,
+						a.image_width, a.image_height, a.mimetype,
 						u.id_user, u.first_name, u.surname
 					FROM assets AS a
 					LEFT JOIN users AS u ON a.id_user_uploaded = u.id_user

models/Asset.php

@@ -32,7 +32,7 @@ class Asset
 				$this->$attribute = $value;
 		}
 
-		if (!empty($data['date_captured']) && $data['date_captured'] !== 'NULL')
+		if (isset($data['date_captured']) && $data['date_captured'] !== 'NULL' && !is_object($data['date_captured']))
 			$this->date_captured = new DateTime($data['date_captured']);
 	}
 
@@ -108,7 +108,7 @@ class Asset
 				'_' => '_',
 			]);
 
-		return $return_format == 'object' ? new Asset($row) : $row;
+		return $return_format === 'object' ? new static($row) : $row;
 	}
 
 	public static function fromIds(array $id_assets, $return_format = 'array')
@@ -168,7 +168,7 @@ class Asset
 		foreach ($thumbnails as $thumb)
 			$assets[$thumb[0]]['thumbnails'][$thumb[1]] = $thumb[2];
 
-		if ($return_format == 'array')
+		if ($return_format === 'array')
 			return $assets;
 		else
 		{

models/Database.php

@@ -306,9 +306,11 @@ class Database
 		if (defined("DB_LOG_QUERIES") && DB_LOG_QUERIES)
 			$this->logged_queries[] = $db_string;
 
-		$return = @mysqli_query($this->connection, $db_string, empty($this->unbuffered) ? MYSQLI_STORE_RESULT : MYSQLI_USE_RESULT);
-
-		if (!$return)
+		try
+		{
+			$return = @mysqli_query($this->connection, $db_string, empty($this->unbuffered) ? MYSQLI_STORE_RESULT : MYSQLI_USE_RESULT);
+		}
+		catch (Exception $e)
 		{
 			$clean_sql = implode("\n", array_map('trim', explode("\n", $db_string)));
 			trigger_error($this->error() . '<br>' . $clean_sql, E_USER_ERROR);

models/Image.php

@@ -12,12 +12,6 @@ class Image extends Asset
 	const TYPE_LANDSCAPE = 2;
 	const TYPE_PORTRAIT = 4;
 
-	protected function __construct(array $data)
-	{
-		foreach ($data as $attribute => $value)
-			$this->$attribute = $value;
-	}
-
 	public static function fromId($id_asset, $return_format = 'object')
 	{
 		$asset = parent::fromId($id_asset, 'array');

public/css/default.css

@@ -366,8 +366,8 @@ div.polaroid a {
 /* Album button box
 ---------------------*/
 .album_button_box {
-	float: right;
 	display: flex;
+	justify-content: flex-end;
 	margin-bottom: 3rem;
 }
 .album_button_box > a,

templates/AlbumButtonBox.php

@@ -22,7 +22,7 @@ class AlbumButtonBox extends Template
 	public function html_main()
 	{
 		echo '
-			<div class="album_button_box">';
+			<div class="container album_button_box">';
 
 		foreach ($this->buttons as $button)
 			echo '

templates/PhotoPage.php

@@ -174,7 +174,10 @@ class PhotoPage extends Template
 		echo '
 					</ul>';
 
-		$this->printNewTagScript($tagKind, $tagListId, $newTagId);
+		if ($allowLinkingNewTags)
+		{
+			$this->printNewTagScript($tagKind, $tagListId, $newTagId);
+		}
 	}
 
 	private function printNewTagScript($tagKind, $tagListId, $newTagId)

templates/TabularData.php

@@ -8,7 +8,7 @@
 
 class TabularData extends SubTemplate
 {
-	private GenericTable $_t;
+	protected GenericTable $_t;
 
 	public function __construct(GenericTable $table)
 	{
@@ -16,6 +16,47 @@ class TabularData extends SubTemplate
 	}
 
 	protected function html_content()
+	{
+		$this->renderTitle();
+
+		foreach ($this->_subtemplates as $template)
+			$template->html_main();
+
+		// Showing an inline form?
+		$pager = $this->_t->getPageIndex();
+		if (!empty($pager) || isset($this->_t->form_above))
+			$this->renderPaginationForm($pager, $this->_t->form_above);
+
+		$tableClass = $this->_t->getTableClass();
+		if ($tableClass)
+			echo '
+			<div class="', $tableClass, '">';
+
+		// Build the table!
+		echo '
+				<table class="table table-striped table-condensed">';
+
+		$this->renderTableHead($this->_t->getHeader());
+		$this->renderTableBody($this->_t->getBody());
+
+		echo '
+				</table>';
+
+		if ($tableClass)
+			echo '
+			</div>';
+
+		// Showing an inline form?
+		if (!empty($pager) || isset($this->_t->form_below))
+			$this->renderPaginationForm($pager, $this->_t->form_below);
+
+		$title = $this->_t->getTitle();
+		if (!empty($title))
+			echo '
+			</div>';
+	}
+
+	protected function renderTitle()
 	{
 		$title = $this->_t->getTitle();
 		if (!empty($title))
@@ -25,43 +66,32 @@ class TabularData extends SubTemplate
 			<div class="generic-table', !empty($titleclass) ? ' ' . $titleclass : '', '">
 				<h1>', htmlspecialchars($title), '</h1>';
 		}
+	}
 
-		foreach ($this->_subtemplates as $template)
-			$template->html_main();
-
-		// Showing an inline form?
-		$pager = $this->_t->getPageIndex();
-		if (!empty($pager) || isset($this->_t->form_above))
-		{
-			echo '
-				<div class="row clearfix justify-content-end">';
-
-			// Page index?
-			if (!empty($pager))
-				PageIndexWidget::paginate($pager);
-
-			// Form controls?
-			if (isset($this->_t->form_above))
-				$this->showForm($this->_t->form_above);
-
-			echo '
-				</div>';
-		}
-
-		$tableClass = $this->_t->getTableClass();
-		if ($tableClass)
-			echo '
-			<div class="', $tableClass, '">';
-
-		// Build the table!
+	protected function renderPaginationForm($pager, $form, $class='row')
+	{
+		echo '
+			<div class="', $class, ' clearfix justify-content-end">';
+
+		// Page index?
+		if (!empty($pager))
+			PageIndexWidget::paginate($pager);
+
+		// Form controls?
+		if (isset($form))
+			$this->renderInlineForm($form);
+
+		echo '
+			</div>';
+	}
+
+	protected function renderTableHead(array $headers)
+	{
 		echo '
-				<table class="table table-striped table-condensed">
 					<thead>
 						<tr>';
 
-		// Show all headers in their full glory!
-		$header = $this->_t->getHeader();
-		foreach ($header as $th)
+		foreach ($headers as $th)
 		{
 			echo '
 							<th', (!empty($th['width']) ? ' width="' . $th['width'] . '"' : ''), (!empty($th['class']) ? ' class="' . $th['class'] . '"' : ''), ($th['colspan'] > 1 ? ' colspan="' . $th['colspan'] . '"' : ''), ' scope="', $th['scope'], '">',
@@ -75,11 +105,14 @@ class TabularData extends SubTemplate
 
 		echo '
 						</tr>
-					</thead>
+					</thead>';
+	}
+
+	protected function renderTableBody($body)
+	{
+		echo '
 					<tbody>';
 
-		// The body is what we came to see!
-		$body = $this->_t->getBody();
 		if (is_array($body))
 		{
 			foreach ($body as $tr)
@@ -90,59 +123,31 @@ class TabularData extends SubTemplate
 				foreach ($tr['cells'] as $td)
 				{
 					echo '
-							<td', (!empty($td['width']) ? ' width="' . $td['width'] . '"' : ''), '>';
-
-					if (!empty($td['class']))
-						echo '<span class="', $td['class'], '">', $td['value'], '</span>';
-					else
-						echo $td['value'];
-
-					echo '</td>';
+							<td',
+								(!empty($td['class']) ? ' class="' . $td['class'] . '"' : ''),
+								(!empty($td['width']) ? ' width="' . $td['width'] . '"' : ''), '>',
+								$td['value'],
+							'</td>';
 				}
 
 				echo '
 						</tr>';
 			}
 		}
-		// !!! Sum colspan!
 		else
+		{
+			$header = $this->_t->getHeader();
 			echo '
 						<tr>
 							<td colspan="', count($header), '" class="fullwidth">', $body, '</td>
 						</tr>';
-
-		echo '
-					</tbody>
-				</table>';
-
-		if ($tableClass)
-			echo '
-			</div>';
-
-		// Showing an inline form?
-		if (!empty($pager) || isset($this->_t->form_below))
-		{
-			echo '
-				<div class="row clearfix justify-content-end">';
-
-			// Page index?
-			if (!empty($pager))
-				PageIndexWidget::paginate($pager);
-
-			// Form controls?
-			if (isset($this->_t->form_below))
-				$this->showForm($this->_t->form_below);
-
-			echo '
-				</div>';
 		}
 
-		if (!empty($title))
-			echo '
-			</div>';
+		echo '
+					</tbody>';
 	}
 
-	protected function showForm($form)
+	protected function renderInlineForm($form)
 	{
 		if (!isset($form['is_embed']))
 			echo '