claunia/qemudb
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Screenshot::delete() and Image::delete() should check if the file exists before attempting to

Browse Source 
call unlink() on it. Fixes three php errors that occur during the unit tests.
This commit is contained in:
Chris Morgan
2007-06-09 22:17:32 +00:00
committed by WineHQ
parent 4cbbb2fbb6
commit ea2de6fe3c
2 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
include/image.php
View File

@@ -391,6 +391,8 @@ class Image {
*/ */
function delete() function delete()
{ {
// if the file exists, delete it
if(is_file($this->sFile))
unlink($this->sFile); unlink($this->sFile);
} }

7
include/screenshot.php
View File

@@ -139,7 +139,12 @@ class Screenshot {
$this->oScreenshotImage->delete(); $this->oScreenshotImage->delete();
$this->oThumbnailImage->delete(); $this->oThumbnailImage->delete();
unlink(appdb_fullpath("/data/screenshots/originals/".$this->iScreenshotId));
// if the original file exists, delete it
$sOriginalFilename = appdb_fullpath("/data/screenshots/originals/".$this->iScreenshotId);
if(is_file($sOriginalFilename))
unlink($sOriginalFilename);
if(!$bSilent) if(!$bSilent)
$this->mailMaintainers(true); $this->mailMaintainers(true);
} }