-
-
Notifications
You must be signed in to change notification settings - Fork 795
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
[18.0][MIG] sql_request_abstract: Migration to 18.0 #937
Merged
OCA-git-bot
merged 91 commits into
OCA:18.0
from
thienvh332:18.0-mig-sql_request_abstract
Nov 12, 2024
Merged
Changes from all commits
Commits
Show all changes
91 commits
Select commit
Hold shift + click to select a range
327ffd3
create a new module sql_request_abstract
legalsylvain c4e3fba
Fix version
florian-dacosta 6fbba2c
various improvement. Courtesy @dreispt review
legalsylvain 4983aea
OCA Transbot updated translations from Transifex
oca-transbot 60d1886
OCA Transbot updated translations from Transifex
oca-transbot 8a460a1
[MIG] sql_request_abstract v10
serpentcs-dev1 5866da7
OCA Transbot updated translations from Transifex
oca-transbot a1b5d5f
remove obsolete .pot files [ci skip]
sbidoul 77335eb
[UPD] Update sql_request_abstract.pot
oca-travis b9868b8
[FIX] mogrify doesn't allow dicts
MiquelRForgeFlow b6fa845
[UPD] Update sql_request_abstract.pot
oca-travis 6e17849
[MIG] sql_request_abstract: Migration to 11.0
HviorForgeFlow 086d22f
[UPD] Update sql_request_abstract.pot
oca-travis caacdf8
Update translation files
oca-transbot 4170f30
[12.0][MIG] Migrate module sql_request_abstract to v12
AdriaGForgeFlow 08cdd78
[UPD] README.rst
OCA-git-bot d235020
[UPD] Update sql_request_abstract.pot
oca-travis ec4dc6a
Update translation files
oca-transbot 2fa81b5
FIX sql_request_abstract: b64encode use a Bytes like object
bealdav ee965f3
[UPD] Update sql_request_abstract.pot
oca-travis 42533ae
Update translation files
oca-transbot c1e2084
[UPD] README.rst
OCA-git-bot d1fa806
Translated using Weblate (Chinese (Simplified))
liweijie0812 27fcad7
Translated using Weblate (Portuguese (Brazil))
18412dd
ADD module mrp_component_consumption_gap
bealdav aef382e
[UPD] Update sql_request_abstract.pot
oca-travis 9aabaf3
Update translation files
oca-transbot df942a4
[IMP] sql_request_abstract black, isort
eb100b4
[MIG] sql_request_abstract: Migration to 13.0
7a51687
[UPD] Update sql_request_abstract.pot
oca-travis 90fcf1b
[UPD] README.rst
OCA-git-bot 2d5c0fb
sql_request_abstract 13.0.1.0.1
OCA-git-bot d66a2e3
Update translation files
oca-transbot 67988fe
[IMP] sql_request_abstract: black, isort, prettier
mourad-ehm 0bcbe33
[MIG] sql_request_abstract: Migration to 14.0
mourad-ehm f3a441e
[UPD] Update sql_request_abstract.pot
oca-travis 34c4faa
[UPD] README.rst
OCA-git-bot 487d099
Migrate to 14.0
florian-dacosta dae5ee8
[UPD] Update sql_request_abstract.pot
oca-travis e25cb66
Update translation files
oca-transbot 3a57086
Translated using Weblate (French)
Yvesldff cf70b0f
Update roadmap for v15 and make some usability improvements
florian-dacosta ef1cefc
[UPD] README.rst
OCA-git-bot d45f40c
[PORT] 14.0 port #2061 : sql_export : add preview button
legalsylvain 0f39f66
[ADD] maintainers to module sql_request_abstract
legalsylvain ee9b1bc
[Fix]: Add header feature in latest v14
hkapatel-initos bffcad9
[UPD] README.rst
OCA-git-bot 91d6bc9
[UPD] README.rst
OCA-git-bot df48e75
sql_request_abstract 14.0.1.0.2
OCA-git-bot 6e588f2
sql_request_abstract 14.0.1.1.0
OCA-git-bot d05a4d1
[IMP] sql_request_abstract: black, isort, prettier
mariadforgeflow 5e52e97
[MIG] sql_request_abstract: Migration to 15.0
mariadforgeflow 77a166e
[UPD] Update sql_request_abstract.pot
d98eb8b
[UPD] README.rst
OCA-git-bot 5c876f8
Translated using Weblate (Catalan)
jabelchi 2645c5c
[MIG] sql_request_abstract from 15.0 to 16.0
legalsylvain 1bc695c
[UPD] Update sql_request_abstract.pot
ae1150e
[UPD] README.rst
OCA-git-bot f9591c5
Update translation files
weblate 588636e
Translated using Weblate (Spanish (Argentina))
ibuioli 1b328a0
Translated using Weblate (Italian)
mymage 08383e5
Translated using Weblate (Spanish)
Ivorra78 a424516
[UPD] Update sql_request_abstract.pot
5d3ffd8
Update translation files
weblate a723a37
[UPD] README.rst
OCA-git-bot bc8c266
Translated using Weblate (Spanish (Argentina))
ibuioli b86d57a
Translated using Weblate (Spanish)
Ivorra78 bb97de1
Update translation files
weblate e67fde2
[UPD] Update sql_request_abstract.pot
693ad0f
Update translation files
weblate d4ff237
Translated using Weblate (Italian)
mymage 8b5b181
[IMP] sql_request_abstract: pre-commit auto fixes
nguyenminhchien b125c4a
[MIG] sql_request_abstract: Migration to 17.0
nguyenminhchien e5d9ffe
[UPD] Update sql_request_abstract.pot
e232423
[BOT] post-merge updates
OCA-git-bot 0f82742
Update translation files
weblate 27e8fe8
Update translation files
weblate c72f4ac
Translated using Weblate (Spanish)
Ivorra78 5999c6c
Translated using Weblate (Spanish)
Ivorra78 0375996
Translated using Weblate (Italian)
mymage fc05f9d
Translated using Weblate (Swedish)
jakobkrabbe 728568b
[UPD] Update sql_request_abstract.pot
2b0a1a7
Update translation files
weblate 247486a
[UPD] Update sql_request_abstract.pot
54d20f5
Update translation files
weblate 464033c
Translated using Weblate (Spanish (Argentina))
ibuioli 154f376
[IMP] Make possible to make sql queries on an external database
florian-dacosta 1f0844c
[FIX] sql_request_abstract: regression
thomaspaulb 0b59e05
[IMP] sql_request_abstract: apply pre-commit auto fixes
thienvh332 a1f169c
[MIG] sql_request_abstract: Migration to 18.0
thienvh332 78b3649
add checklog-odoo.cfg
thienvh332 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
[checklog-odoo] | ||
ignore= | ||
WARNING.*0 failed, 0 error\(s\).* | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,158 @@ | ||
==================== | ||
SQL Request Abstract | ||
==================== | ||
|
||
.. | ||
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! | ||
!! This file is generated by oca-gen-addon-readme !! | ||
!! changes will be overwritten. !! | ||
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! | ||
!! source digest: sha256:d9671a76ee25d212c63c7f1a747acea88c2778536f30430a3e619352852e9bc9 | ||
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! | ||
|
||
.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png | ||
:target: https://odoo-community.org/page/development-status | ||
:alt: Beta | ||
.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png | ||
:target: http://www.gnu.org/licenses/agpl-3.0-standalone.html | ||
:alt: License: AGPL-3 | ||
.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Freporting--engine-lightgray.png?logo=github | ||
:target: https://github.com/OCA/reporting-engine/tree/18.0/sql_request_abstract | ||
:alt: OCA/reporting-engine | ||
.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png | ||
:target: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-sql_request_abstract | ||
:alt: Translate me on Weblate | ||
.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png | ||
:target: https://runboat.odoo-community.org/builds?repo=OCA/reporting-engine&target_branch=18.0 | ||
:alt: Try me on Runboat | ||
|
||
|badge1| |badge2| |badge3| |badge4| |badge5| | ||
|
||
This module provides an abstract model to manage SQL Select requests on | ||
database. It is not usefull for itself. You can see an exemple of | ||
implementation in the 'sql_export' module. (same repository). | ||
|
||
Implemented features | ||
-------------------- | ||
|
||
- Add some restrictions in the sql request: | ||
|
||
- you can only read datas. No update, deletion or creation are | ||
possible. | ||
- some tables are not allowed, because they could contains clear | ||
password or keys. For the time being ('ir_config_parameter'). | ||
|
||
- The request can be in a 'draft' or a 'SQL Valid' status. To be valid, | ||
the request has to be cleaned, checked and tested. All of this | ||
operations can be disabled in the inherited modules. | ||
|
||
- This module two new groups: | ||
|
||
- SQL Request / User : Can see all the sql requests by default and | ||
execute them, if they are valid. | ||
- SQL Request / Manager : has full access on sql requests. | ||
|
||
**Table of contents** | ||
|
||
.. contents:: | ||
:local: | ||
|
||
Configuration | ||
============= | ||
|
||
To configure the use of an external database, you need to edit the main | ||
configuration file of your instance and add the external database | ||
configuration with following keys : \* external_db_user \* | ||
external_db_password \* external_db_name \* external_db_host \* | ||
external_db_port | ||
|
||
Usage | ||
===== | ||
|
||
Inherit the model: | ||
|
||
.. code:: python | ||
|
||
from odoo import models | ||
|
||
class MyModel(models.model) | ||
_name = 'my.model' | ||
_inherit = ['sql.request.mixin'] | ||
|
||
_sql_request_groups_relation = 'my_model_groups_rel' | ||
|
||
_sql_request_users_relation = 'my_model_users_rel' | ||
|
||
See implementations in the modules ``bi_sql_editor`` and ``sql_export``. | ||
(same OCA/reporting-engine repository) | ||
|
||
Development | ||
=========== | ||
|
||
This module add the 'pgsql' mode syntax for the ace widget. (the ace | ||
widget is used in odoo web module, but only with the xml and python | ||
mode). | ||
|
||
The file is a copy of the file present here | ||
(https://github.com/ajaxorg/ace-builds/blob/v1.12.3/src/mode-pgsql.js | ||
(Release 18 Oct 2022) | ||
|
||
Bug Tracker | ||
=========== | ||
|
||
Bugs are tracked on `GitHub Issues <https://github.com/OCA/reporting-engine/issues>`_. | ||
In case of trouble, please check there if your issue has already been reported. | ||
If you spotted it first, help us to smash it by providing a detailed and welcomed | ||
`feedback <https://github.com/OCA/reporting-engine/issues/new?body=module:%20sql_request_abstract%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_. | ||
|
||
Do not contact contributors directly about support or help with technical issues. | ||
|
||
Credits | ||
======= | ||
|
||
Authors | ||
------- | ||
|
||
* GRAP | ||
* Akretion | ||
|
||
Contributors | ||
------------ | ||
|
||
- Florian da Costa <florian.dacosta@akretion.com> | ||
- Sylvain LE GAL (https://twitter.com/legalsylvain) | ||
- Alfadil Tabar (alfadil.tabar@gmail.com) | ||
- Helly kapatel <helly.kapatel@initos.com> | ||
- Nguyen Minh Chien <chien@trobz.com> | ||
- Vo Hong Thien <thienvh@trobz.com> | ||
|
||
Other credits | ||
------------- | ||
|
||
The migration of this module from 17.0 to 18.0 was financially supported | ||
by Camptocamp. | ||
|
||
Maintainers | ||
----------- | ||
|
||
This module is maintained by the OCA. | ||
|
||
.. image:: https://odoo-community.org/logo.png | ||
:alt: Odoo Community Association | ||
:target: https://odoo-community.org | ||
|
||
OCA, or the Odoo Community Association, is a nonprofit organization whose | ||
mission is to support the collaborative development of Odoo features and | ||
promote its widespread use. | ||
|
||
.. |maintainer-legalsylvain| image:: https://github.com/legalsylvain.png?size=40px | ||
:target: https://github.com/legalsylvain | ||
:alt: legalsylvain | ||
|
||
Current `maintainer <https://odoo-community.org/page/maintainer-role>`__: | ||
|
||
|maintainer-legalsylvain| | ||
|
||
This module is part of the `OCA/reporting-engine <https://github.com/OCA/reporting-engine/tree/18.0/sql_request_abstract>`_ project on GitHub. | ||
|
||
You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
from . import models | ||
from . import sql_db |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
# Copyright (C) 2017 - Today: GRAP (http://www.grap.coop) | ||
# @author: Sylvain LE GAL (https://twitter.com/legalsylvain) | ||
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). | ||
|
||
{ | ||
"name": "SQL Request Abstract", | ||
"version": "18.0.1.0.0", | ||
"author": "GRAP,Akretion,Odoo Community Association (OCA)", | ||
"maintainers": ["legalsylvain"], | ||
"website": "https://github.com/OCA/reporting-engine", | ||
"license": "AGPL-3", | ||
"category": "Tools", | ||
"summary": "Abstract Model to manage SQL Requests", | ||
"depends": ["mail"], | ||
"data": [ | ||
"security/ir_module_category.xml", | ||
"security/res_groups.xml", | ||
"security/ir.model.access.csv", | ||
"views/view_sql_request_mixin.xml", | ||
], | ||
"assets": { | ||
"web._assets_core": [ | ||
"sql_request_abstract/static/src/js/code_editor.esm.js", | ||
], | ||
"web.ace_lib": [ | ||
"sql_request_abstract/static/lib/ace/mode-pgsql.js", | ||
], | ||
}, | ||
"installable": True, | ||
} |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this addition ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This module originally had no test case and the new oca-ci test case was raising errors even with WARNING. So I added it to avoid being labeled as ERROR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hum. something is wrong if CI is red if there are no test, it is a valid use case. I think that something should be fixed in checklog-odoo tools instead.
@sbidoul : what do you think ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't want to special case this in checklog-odoo, because we have had situations where for some reason tests did not run had that error was actually helpful.
This problem will only occur when the first module migrated has no test, so in practice this should not occur often, I guess.