Skip to content

Commit 85f17f9

Browse files
committed
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
1 parent 599d2ce commit 85f17f9

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

pageindex/page_index.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -807,9 +807,9 @@ async def process_and_check_item(incorrect_item):
807807
page_contents=[]
808808
for page_index in range(prev_correct, next_correct+1):
809809
# Add bounds checking to prevent IndexError
810-
list_index = page_index - start_index
811-
if list_index >= 0 and list_index < len(page_list):
812-
page_text = f"<physical_index_{page_index}>\n{page_list[list_index][0]}\n<physical_index_{page_index}>\n\n"
810+
page_list_idx = page_index - start_index
811+
if page_list_idx >= 0 and page_list_idx < len(page_list):
812+
page_text = f"<physical_index_{page_index}>\n{page_list[page_list_idx][0]}\n<physical_index_{page_index}>\n\n"
813813
page_contents.append(page_text)
814814
else:
815815
continue

0 commit comments

Comments
 (0)