Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix page high bug about parent page and report bug about leaf page #2

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

yoa1226
Copy link

@yoa1226 yoa1226 commented Mar 5, 2024

Hi,

This PR contains 4 commits to solve 3 issues I've encountered when using this great tool 😃

Thanks
yoa

  1. The first two commit is about adding empty value in retrieve_items and replace type with index_type in class Tree #1.

  2. The third commit is about displaying page high key and blank key in parent page.

  3. The fourth commit is about displaying page high key in leaf page . I believe there is a bug, but I are not able to fix it.

bdrouvot and others added 5 commits November 19, 2021 10:38
To avoid possible errors like:

Traceback (most recent call last):
  File "inspector/command.py", line 51, in <module>
    main(sys.argv[1:])
  File "inspector/command.py", line 47, in main
    render_tree(tree, file_path)
  File "/root/pageinspect_inspector/inspector/renderer.py", line 29, in render_tree
    html_file.write(template.render(tree=tree))
  File "/usr/local/lib/python3.7/site-packages/Jinja2-3.0.3-py3.7.egg/jinja2/environment.py", line 1291, in render
    self.environment.handle_exception()
  File "/usr/local/lib/python3.7/site-packages/Jinja2-3.0.3-py3.7.egg/jinja2/environment.py", line 925, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "/root/pageinspect_inspector/inspector/templates/render_tree.html", line 282, in top-level template code
    {% include 'render_page.html' %}
  File "/root/pageinspect_inspector/inspector/templates/render_page.html", line 28, in top-level template code
    [{% for val in item.value %}
TypeError: 'NoneType' object is not iterable
Filtering on index_type was not working in render_page.html due to this issue.
adding empty value in retrieve_items and replace type with index_type in class Tree
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants