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 12 years ago

Closed 12 years ago

#733 closed bug (fixed)

jFile::removeDir failed on Windows OS

Reported by: foxmask Owned by:
Priority: normal Milestone: Jelix 1.0.7
Component: jelix:utils Version: 1.0.6
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

When i do

 jFile::removeDir($path,true)

i get the error :

[warning 2] rmdir(D:\wamp\www_appmanager\appmgr\modules\news\classes) 
[<a href='function.rmdir'>function.rmdir</a>]: 
Permission denied D:\wamp\www_appmanager\lib\jelix\utils\jFile.class.php 117

the permission denied is wrong as there is not permission defined on my filesystem with one line by directory name.

now i still have all the "empty" directories.

Attachments (1)

jfile.patch (373 bytes) - added by foxmask 12 years ago.
patch to unset $var at the end of the read

Download all attachments as: .zip

Change History (6)

comment:1 Changed 12 years ago by laurentj

  • Milestone set to jelix 1.1

We should see if it is the same thing in the 1.0.x branch

comment:2 Changed 12 years ago by laurentj

Since it uses only php functions to remove this directory, I don't know what I could do.

If PHP indicates that there is a permission issue, so there is a permission issue. Bad installation ? Who create this directory ? apache ? you ? does the given path really exist ? What is the real content of the given parameter $path ?

comment:3 Changed 12 years ago by foxmask

the same error had been done here

http://bugs.php.net/bug.php?id=42948

to fix this bug in the framework i put a patch in attachement

Changed 12 years ago by foxmask

patch to unset $var at the end of the read

comment:4 Changed 12 years ago by foxmask

  • review set to review?

comment:5 Changed 12 years ago by laurentj

  • Milestone changed from jelix 1.1 to Jelix 1.0.7
  • Resolution set to fixed
  • review changed from review? to review+
  • Status changed from new to closed
  • Version changed from trunk to 1.0.6

Thanks for this patch !

Included in the trunk and in the 1.0.x branch.

Note: See TracTickets for help on using tickets.