@TomMalbran isn't that basically what it's doing now (sans the actually closing of documents)?  Feels cleaner if handleFileCloseAll() just handles closing all documents as its name implies and refactor the logic to into a function in DocumentManager that we can pass an array to and it does the heavy lifting.