You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Today I hit a exception while I was trying to compress with Django compressor.
I was not the only one who was seeing this exception. So I quickly found a open PR for that.
It seemed like more people were seeing this exception in combination with sorl-thumbnail.
Can someone check this situation and provide us feedback so we get more insights if the 'nodelist_file' attribute should remain like it is or if it should be changed.
The text was updated successfully, but these errors were encountered:
Today I hit a exception while I was trying to compress with Django compressor.
I was not the only one who was seeing this exception. So I quickly found a open PR for that.
It seemed like more people were seeing this exception in combination with sorl-thumbnail.
django-compressor/django-compressor#605
As you can see in this pull request it tries to patch the fact that a nodelist of a node may not return a list which crashes the compressor.
The ThumbnailNode contains a 'nodelist_file' in the child_nodelists attribute. As you read the code you see that this value may stay None.
I'm not sure if his is correct. The NodeList class of the django template base module inherits from a list. So it seems that 'nodelist_file' should also be a (empty) list.
https://github.com/django/django/blob/master/django/template/base.py#L984
Can someone check this situation and provide us feedback so we get more insights if the 'nodelist_file' attribute should remain like it is or if it should be changed.
The text was updated successfully, but these errors were encountered: