It seems a bit of a convoluted method of traversing the directories, especially with changing the current working directory so often. You could change the call to the second glob (in the foreach) to use full (or relative) paths rather than just *. Or you could use the handy RecursiveDirectoryIterator which makes life easier.