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.
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)
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?
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.
5 hours of code non-stop. I'm so tired. Sorry for the check_install.tpl which have nothing to do there.