captchta_helper and Severity: Warning --> unlink(./captcha/fonts) Is a directory #1678

Closed
Lechus opened this Issue Aug 2, 2012 · 2 comments

Projects

None yet

3 participants

@Lechus

I think it's something to be fixed by replacing
if ($filename != "." and $filename != ".." and $filename != "index.html")
to
if ($filename != "." and $filename != ".." and $filename != "index.html" and $filename != "fonts")

@pickupman pickupman added a commit to pickupman/CodeIgniter that referenced this issue Aug 19, 2012
@pickupman pickupman Fixes issue #1678 07cdcf8
@pickupman

I have updated the code to check if the file is a .jpg file since that are the files that are generated by the captcha helper. Previously the code only ignored, '.', '..', 'index.html' filenames. It would also try and delete a subfolder that may have contained the font_path.

@alexbilbie alexbilbie closed this Sep 2, 2012
@alexbilbie

Fixed in c557083

@nonchip nonchip pushed a commit to nonchip/CodeIgniter that referenced this issue Jun 29, 2013
@pickupman pickupman Fixes issue #1678 efba64d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment