Download & Extend

Warning on Flush of a Preset

Project:ImageCache
Version:6.x-1.0-alpha2
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Hi, I try to flush a preset, but an error occurred.

warning: rmdir([...]\files\imagecache\team_minifoto) [function.rmdir]: Directory not empty in [..,]\modules\imagecache\imagecache.module on line 695.

After that I check the drectory's contents and I found that it's empty.
The error is that the $d directory object is never closed, and removing it while still open raise that error. I solved adding (in a clean way) the next lines:

<?php
  $d
->close();
?>

before the call to rmdir() resulting in having this code:

<?php
$d
->close();
rmdir($path);
?>

I'm using latest 6.x version avaiable.
Regards

Comments

#1

Status:active» needs review

#2

Status:needs review» needs work

please provide a patch.

#3

In attach my patch. I'm using latest 6.x-BETA1 CVS version. Let me know if further info are needed!

Regards

AttachmentSizeStatusTest resultOperations
imagecache.module.patch522 bytesIgnored: Check issue status.NoneNone

#4

Status:needs work» fixed

thanks, committed to HEAD.

#5

committed this to DRUPAL-5--2 as well...

#6

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

nobody click here