[MIG][9.0] website_blog_mgmt: Migrated to V 9.0

[sergio-teruel]

@lmignon Please can you review? I tested in V9.0

[elicoidal]

Please use short headers in py files

[sergio-teruel]

@elicoidal Changes done

[sergio-teruel]

@pedrobaeza changes done

[lmignon]

@sergio-incaser Using short headers doesn't mean the complete remove of the headers. You must preserve the copyright.

# -*- coding: utf-8 -*-
# Copyright 2015 ACSONE SA/NV (<http://acsone.eu>)
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).

[lmignon]

PLZ avoid to reformat the code if the code is not modified. These changes are not necessary and IMO the readability is not improved.

[pedrobaeza]

I agree, although sometimes the style use by the first contributor is not the one of my choice (which in this case, it is), I prefer not to change it because it creates larger diff and add a lot of noise on the needed changes to make this work.

[sergio-teruel]

@pedrobaeza @elicoidal changes done
@lmignon sorry for the confusion.

[lmignon]

👍 LGTM Thank you @sergio-incaser

[pedrobaeza]

👍 waiting for Travis

[sergio-teruel]

@lmignon @pedrobaeza I remembered now that I changed test_website_blog_flow.py to pass flake8 tests:

20:17: E126 continuation line over-indented for hanging indent

[pedrobaeza]

OK, change it then

[lmignon]

@sergio-incaser NOK the error in travis comes from website_blog_mgmt/post_init.py:5:1: E302 expected 2 blank lines, found 1 https://travis-ci.org/OCA/website/jobs/105631961#L380
That's the only one

[sergio-teruel]

@lmignon @pedrobaeza All ready. My pep8 local version was old. 😰

[pedrobaeza]

Merging...

[lmignon]

@sergio-incaser no stress 😏