We have a nice little function file_unmanaged_delete_recursive() that does the work in simpletest_uninstall() for us, so we should use it.

CommentFileSizeAuthor
#1 968476-simpletest-uninstall-D7.patch4.47 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 968476-simpletest-uninstall-D7.patch4.47 KB

Also moves simpletest_uninstall() below simpletest_schema() as that's our standard .install hook ordering, and fixes whitespace, and a minor query simplification.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

Nice little cleanup

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Nice clean-up indeed. Committed to CVS HEAD.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.