developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 13 years ago

Closed 13 years ago

#519 closed bug (fixed)

jFile::removeDir crash if there are a symlink in

Reported by: bastnic Owned by: bastnic
Priority: normal Milestone: Jelix 1.0.3
Component: jelix:utils Version: 1.0.2
Severity: major Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

in PHP is_dir return true on a link so the function should test if is_link then do unlink instead of a removeDir. 

[warning 1] rmdir(/tmp/test/monappli/machin/toto) [<a href='function.rmdir'>function.rmdir</a>]: N'est pas un répertoire /home/bast/tmp/lib/jelix/utils/jFile.class.php 116

[warning 1] rmdir(/tmp/test/monappli/machin/toto) [<a href='function.rmdir'>function.rmdir</a>]: Le répertoire n'est pas vide. /home/bast/tmp/lib/jelix/utils/jFile.class.php 116

Attachments (1)

removedir.diff​ (566 bytes) - added by bastnic 13 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 13 years ago by bastnic

  • Status changed from new to assigned

Changed 13 years ago by bastnic

comment:2 Changed 13 years ago by bastnic

  • review set to review?

5 hours of code non-stop. I'm so tired. Sorry for the check_install.tpl which have nothing to do there.

comment:3 Changed 13 years ago by laurentj

  • review changed from review? to review+
  • Severity changed from normal to major

It's ok. Land it in trunk and 1.0.x branch. Thanks :-)

comment:4 Changed 13 years ago by bastnic

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.