Skip to content
Snippets Groups Projects
Commit bb5f59d02ac3 authored by Vincent Hatakeyama's avatar Vincent Hatakeyama
Browse files

:shirt: pylint

parent f3449773dd33
No related branches found
No related tags found
2 merge requests!232.6.0 ✨ Redner server an use an unix socket too so also handle it,!18✨ Redner server an use an unix socket too so also handle it
Hide whitespace changes
Inline Side-by-side
controllers/main.py
+ 2
− 2
View file @ bb5f59d0
......@@ -17,7 +17,7 @@
@route()
def report_routes(self, reportname, docids=None, converter=None, **data):
if converter != "redner":
return super(ReportController, self).report_routes(
return super().report_routes(
reportname=reportname,
docids=docids,
converter=converter,
......@@ -72,7 +72,7 @@
requestcontent = json.loads(data)
url, report_type = requestcontent[0], requestcontent[1]
if "redner" not in report_type:
return super(ReportController, self).report_download(data, token)
return super().report_download(data, token)
try:
reportname = url.split("/report/redner/")[1].split("?")[0]
docids = None
......
models/ir_actions_report.py
+ 3
− 3
View file @ bb5f59d0
......@@ -5,7 +5,7 @@
from odoo.exceptions import ValidationError
from odoo.tools.safe_eval import safe_eval
from odoo.addons.redner.utils.formats import Formats
from ..utils.formats import Formats
_logger = logging.getLogger(__name__)
......@@ -171,8 +171,8 @@
save_in_attachment = {}
if res_ids:
# Dispatch the records by ones having an attachment
Model = self.env[self.model]
record_ids = Model.browse(res_ids)
model = self.env[self.model]
record_ids = model.browse(res_ids)
if self.attachment:
for record_id in record_ids:
attachment_id = self.retrieve_attachment(record_id)
......
models/mail_template.py
+ 4
− 6
View file @ bb5f59d0
......@@ -87,7 +87,7 @@
"We received an unexpected error from redner server. "
"Please contact your administrator"
)
)
) from e
values["body_html"] = (
base64.b64decode(res[0]["body"]).decode("utf-8") if res else ""
)
......@@ -99,9 +99,7 @@
def generate_email(self, res_ids, fields=None):
self.ensure_one()
results = super(MailTemplate, self).generate_email(
res_ids, fields=fields
)
results = super().generate_email(res_ids, fields=fields)
if not self.is_redner_template:
return results
......@@ -120,5 +118,5 @@
def render_variable_hook(self, variables):
""" Override to add additional variables in mail "render template" func
"""
variables.update({"image": lambda value: image(value)})
return super(MailTemplate, self).render_variable_hook(variables)
variables.update({"image": image})
return super().render_variable_hook(variables)
models/redner.py
+ 5
− 5
View file @ bb5f59d0
......@@ -14,7 +14,7 @@
logger = logging.getLogger(__name__)
class Redner(object):
class Redner:
def __init__(self, api_key, server_url, account, timeout):
"""Initialize the API client
......@@ -95,8 +95,8 @@
response = r.text
if not str(r.status_code).startswith("2"):
logger.error("Bad response from Redner: %s" % response)
raise ValidationError("Unexpected redner error: %r" % response)
_logger.error("Bad response from Redner: %s", response)
raise ValidationError(_("Unexpected redner error: %r") % response)
return r.json()
......@@ -104,10 +104,10 @@
"""Try to establish a connection to server"""
conn = self.session.get(self.server_url, timeout=self.timeout)
if conn.status_code != requests.codes.ok:
raise ValidationError("Cannot Establish a connection to server")
raise ValidationError(_("Cannot Establish a connection to server"))
return conn
def __repr__(self):
return "<Redner %s>" % self.api_key
......@@ -108,10 +108,10 @@
return conn
def __repr__(self):
return "<Redner %s>" % self.api_key
class Templates(object):
class Templates:
def __init__(self, master):
self.master = master
......
models/redner_template.py
+ 11
− 9
View file @ bb5f59d0
......@@ -47,7 +47,7 @@
)
language = fields.Selection(
string=_("Language"),
string="Language",
selection=[
("text/html|mustache", "HTML + mustache"),
("text/mjml|mustache", "MJML + mustache"),
......@@ -116,7 +116,7 @@
version=fields.Datetime.now(),
)
return super(RednerTemplate, self).create(vals)
return super().create(vals)
@api.multi
def write(self, vals):
......@@ -133,7 +133,7 @@
redner_id = self.redner_id
vals["redner_id"] = vals["name"]
ret = super(RednerTemplate, self).write(vals)
ret = super().write(vals)
for record in self:
try:
produces, language = record.language.split("|")
......@@ -156,5 +156,5 @@
version=record.write_date,
)
except Exception as e:
logger.error("Failed to update redner template :%s" % e)
logger.error("Failed to update redner template :%s", e)
raise ValidationError(
......@@ -160,6 +160,6 @@
raise ValidationError(
_("Failed to update render template, %s" % e)
)
_("Failed to update render template, %s") % e
) from e
return ret
@api.multi
......@@ -173,9 +173,9 @@
except Exception:
pass
return super(RednerTemplate, self).unlink()
return super().unlink()
@api.multi
def copy(self, default=None):
self.ensure_one()
default = dict(default or {}, name=_("%s (copy)") % self.name)
......@@ -177,9 +177,9 @@
@api.multi
def copy(self, default=None):
self.ensure_one()
default = dict(default or {}, name=_("%s (copy)") % self.name)
return super(RednerTemplate, self).copy(default)
return super().copy(default)
@api.one
@api.depends("body", "template_data")
......@@ -215,7 +215,9 @@
)
except Exception as e:
logger.warning("Failed to fetch account template varlist :%s" % e)
logger.warning(
("Failed to fetch account template varlist :%s") % e
)
return []
@api.one
......
models/render_report.py
+ 5
− 6
View file @ bb5f59d0
......@@ -14,8 +14,8 @@
from odoo import _, api, fields, models
from odoo.exceptions import ValidationError
from odoo.addons.redner.utils import formats
from odoo.addons.redner.utils.formats import Formats
from ..utils import formats
from ..utils.formats import Formats
logger = logging.getLogger(__name__)
......@@ -75,7 +75,7 @@
"We received an unexpected error from redner server. "
"Please contact your administrator"
)
)
) from e
content = base64.b64decode(res[0]["body"])
with open(report_file, "wb") as f:
......@@ -89,7 +89,7 @@
self.ensure_one()
attachment = existing_reports_attachment.get(model_instance.id)
if attachment and self.ir_actions_report_id.attachment_use:
content = base64.decodestring(attachment.datas)
content = base64.decodebytes(attachment.datas)
report_file = tempfile.mktemp(
"." + self.ir_actions_report_id.redner_filetype
)
......@@ -141,8 +141,7 @@
return reports_path[0], filetype
if filetype == formats.FORMAT_PDF:
return self._merge_pdf(reports_path), formats.FORMAT_PDF
else:
return self._zip_results(reports_path), "zip"
return self._zip_results(reports_path), "zip"
@api.model
def _cleanup_tempfiles(self, temporary_files):
......
utils/formats.py
+ 3
− 6
View file @ bb5f59d0
# -*- coding: utf-8 -*-
DEFAULT_MIMETYPE = "application/octet-stream"
FORMAT_WORD97 = "doc"
......@@ -17,7 +15,7 @@
pass
class Format(object):
class Format:
"""A format representation that contains:
a name we use in our applications
an ODF name (like: 'MS Word 2003 XML') which is the name you must
......@@ -36,7 +34,7 @@
self.native = native
class Formats(object):
class Formats:
def __init__(self):
self._formats = {
......@@ -108,5 +106,4 @@
"""
if nativeonly:
return [f for f in self._formats if self.get_format(f).native]
else:
return [f for f in self._formats]
return list(self._formats)
views/menu.xml
+ 9
− 11
View file @ bb5f59d0
<odoo>
<data>
......@@ -3,6 +2,6 @@
<!--Setting > Redner -->
<menuitem id="redner_menu"
name="Redner"
parent="base.menu_custom" />
<!--Setting > Redner -->
<menuitem id="redner_menu"
name="Redner"
parent="base.menu_custom" />
......@@ -8,8 +7,7 @@
<!--Setting > Redner > Templates-->
<menuitem name="Templates" id="redner_template_menu"
parent="redner_menu"
action="redner_template_action"
sequence="8" />
</data>
<!--Setting > Redner > Templates-->
<menuitem name="Templates" id="redner_template_menu"
parent="redner_menu"
action="redner_template_action"
sequence="8" />
</odoo>
\ No newline at end of file
views/redner_template_views.xml
+ 74
− 76
View file @ bb5f59d0
<?xml version="1.0" encoding="utf-8"?>
<odoo>
<data>
<record id="redner_template_view_search" model="ir.ui.view">
<field name="name">redner.template.search</field>
<field name="model">redner.template</field>
<field name="arch" type="xml">
<search string="Redner">
<filter string="Archived" name="inactive" domain="[('active','=',False)]"/>
</search>
</field>
</record>
<record id="redner_template_view_search" model="ir.ui.view">
<field name="name">redner.template.search</field>
<field name="model">redner.template</field>
<field name="arch" type="xml">
<search string="Redner">
<filter string="Archived" name="inactive" domain="[('active','=',False)]"/>
</search>
</field>
</record>
<record id="redner_template_view_tree" model="ir.ui.view">
<field name="name">redner.template.tree</field>
<field name="model">redner.template</field>
<field name="arch" type="xml">
<tree>
<field name="name"/>
<field name="language"/>
<field name="detected_keywords"/>
</tree>
</field>
</record>
......@@ -13,13 +24,16 @@
<record id="redner_template_view_tree" model="ir.ui.view">
<field name="name">redner.template.tree</field>
<field name="model">redner.template</field>
<field name="arch" type="xml">
<tree>
<field name="name"/>
<field name="language"/>
<field name="detected_keywords"/>
</tree>
</field>
</record>
<record id="redner_template_view_form" model="ir.ui.view">
<field name="name">redner.template.form</field>
<field name="model">redner.template</field>
<field name="arch" type="xml">
<form string="Rednerd Template">
<sheet>
<div class="oe_button_box" name="button_box">
<button name="toggle_active" type="object"
class="oe_stat_button" icon="fa-archive">
<field name="active" widget="boolean_button"
options='{"terminology": "archive"}'/>
</button>
</div>
......@@ -25,8 +39,10 @@
<record id="redner_template_view_form" model="ir.ui.view">
<field name="name">redner.template.form</field>
<field name="model">redner.template</field>
<field name="arch" type="xml">
<form string="Rednerd Template">
<sheet>
<div class="oe_title">
<label for="name" class="oe_edit_only"/>
<h1><field name="name"/></h1>
<group>
<field name="language"/>
<field name="detected_keywords"/>
</group>
</div>
......@@ -32,13 +48,7 @@
<div class="oe_button_box" name="button_box">
<button name="toggle_active" type="object"
class="oe_stat_button" icon="fa-archive">
<field name="active" widget="boolean_button"
options='{"terminology": "archive"}'/>
</button>
</div>
<div class="oe_title">
<label for="name" class="oe_edit_only"/>
<h1><field name="name"/></h1>
<notebook>
<page string="Template Body" attrs="{'invisible': [('language', '=', 'application/vnd.oasis.opendocument.text|od+mustache')]}">
<field name="body" widget="ace" attrs="{'required': [('language', '!=', 'application/vnd.oasis.opendocument.text|od+mustache')]}"/>
</page>
<page string="Template Libreoffice" attrs="{'invisible': [('language', '!=', 'application/vnd.oasis.opendocument.text|od+mustache')]}">
<group>
......@@ -44,4 +54,3 @@
<group>
<field name="language"/>
<field name="detected_keywords"/>
<field name="template_data" filename="name" nolabel="1" />
</group>
......@@ -47,3 +56,9 @@
</group>
</div>
</page>
<page string="Settings">
<group>
<field name="locale_id" />
</group>
</page>
</notebook>
......@@ -49,17 +64,6 @@
<notebook>
<page string="Template Body" attrs="{'invisible': [('language', '=', 'application/vnd.oasis.opendocument.text|od+mustache')]}">
<field name="body" widget="ace" attrs="{'required': [('language', '!=', 'application/vnd.oasis.opendocument.text|od+mustache')]}"/>
</page>
<page string="Template Libreoffice" attrs="{'invisible': [('language', '!=', 'application/vnd.oasis.opendocument.text|od+mustache')]}">
<group>
<field name="template_data" filename="name" nolabel="1" />
</group>
</page>
<page string="Settings">
<group>
<field name="locale_id" />
</group>
</page>
</notebook>
</sheet>
</form>
</field>
</record>
......@@ -65,12 +69,7 @@
</sheet>
</form>
</field>
</record>
<record id="redner_template_action" model="ir.actions.act_window">
<field name="name">Redner Templates</field>
<field name="res_model">redner.template</field>
<field name="view_mode">tree,form</field>
</record>
<record id="redner_template_action" model="ir.actions.act_window">
<field name="name">Redner Templates</field>
<field name="res_model">redner.template</field>
<field name="view_mode">tree,form</field>
</record>
......@@ -76,11 +75,11 @@
<!-- Set up an action to send multiple templates to the redner server. -->
<record id="send_to_rednerd_server_action" model="ir.actions.server">
<field name="binding_model_id" ref="model_redner_template" />
<field name="code">records.send_to_rednerd_server()</field>
<field name="model_id" ref="model_redner_template" />
<field name="name">Send to rednerd server</field>
<field name="state">code</field>
<field name="type">ir.actions.server</field>
</record>
<!-- Set up an action to send multiple templates to the redner server. -->
<record id="send_to_rednerd_server_action" model="ir.actions.server">
<field name="binding_model_id" ref="model_redner_template" />
<field name="code">records.send_to_rednerd_server()</field>
<field name="model_id" ref="model_redner_template" />
<field name="name">Send to rednerd server</field>
<field name="state">code</field>
<field name="type">ir.actions.server</field>
</record>
......@@ -86,3 +85,2 @@
</data>
</odoo>
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment