arno/lychee
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Added more error logging to Album class (#143)

Browse Source
This commit is contained in:
Tobias Reich 2014-05-06 21:37:21 +02:00
parent 83188ae9d9
commit d9b48f4080
1 changed files with 14 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
php/modules/Album.php
View File

@ -259,13 +259,19 @@ class Album extends Module {
# Create zip # Create zip
$zip = new ZipArchive(); $zip = new ZipArchive();
if ($zip->open($filename, ZIPARCHIVE::CREATE)!==TRUE) return false; if ($zip->open($filename, ZIPARCHIVE::CREATE)!==TRUE) {
Log::error($this->database, __METHOD__, __LINE__, 'Could not create ZipArchive');
return false;
}
# Execute query # Execute query
$photos = $this->database->query($photos); $photos = $this->database->query($photos);
# Check if album empty # Check if album empty
if ($photos->num_rows==0) return false; if ($photos->num_rows==0) {
Log::error($this->database, __METHOD__, __LINE__, 'Could not create ZipArchive without images');
return false;
}
# Parse each path # Parse each path
$files = array(); $files = array();
@ -367,7 +373,7 @@ class Album extends Module {
# Parse # Parse
$description = htmlentities($description); $description = htmlentities($description);
if (strlen($description)>1000) return false; if (strlen($description)>1000) $description = substr($description, 0, 1000);
# Execute query # Execute query
$result = $this->database->query("UPDATE lychee_albums SET description = '$description' WHERE id IN ($this->albumIDs);"); $result = $this->database->query("UPDATE lychee_albums SET description = '$description' WHERE id IN ($this->albumIDs);");
@ -532,7 +538,11 @@ class Album extends Module {
# Call plugins # Call plugins
$this->plugins(__METHOD__, 1, func_get_args()); $this->plugins(__METHOD__, 1, func_get_args());
if ($error||!$result) return false; if ($error) return false;
if (!$result) {
Log::error($this->database, __METHOD__, __LINE__, $this->database->error);
return false;
}
return true; return true;
} }