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 logger reference #1618

Closed
wants to merge 1 commit into from
Closed

Fix logger reference #1618

wants to merge 1 commit into from

Conversation

rail
Copy link

@rail rail commented Feb 8, 2015

This fixes the following exception:

PythonAction Error
Traceback (most recent call last):
File "/home/rail/.virtualenvs/nikola/local/lib/python2.7/site-packages/doit/action.py", line 372, in execute
returned_value = self.py_callable(_self.args, *_kwargs)
File "/home/rail/.virtualenvs/nikola/local/lib/python2.7/site-packages/nikola/plugins/task/scale_images.py", line 68, in process_image
self.resize_image(src, '.thumbnail'.join(os.path.splitext(dst)), self.kw['thumbnail_size'])
File "/home/rail/.virtualenvs/nikola/local/lib/python2.7/site-packages/nikola/image_processing.py", line 85, in resize_image
self.logger.warn("Can't thumbnail {0}, using original "
AttributeError: 'ScaleImage' object has no attribute 'logger'

@Kwpolska Kwpolska added this to the v7.3.1 milestone Feb 8, 2015
@Kwpolska Kwpolska self-assigned this Feb 8, 2015
@Kwpolska
Copy link
Member

Kwpolska commented Feb 8, 2015

Duplicate of #1598, which was already fixed in 50828ba (slightly differently).

@Kwpolska Kwpolska closed this Feb 8, 2015
@rail
Copy link
Author

rail commented Feb 8, 2015

Ooops, sorry about that. I tried to search before submitting the PR, but looks like my search-fu failed. :(

@rail rail deleted the fix_logging branch February 8, 2015 11:01
@Kwpolska
Copy link
Member

Kwpolska commented Feb 8, 2015

No problem. (You could’ve tried reproducing with the GitHub master though, that would get you a correct warning there)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants