From 85f17f9955edc2318cab3a0afff0e3c00184af8c Mon Sep 17 00:00:00 2001 From: BukeLy Date: Mon, 16 Mar 2026 14:19:51 +0800 Subject: [PATCH] Fix list_index variable shadowing in fix_incorrect_toc The loop variable `list_index = page_index - start_index` was overwriting the outer `list_index = incorrect_item['list_index']`, causing results to be written back to wrong index positions. Rename the loop variable to `page_list_idx` to avoid shadowing. Closes #66 --- pageindex/page_index.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pageindex/page_index.py b/pageindex/page_index.py index 16e5801..d646bb9 100644 --- a/pageindex/page_index.py +++ b/pageindex/page_index.py @@ -807,9 +807,9 @@ async def fix_incorrect_toc(toc_with_page_number, page_list, incorrect_results, page_contents=[] for page_index in range(prev_correct, next_correct+1): # Add bounds checking to prevent IndexError - list_index = page_index - start_index - if list_index >= 0 and list_index < len(page_list): - page_text = f"\n{page_list[list_index][0]}\n\n\n" + page_list_idx = page_index - start_index + if page_list_idx >= 0 and page_list_idx < len(page_list): + page_text = f"\n{page_list[page_list_idx][0]}\n\n\n" page_contents.append(page_text) else: continue