[MIG] sql_request_abstract: Migration to 13.0
parent
e9e09ae8ea
commit
bd57a83035
|
@ -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)
|
||||||
|
|
Loading…
Reference in New Issue