django-blog-zinnia icon indicating copy to clipboard operation
django-blog-zinnia copied to clipboard

IndexErrors from breadcrumbs.py line 95

Open mcfletch opened this issue 4 years ago • 1 comments

https://github.com/Fantomas42/django-blog-zinnia/blob/881101a9d1d455b2fc581d6f4ae0947cdd8126c6/zinnia/breadcrumbs.py#L95

Has an assumption that breadcrumbs(...) will return a non-empty list. However it seems that certain crawlers are getting to the archive pages without a breadcrumb trail. I believe it should be sufficient, in this case to simply check:

if breadcrumbs:

before the call to set the url on the last breadcrumb.

mcfletch avatar Jun 03 '20 14:06 mcfletch

@mcfletch

I just created a fork of this project and updated the code to work with modern versions of Django (including 4.0+).

Please visit https://github.com/arrobalytics/django-blog-zinnia if you want to contribute. Don't forget to star the project to show your support.

Visit the new site https://www.djangoblogzinnia.com for documentation, demo and other resources.

elarroba avatar Jan 07 '22 17:01 elarroba