Remove Leftover Files Screen

[jassing]

I had a client ask how he could re-use my "left over files" screen (in post uninstall, it shows a list of files still installed)

So I cleaned up the code a bit & exported as a gallery screen.

The default is (as an example)
c:\documents and settings\all users\indigorose\setup factory\8.0\screens\Remove Leftovers

but it can easily go into your setupdata folder screens\Remove Leftovers

what it does:

On loading of the screen, it scans the app folder & app-data folder (provided you use the convension of Companyname\ProductName) and finds left over files & folders.

The user then selects (via checkboxes) the file(s) they want to delete.
Press the delete button and the file(s) selected are removed
When "going to the next screen", any "empty" folders are removed.

Please note - I cleaned this up & ran it on my dev machine, so I'm fairly confident that it's 'good to go' -- but as with anything that deletes files & folders, you should test on a test machine -- I will not be held liable for any lost data. ie: Use at your own risk. It was late when I did this.

Feel free to post comments, praise, and yes, even gripes / problems -- if there's a real problem, I'll fix it.

[jassing]

Embarased to say that I uploaded a version that "didn't exactly work" (typo in code) -- the dangers of testing on ones own system where you have other script files included...
Anyway; the attached should fix the typo / error when trying to click "next".

[redseujac]

Unfortunately this script doesn't work anymore for me in the new version SF9. The leftover files aren't removed anymore. Any idea why it fails?

[jassing]

Unfortunately this script doesn't work anymore for me in the new version SF9. The leftover files aren't removed anymore. Any idea why it fails?

It was written using SUF8, before SUF9 was announced, and it uses an older LUA. Probably the for loop needs adjusting.

I'll look at it soon.

[redseujac]

Thank you very much, Sir.

I would be grateful if you could make it work in SF9.

I know the LUA has changed and that the previous syntax sometimes must be modified (e.g. a "for" loop to iterate through a table; you must now use the pairs() function as part of this loop).

I've looked in your script but I was unable to detect such a "for" loop. Sure there are "for" loops in your script, but I couldn't find a loop like "for x, y in my_table do" which should be replaced by "for x, y in pairs(my_table) do".

Maybe I am missing something.

Best regards

redseujac

[jassing]

Sorry it's taken me so long to get to this -- been swamped.

I need to debug this more, but it may have explosed an issue in suf9 -- .ItemData is not being populated.
"stay tuned"

[redseujac]

..."stay tuned"

I will! Thank you.

[redseujac]

Reported problem solved with the newest build of SUF9 (9.0.1.0).

Thanks to the Indigo Rose SUF team for the fix.