refactor: - call addChildren() in base class

Author: ilandikov

src/Renderer/HtmlQueryResultsRenderer.ts

@@ -137,15 +137,13 @@ export class HtmlQueryResultsRenderer extends QueryResultsRendererBase {
         this.liElementStack.push(listItemElement);
     }
 
-    protected async addListItem(listItem: ListItem, listItemIndex: number, children: ListItem[]): Promise<void> {
+    protected async addListItem(listItem: ListItem, listItemIndex: number, _children: ListItem[]): Promise<void> {
         const listItemElement = this.currentLIElement();
 
         await this.taskLineRenderer.renderListItem(listItemElement, listItem, listItemIndex);
-
-        await this.addChildren(children);
     }
 
-    protected async addTask(task: Task, taskIndex: number, children: ListItem[]): Promise<void> {
+    protected async addTask(task: Task, taskIndex: number, _children: ListItem[]): Promise<void> {
         const isFilenameUnique = this.isFilenameUnique({ task }, this.queryRendererParameters.allMarkdownFiles());
         const listItem = this.currentLIElement();
 
@@ -183,8 +181,6 @@ export class HtmlQueryResultsRenderer extends QueryResultsRendererBase {
         }
 
         this.currentULElement().appendChild(listItem);
-
-        await this.addChildren(children);
     }
 
     protected async addChildren(children: ListItem[]) {

src/Renderer/QueryResultsRendererBase.ts

@@ -188,6 +188,8 @@ export abstract class QueryResultsRendererBase {
                 await this.addListItem(listItem, listItemIndex, listItem.children);
             }
 
+            await this.addChildren(listItem.children);
+
             // The children of this item will be added thanks to recursion and the fact that we always render all children currently
             this.addedListItems.add(listItem);