EFE retry handling refinements

2026-10-03 07:49:52 +00:00 · 2020-12-28 14:49:16 -05:00
parent 7f96111669
commit 054b0dfa06
1 changed files with 4 additions and 1 deletions
@@ -171,7 +171,10 @@ class DocumentEmbeddedContentExtractor {

        // Skip files that already have been unpacked.
        /*
-         * TODO (Jira-7145): Is the logic of this check correct?
+         * TODO (Jira-7145): Is the logic of this check correct? Also note that
+         * this suspect code used to have a bug in that makeOutputFolder() was
+         * called, so the directory was always created here if it did not exist,
+         * making this check only a call to AbstractFile.hasChildren().
         */
        try {
            if (abstractFile.hasChildren()) {