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

KeyError: page_index_folder_index #2646

Closed
ericcotelnu opened this Issue Jan 24, 2017 · 5 comments

Comments

Projects
None yet
4 participants
@ericcotelnu

ericcotelnu commented Jan 24, 2017

If setting PAGE_INDEX = True, with prety much default settings (except usual config), then I get:

$ nikola build
Traceback (most recent call last):
  File "/usr/local/bin/nikola", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.5/dist-packages/nikola/__main__.py", line 171, in main
    _ = DN.run(oargs)
  File "/usr/local/lib/python3.5/dist-packages/nikola/__main__.py", line 339, in run
    self.nikola.init_plugins()
  File "/usr/local/lib/python3.5/dist-packages/nikola/nikola.py", line 1077, in init_plugins
    self._activate_plugins_of_category("SignalHandler")
  File "/usr/local/lib/python3.5/dist-packages/nikola/nikola.py", line 1233, in _activate_plugins_of_category
    plugin_info.plugin_object.set_site(self)
  File "/usr/local/lib/python3.5/dist-packages/nikola/plugins/misc/taxonomies_classifier.py", line 328, in set_site
    self._register_path_handlers(taxonomy)
  File "/usr/local/lib/python3.5/dist-packages/nikola/plugins/misc/taxonomies_classifier.py", line 316, in _register_path_handlers
    doc = taxonomy.path_handler_docstrings[name]
KeyError: 'page_index_folder_index'

It seems silly, but I'd rather post this for others who might make the same mistake.

@ericcotelnu

This comment has been minimized.

Show comment
Hide comment
@ericcotelnu

ericcotelnu Jan 25, 2017

Ace, thanks folks. Love that label ("brown paper bag") too, made me LOL when I saw it.

ericcotelnu commented Jan 25, 2017

Ace, thanks folks. Love that label ("brown paper bag") too, made me LOL when I saw it.

@sinewalker

This comment has been minimized.

Show comment
Hide comment
@sinewalker

sinewalker Mar 24, 2017

Applied the patch locally myself, which fixes the crash. But it doesn't render the index. Setting STORY_INDEX = True gives a WARNING, but also doesn't render the index. Here's my PAGES list:

PAGES = (
    ("pages/*.rst", "pg", "story.tmpl"),
    ("pages/*.txt", "pg", "story.tmpl"),
    ("pages/*.ipynb", "pg", "story.tmpl"),
    ("pages/*.md", "pg", "story.tmpl"),
    ("old_site/stories/*.wp", "pg", "story.tmpl"),
)

So I'm expecting output/pg/index.html (which was being rendered at least in 7.7.1) but it's not there.

sinewalker commented Mar 24, 2017

Applied the patch locally myself, which fixes the crash. But it doesn't render the index. Setting STORY_INDEX = True gives a WARNING, but also doesn't render the index. Here's my PAGES list:

PAGES = (
    ("pages/*.rst", "pg", "story.tmpl"),
    ("pages/*.txt", "pg", "story.tmpl"),
    ("pages/*.ipynb", "pg", "story.tmpl"),
    ("pages/*.md", "pg", "story.tmpl"),
    ("old_site/stories/*.wp", "pg", "story.tmpl"),
)

So I'm expecting output/pg/index.html (which was being rendered at least in 7.7.1) but it's not there.

@Kwpolska

This comment has been minimized.

Show comment
Hide comment
@Kwpolska

Kwpolska Mar 24, 2017

Member

@felixfontein, any ideas about why PAGE_INDEX doesn’t work at all?

Member

Kwpolska commented Mar 24, 2017

@felixfontein, any ideas about why PAGE_INDEX doesn’t work at all?

@felixfontein

This comment has been minimized.

Show comment
Hide comment
@felixfontein

felixfontein Mar 24, 2017

Contributor

Yep, there were two very stupid bugs which each caused the plugin not to work. Should be fixed in #2701.

Contributor

felixfontein commented Mar 24, 2017

Yep, there were two very stupid bugs which each caused the plugin not to work. Should be fixed in #2701.

@felixfontein

This comment has been minimized.

Show comment
Hide comment
@felixfontein

felixfontein Mar 24, 2017

Contributor

(I won't be too much online this weekend, so feel free to adjust the PR if you want.)

Contributor

felixfontein commented Mar 24, 2017

(I won't be too much online this weekend, so feel free to adjust the PR if you want.)

Kwpolska added a commit that referenced this issue Mar 25, 2017

Fix #2646 -- make page indexes actually work
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment