Fantomas42 / django-blog-zinnia

Simple yet powerful and really extendable application for managing a blog within your Django Web site.

http://django-blog-zinnia.com/

BSD 3-Clause "New" or "Revised" License

2.11k stars 730 forks source link

Fix paginating to second page without breadcrumbs. #551 #554

Closed jplehmann closed 4 years ago

jplehmann commented 5 years ago

Fixes: Pagination Causes ValueError from Breadcrumbs #551

What is the purpose of your pull request?

[x] Bug fix
[ ] New feature

Proposed changes

The proposed change is very simple, and simply checks that the accessed variable is defined.

coveralls commented 5 years ago

Coverage remained the same at 100.0% when pulling 0e48363f2b22e571e19d6bfaf8f9e406b617f89f on jplehmann:develop into b4949304b104a8e1a7a7a0773cbfd024313c3a15 on Fantomas42:develop.

jplehmann commented 4 years ago

@Fantomas42 any chance this can get merged?

jplehmann commented 4 years ago

You can easily see that it's a < 1 line of defensive code.