mirror of
https://github.com/OCA/reporting-engine.git
synced 2025-02-16 16:30:38 +02:00
[MIG] sql_request_abstract: Migration to 13.0
This commit is contained in:
@@ -95,7 +95,6 @@ class SQLRequestMixin(models.AbstractModel):
|
|||||||
)
|
)
|
||||||
|
|
||||||
# Action Section
|
# Action Section
|
||||||
@api.multi
|
|
||||||
def button_validate_sql_expression(self):
|
def button_validate_sql_expression(self):
|
||||||
for item in self:
|
for item in self:
|
||||||
if item._clean_query_enabled:
|
if item._clean_query_enabled:
|
||||||
@@ -106,12 +105,10 @@ class SQLRequestMixin(models.AbstractModel):
|
|||||||
item._check_execution()
|
item._check_execution()
|
||||||
item.state = "sql_valid"
|
item.state = "sql_valid"
|
||||||
|
|
||||||
@api.multi
|
|
||||||
def button_set_draft(self):
|
def button_set_draft(self):
|
||||||
self.write({"state": "draft"})
|
self.write({"state": "draft"})
|
||||||
|
|
||||||
# API Section
|
# API Section
|
||||||
@api.multi
|
|
||||||
def _execute_sql_request(
|
def _execute_sql_request(
|
||||||
self,
|
self,
|
||||||
params=None,
|
params=None,
|
||||||
@@ -229,7 +226,6 @@ class SQLRequestMixin(models.AbstractModel):
|
|||||||
% (minor_version)
|
% (minor_version)
|
||||||
)
|
)
|
||||||
|
|
||||||
@api.multi
|
|
||||||
def _clean_query(self):
|
def _clean_query(self):
|
||||||
self.ensure_one()
|
self.ensure_one()
|
||||||
query = self.query.strip()
|
query = self.query.strip()
|
||||||
@@ -237,7 +233,6 @@ class SQLRequestMixin(models.AbstractModel):
|
|||||||
query = query[:-1]
|
query = query[:-1]
|
||||||
self.query = query
|
self.query = query
|
||||||
|
|
||||||
@api.multi
|
|
||||||
def _check_prohibited_words(self):
|
def _check_prohibited_words(self):
|
||||||
"""Check if the query contains prohibited words, to avoid maliscious
|
"""Check if the query contains prohibited words, to avoid maliscious
|
||||||
SQL requests"""
|
SQL requests"""
|
||||||
@@ -255,7 +250,6 @@ class SQLRequestMixin(models.AbstractModel):
|
|||||||
% (word)
|
% (word)
|
||||||
)
|
)
|
||||||
|
|
||||||
@api.multi
|
|
||||||
def _check_execution(self):
|
def _check_execution(self):
|
||||||
"""Ensure that the query is valid, trying to execute it. A rollback
|
"""Ensure that the query is valid, trying to execute it. A rollback
|
||||||
is done after."""
|
is done after."""
|
||||||
@@ -273,7 +267,6 @@ class SQLRequestMixin(models.AbstractModel):
|
|||||||
self._rollback_savepoint(rollback_name)
|
self._rollback_savepoint(rollback_name)
|
||||||
return res
|
return res
|
||||||
|
|
||||||
@api.multi
|
|
||||||
def _prepare_request_check_execution(self):
|
def _prepare_request_check_execution(self):
|
||||||
"""Overload me to replace some part of the query, if it contains
|
"""Overload me to replace some part of the query, if it contains
|
||||||
parameters"""
|
parameters"""
|
||||||
|
|||||||
@@ -1,2 +1,3 @@
|
|||||||
* Florian da Costa <florian.dacosta@akretion.com>
|
* Florian da Costa <florian.dacosta@akretion.com>
|
||||||
* Sylvain LE GAL (https://twitter.com/legalsylvain)
|
* Sylvain LE GAL (https://twitter.com/legalsylvain)
|
||||||
|
* Alfadil Tabar (alfadil.tabar@gmail.com)
|
||||||
|
|||||||
Reference in New Issue
Block a user