Compare commits
3 Commits
e04ab5b009
...
16.0-sale_
| Author | SHA1 | Date | |
|---|---|---|---|
| 5d8293f73e | |||
| a140edb60e | |||
| 64fca2a574 |
20
.editorconfig
Normal file
20
.editorconfig
Normal file
@@ -0,0 +1,20 @@
|
|||||||
|
# Configuration for known file extensions
|
||||||
|
[*.{css,js,json,less,md,py,rst,sass,scss,xml,yaml,yml}]
|
||||||
|
charset = utf-8
|
||||||
|
end_of_line = lf
|
||||||
|
indent_size = 4
|
||||||
|
indent_style = space
|
||||||
|
insert_final_newline = true
|
||||||
|
trim_trailing_whitespace = true
|
||||||
|
|
||||||
|
[*.{json,yml,yaml,rst,md}]
|
||||||
|
indent_size = 2
|
||||||
|
|
||||||
|
# Do not configure editor for libs and autogenerated content
|
||||||
|
[{*/static/{lib,src/lib}/**,*/static/description/index.html,*/readme/../README.rst}]
|
||||||
|
charset = unset
|
||||||
|
end_of_line = unset
|
||||||
|
indent_size = unset
|
||||||
|
indent_style = unset
|
||||||
|
insert_final_newline = false
|
||||||
|
trim_trailing_whitespace = false
|
||||||
188
.eslintrc.yml
Normal file
188
.eslintrc.yml
Normal file
@@ -0,0 +1,188 @@
|
|||||||
|
env:
|
||||||
|
browser: true
|
||||||
|
es6: true
|
||||||
|
|
||||||
|
# See https://github.com/OCA/odoo-community.org/issues/37#issuecomment-470686449
|
||||||
|
parserOptions:
|
||||||
|
ecmaVersion: 2019
|
||||||
|
|
||||||
|
overrides:
|
||||||
|
- files:
|
||||||
|
- "**/*.esm.js"
|
||||||
|
parserOptions:
|
||||||
|
sourceType: module
|
||||||
|
|
||||||
|
# Globals available in Odoo that shouldn't produce errorings
|
||||||
|
globals:
|
||||||
|
_: readonly
|
||||||
|
$: readonly
|
||||||
|
fuzzy: readonly
|
||||||
|
jQuery: readonly
|
||||||
|
moment: readonly
|
||||||
|
odoo: readonly
|
||||||
|
openerp: readonly
|
||||||
|
owl: readonly
|
||||||
|
luxon: readonly
|
||||||
|
|
||||||
|
# Styling is handled by Prettier, so we only need to enable AST rules;
|
||||||
|
# see https://github.com/OCA/maintainer-quality-tools/pull/618#issuecomment-558576890
|
||||||
|
rules:
|
||||||
|
accessor-pairs: warn
|
||||||
|
array-callback-return: warn
|
||||||
|
callback-return: warn
|
||||||
|
capitalized-comments:
|
||||||
|
- warn
|
||||||
|
- always
|
||||||
|
- ignoreConsecutiveComments: true
|
||||||
|
ignoreInlineComments: true
|
||||||
|
complexity:
|
||||||
|
- warn
|
||||||
|
- 15
|
||||||
|
constructor-super: warn
|
||||||
|
dot-notation: warn
|
||||||
|
eqeqeq: warn
|
||||||
|
global-require: warn
|
||||||
|
handle-callback-err: warn
|
||||||
|
id-blacklist: warn
|
||||||
|
id-match: warn
|
||||||
|
init-declarations: error
|
||||||
|
max-depth: warn
|
||||||
|
max-nested-callbacks: warn
|
||||||
|
max-statements-per-line: warn
|
||||||
|
no-alert: warn
|
||||||
|
no-array-constructor: warn
|
||||||
|
no-caller: warn
|
||||||
|
no-case-declarations: warn
|
||||||
|
no-class-assign: warn
|
||||||
|
no-cond-assign: error
|
||||||
|
no-const-assign: error
|
||||||
|
no-constant-condition: warn
|
||||||
|
no-control-regex: warn
|
||||||
|
no-debugger: error
|
||||||
|
no-delete-var: warn
|
||||||
|
no-div-regex: warn
|
||||||
|
no-dupe-args: error
|
||||||
|
no-dupe-class-members: error
|
||||||
|
no-dupe-keys: error
|
||||||
|
no-duplicate-case: error
|
||||||
|
no-duplicate-imports: error
|
||||||
|
no-else-return: warn
|
||||||
|
no-empty-character-class: warn
|
||||||
|
no-empty-function: error
|
||||||
|
no-empty-pattern: error
|
||||||
|
no-empty: warn
|
||||||
|
no-eq-null: error
|
||||||
|
no-eval: error
|
||||||
|
no-ex-assign: error
|
||||||
|
no-extend-native: warn
|
||||||
|
no-extra-bind: warn
|
||||||
|
no-extra-boolean-cast: warn
|
||||||
|
no-extra-label: warn
|
||||||
|
no-fallthrough: warn
|
||||||
|
no-func-assign: error
|
||||||
|
no-global-assign: error
|
||||||
|
no-implicit-coercion:
|
||||||
|
- warn
|
||||||
|
- allow: ["~"]
|
||||||
|
no-implicit-globals: warn
|
||||||
|
no-implied-eval: warn
|
||||||
|
no-inline-comments: warn
|
||||||
|
no-inner-declarations: warn
|
||||||
|
no-invalid-regexp: warn
|
||||||
|
no-irregular-whitespace: warn
|
||||||
|
no-iterator: warn
|
||||||
|
no-label-var: warn
|
||||||
|
no-labels: warn
|
||||||
|
no-lone-blocks: warn
|
||||||
|
no-lonely-if: error
|
||||||
|
no-mixed-requires: error
|
||||||
|
no-multi-str: warn
|
||||||
|
no-native-reassign: error
|
||||||
|
no-negated-condition: warn
|
||||||
|
no-negated-in-lhs: error
|
||||||
|
no-new-func: warn
|
||||||
|
no-new-object: warn
|
||||||
|
no-new-require: warn
|
||||||
|
no-new-symbol: warn
|
||||||
|
no-new-wrappers: warn
|
||||||
|
no-new: warn
|
||||||
|
no-obj-calls: warn
|
||||||
|
no-octal-escape: warn
|
||||||
|
no-octal: warn
|
||||||
|
no-param-reassign: warn
|
||||||
|
no-path-concat: warn
|
||||||
|
no-process-env: warn
|
||||||
|
no-process-exit: warn
|
||||||
|
no-proto: warn
|
||||||
|
no-prototype-builtins: warn
|
||||||
|
no-redeclare: warn
|
||||||
|
no-regex-spaces: warn
|
||||||
|
no-restricted-globals: warn
|
||||||
|
no-restricted-imports: warn
|
||||||
|
no-restricted-modules: warn
|
||||||
|
no-restricted-syntax: warn
|
||||||
|
no-return-assign: error
|
||||||
|
no-script-url: warn
|
||||||
|
no-self-assign: warn
|
||||||
|
no-self-compare: warn
|
||||||
|
no-sequences: warn
|
||||||
|
no-shadow-restricted-names: warn
|
||||||
|
no-shadow: warn
|
||||||
|
no-sparse-arrays: warn
|
||||||
|
no-sync: warn
|
||||||
|
no-this-before-super: warn
|
||||||
|
no-throw-literal: warn
|
||||||
|
no-undef-init: warn
|
||||||
|
no-undef: error
|
||||||
|
no-unmodified-loop-condition: warn
|
||||||
|
no-unneeded-ternary: error
|
||||||
|
no-unreachable: error
|
||||||
|
no-unsafe-finally: error
|
||||||
|
no-unused-expressions: error
|
||||||
|
no-unused-labels: error
|
||||||
|
no-unused-vars: error
|
||||||
|
no-use-before-define: error
|
||||||
|
no-useless-call: warn
|
||||||
|
no-useless-computed-key: warn
|
||||||
|
no-useless-concat: warn
|
||||||
|
no-useless-constructor: warn
|
||||||
|
no-useless-escape: warn
|
||||||
|
no-useless-rename: warn
|
||||||
|
no-void: warn
|
||||||
|
no-with: warn
|
||||||
|
operator-assignment: [error, always]
|
||||||
|
prefer-const: warn
|
||||||
|
radix: warn
|
||||||
|
require-yield: warn
|
||||||
|
sort-imports: warn
|
||||||
|
spaced-comment: [error, always]
|
||||||
|
strict: [error, function]
|
||||||
|
use-isnan: error
|
||||||
|
valid-jsdoc:
|
||||||
|
- warn
|
||||||
|
- prefer:
|
||||||
|
arg: param
|
||||||
|
argument: param
|
||||||
|
augments: extends
|
||||||
|
constructor: class
|
||||||
|
exception: throws
|
||||||
|
func: function
|
||||||
|
method: function
|
||||||
|
prop: property
|
||||||
|
return: returns
|
||||||
|
virtual: abstract
|
||||||
|
yield: yields
|
||||||
|
preferType:
|
||||||
|
array: Array
|
||||||
|
bool: Boolean
|
||||||
|
boolean: Boolean
|
||||||
|
number: Number
|
||||||
|
object: Object
|
||||||
|
str: String
|
||||||
|
string: String
|
||||||
|
requireParamDescription: false
|
||||||
|
requireReturn: false
|
||||||
|
requireReturnDescription: false
|
||||||
|
requireReturnType: false
|
||||||
|
valid-typeof: warn
|
||||||
|
yoda: warn
|
||||||
42
.gitea/workflows/pre-commit.yml
Normal file
42
.gitea/workflows/pre-commit.yml
Normal file
@@ -0,0 +1,42 @@
|
|||||||
|
name: pre-commit
|
||||||
|
|
||||||
|
on:
|
||||||
|
pull_request:
|
||||||
|
branches:
|
||||||
|
- "16.0*"
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
pre-commit:
|
||||||
|
runs-on: ubuntu-22.04
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
- uses: actions/setup-python@v5
|
||||||
|
with:
|
||||||
|
python-version: "3.11"
|
||||||
|
- name: Get python version
|
||||||
|
run: echo "PY=$(python -VV | sha256sum | cut -d' ' -f1)" >> $GITHUB_ENV
|
||||||
|
# - uses: actions/cache@v4
|
||||||
|
# with:
|
||||||
|
# path: ~/.cache/pre-commit
|
||||||
|
# key: pre-commit|${{ env.PY }}|${{ hashFiles('.pre-commit-config.yaml') }}
|
||||||
|
- name: Install pre-commit
|
||||||
|
run: pip install pre-commit
|
||||||
|
- name: Run pre-commit
|
||||||
|
run: pre-commit run --all-files --show-diff-on-failure --color=always
|
||||||
|
env:
|
||||||
|
# Consider valid a PR that changes README fragments but doesn't
|
||||||
|
# change the README.rst file itself. It's not really a problem
|
||||||
|
# because the bot will update it anyway after merge. This way, we
|
||||||
|
# lower the barrier for functional contributors that want to fix the
|
||||||
|
# readme fragments, while still letting developers get README
|
||||||
|
# auto-generated (which also helps functionals when using runboat).
|
||||||
|
# DOCS https://pre-commit.com/#temporarily-disabling-hooks
|
||||||
|
SKIP: oca-gen-addon-readme
|
||||||
|
- name: Check that all files generated by pre-commit are in git
|
||||||
|
run: |
|
||||||
|
newfiles="$(git ls-files --others --exclude-from=.gitignore)"
|
||||||
|
if [ "$newfiles" != "" ] ; then
|
||||||
|
echo "Please check-in the following files:"
|
||||||
|
echo "$newfiles"
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
150
.pre-commit-config.yaml
Normal file
150
.pre-commit-config.yaml
Normal file
@@ -0,0 +1,150 @@
|
|||||||
|
exclude: |
|
||||||
|
(?x)
|
||||||
|
# NOT INSTALLABLE ADDONS
|
||||||
|
# END NOT INSTALLABLE ADDONS
|
||||||
|
# Files and folders generated by bots, to avoid loops
|
||||||
|
^setup/|/static/description/index\.html$|
|
||||||
|
# We don't want to mess with tool-generated files
|
||||||
|
.svg$|/tests/([^/]+/)?cassettes/|^.copier-answers.yml$|^.github/|^eslint.config.cjs|^prettier.config.cjs|
|
||||||
|
# Maybe reactivate this when all README files include prettier ignore tags?
|
||||||
|
^README\.md$|
|
||||||
|
# Library files can have extraneous formatting (even minimized)
|
||||||
|
/static/(src/)?lib/|
|
||||||
|
# Repos using Sphinx to generate docs don't need prettying
|
||||||
|
^docs/_templates/.*\.html$|
|
||||||
|
# Don't bother non-technical authors with formatting issues in docs
|
||||||
|
readme/.*\.(rst|md)$|
|
||||||
|
# Ignore build and dist directories in addons
|
||||||
|
/build/|/dist/|
|
||||||
|
# Ignore test files in addons
|
||||||
|
/tests/samples/.*|
|
||||||
|
# You don't usually want a bot to modify your legal texts
|
||||||
|
(LICENSE.*|COPYING.*)
|
||||||
|
default_language_version:
|
||||||
|
python: python3
|
||||||
|
node: "16.17.0"
|
||||||
|
repos:
|
||||||
|
- repo: local
|
||||||
|
hooks:
|
||||||
|
# These files are most likely copier diff rejection junks; if found,
|
||||||
|
# review them manually, fix the problem (if needed) and remove them
|
||||||
|
- id: forbidden-files
|
||||||
|
name: forbidden files
|
||||||
|
entry: found forbidden files; remove them
|
||||||
|
language: fail
|
||||||
|
files: "\\.rej$"
|
||||||
|
- id: en-po-files
|
||||||
|
name: en.po files cannot exist
|
||||||
|
entry: found a en.po file
|
||||||
|
language: fail
|
||||||
|
files: '[a-zA-Z0-9_]*/i18n/en\.po$'
|
||||||
|
- repo: https://github.com/oca/maintainer-tools
|
||||||
|
rev: f9b919b9868143135a9c9cb03021089cabba8223
|
||||||
|
hooks:
|
||||||
|
# update the NOT INSTALLABLE ADDONS section above
|
||||||
|
- id: oca-update-pre-commit-excluded-addons
|
||||||
|
- id: oca-fix-manifest-website
|
||||||
|
entry:
|
||||||
|
bash -c 'oca-fix-manifest-website "https://git.elabore.coop/elabore/$(basename
|
||||||
|
$(git rev-parse --show-toplevel))"'
|
||||||
|
- id: oca-gen-addon-readme
|
||||||
|
entry:
|
||||||
|
bash -c 'oca-gen-addon-readme
|
||||||
|
--addons-dir=.
|
||||||
|
--branch=$(git symbolic-ref
|
||||||
|
refs/remotes/origin/HEAD | sed "s@^refs/remotes/origin/@@")
|
||||||
|
--repo-name=$(basename $(git rev-parse --show-toplevel))
|
||||||
|
--org-name="Elabore"
|
||||||
|
--if-source-changed --keep-source-digest'
|
||||||
|
|
||||||
|
- repo: https://github.com/OCA/odoo-pre-commit-hooks
|
||||||
|
rev: v0.1.4
|
||||||
|
hooks:
|
||||||
|
- id: oca-checks-odoo-module
|
||||||
|
- id: oca-checks-po
|
||||||
|
args:
|
||||||
|
- --disable=po-pretty-format
|
||||||
|
- repo: local
|
||||||
|
hooks:
|
||||||
|
- id: prettier
|
||||||
|
name: prettier (with plugin-xml)
|
||||||
|
entry: prettier
|
||||||
|
args:
|
||||||
|
- --write
|
||||||
|
- --list-different
|
||||||
|
- --ignore-unknown
|
||||||
|
types: [text]
|
||||||
|
files: \.(css|htm|html|js|json|jsx|less|md|scss|toml|ts|xml|yaml|yml)$
|
||||||
|
language: node
|
||||||
|
additional_dependencies:
|
||||||
|
- "prettier@2.7.1"
|
||||||
|
- "@prettier/plugin-xml@2.2.0"
|
||||||
|
- repo: local
|
||||||
|
hooks:
|
||||||
|
- id: eslint
|
||||||
|
name: eslint
|
||||||
|
entry: eslint
|
||||||
|
args:
|
||||||
|
- --color
|
||||||
|
- --fix
|
||||||
|
verbose: true
|
||||||
|
types: [javascript]
|
||||||
|
language: node
|
||||||
|
additional_dependencies:
|
||||||
|
- "eslint@8.24.0"
|
||||||
|
- "eslint-plugin-jsdoc@"
|
||||||
|
- repo: https://github.com/pre-commit/pre-commit-hooks
|
||||||
|
rev: v5.0.0
|
||||||
|
hooks:
|
||||||
|
- id: trailing-whitespace
|
||||||
|
# exclude autogenerated files
|
||||||
|
exclude: /README\.rst$|\.pot?$
|
||||||
|
- id: end-of-file-fixer
|
||||||
|
# exclude autogenerated files
|
||||||
|
exclude: /README\.rst$|\.pot?$
|
||||||
|
- id: debug-statements
|
||||||
|
- id: fix-encoding-pragma
|
||||||
|
args: ["--remove"]
|
||||||
|
- id: check-case-conflict
|
||||||
|
- id: check-docstring-first
|
||||||
|
- id: check-executables-have-shebangs
|
||||||
|
- id: check-merge-conflict
|
||||||
|
# exclude files where underlines are not distinguishable from merge conflicts
|
||||||
|
exclude: /README\.rst$|^docs/.*\.rst$
|
||||||
|
- id: check-symlinks
|
||||||
|
- id: check-xml
|
||||||
|
- id: mixed-line-ending
|
||||||
|
args: ["--fix=lf"]
|
||||||
|
- repo: https://github.com/PyCQA/docformatter
|
||||||
|
rev: v1.7.7
|
||||||
|
hooks:
|
||||||
|
- id: docformatter
|
||||||
|
args: [
|
||||||
|
"--in-place", # modify the files
|
||||||
|
"--recursive", # run on all the files
|
||||||
|
"--wrap-summaries",
|
||||||
|
"88", # max length of 1st line
|
||||||
|
"--wrap-descriptions",
|
||||||
|
"88", # max length of other lines
|
||||||
|
"--pre-summary-newline", # new line before a long summary
|
||||||
|
"--make-summary-multi-line", # force summary on multilines
|
||||||
|
]
|
||||||
|
additional_dependencies: ["tomli"] # if Python <3.11
|
||||||
|
- repo: https://github.com/astral-sh/ruff-pre-commit
|
||||||
|
rev: v0.12.0
|
||||||
|
hooks:
|
||||||
|
- id: ruff
|
||||||
|
args: [--fix, --exit-non-zero-on-fix]
|
||||||
|
- id: ruff-format
|
||||||
|
- repo: https://github.com/OCA/pylint-odoo
|
||||||
|
rev: v9.1.3
|
||||||
|
hooks:
|
||||||
|
- id: pylint_odoo
|
||||||
|
name: pylint with optional checks
|
||||||
|
args:
|
||||||
|
- --rcfile=.pylintrc
|
||||||
|
- --exit-zero
|
||||||
|
verbose: true
|
||||||
|
- id: pylint_odoo
|
||||||
|
args:
|
||||||
|
- --rcfile=.pylintrc-mandatory
|
||||||
8
.prettierrc.yml
Normal file
8
.prettierrc.yml
Normal file
@@ -0,0 +1,8 @@
|
|||||||
|
# Defaults for all prettier-supported languages.
|
||||||
|
# Prettier will complete this with settings from .editorconfig file.
|
||||||
|
bracketSpacing: false
|
||||||
|
printWidth: 88
|
||||||
|
proseWrap: always
|
||||||
|
semi: true
|
||||||
|
trailingComma: "es5"
|
||||||
|
xmlWhitespaceSensitivity: "strict"
|
||||||
123
.pylintrc
Normal file
123
.pylintrc
Normal file
@@ -0,0 +1,123 @@
|
|||||||
|
|
||||||
|
|
||||||
|
[MASTER]
|
||||||
|
load-plugins=pylint_odoo
|
||||||
|
score=n
|
||||||
|
|
||||||
|
[ODOOLINT]
|
||||||
|
readme-template-url="https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst"
|
||||||
|
manifest-required-authors=Elabore
|
||||||
|
manifest-required-keys=license
|
||||||
|
manifest-deprecated-keys=description,active
|
||||||
|
license-allowed=AGPL-3,GPL-2,GPL-2 or any later version,GPL-3,GPL-3 or any later version,LGPL-3
|
||||||
|
valid-odoo-versions=16.0
|
||||||
|
|
||||||
|
[MESSAGES CONTROL]
|
||||||
|
disable=all
|
||||||
|
|
||||||
|
# This .pylintrc contains optional AND mandatory checks and is meant to be
|
||||||
|
# loaded in an IDE to have it check everything, in the hope this will make
|
||||||
|
# optional checks more visible to contributors who otherwise never look at a
|
||||||
|
# green travis to see optional checks that failed.
|
||||||
|
# .pylintrc-mandatory containing only mandatory checks is used the pre-commit
|
||||||
|
# config as a blocking check.
|
||||||
|
|
||||||
|
enable=anomalous-backslash-in-string,
|
||||||
|
api-one-deprecated,
|
||||||
|
api-one-multi-together,
|
||||||
|
assignment-from-none,
|
||||||
|
attribute-deprecated,
|
||||||
|
class-camelcase,
|
||||||
|
dangerous-default-value,
|
||||||
|
dangerous-view-replace-wo-priority,
|
||||||
|
development-status-allowed,
|
||||||
|
duplicate-id-csv,
|
||||||
|
duplicate-key,
|
||||||
|
duplicate-xml-fields,
|
||||||
|
duplicate-xml-record-id,
|
||||||
|
eval-referenced,
|
||||||
|
eval-used,
|
||||||
|
incoherent-interpreter-exec-perm,
|
||||||
|
license-allowed,
|
||||||
|
manifest-author-string,
|
||||||
|
manifest-deprecated-key,
|
||||||
|
manifest-required-author,
|
||||||
|
manifest-required-key,
|
||||||
|
manifest-version-format,
|
||||||
|
method-compute,
|
||||||
|
method-inverse,
|
||||||
|
method-required-super,
|
||||||
|
method-search,
|
||||||
|
openerp-exception-warning,
|
||||||
|
pointless-statement,
|
||||||
|
pointless-string-statement,
|
||||||
|
print-used,
|
||||||
|
redundant-keyword-arg,
|
||||||
|
redundant-modulename-xml,
|
||||||
|
reimported,
|
||||||
|
relative-import,
|
||||||
|
return-in-init,
|
||||||
|
rst-syntax-error,
|
||||||
|
sql-injection,
|
||||||
|
too-few-format-args,
|
||||||
|
translation-field,
|
||||||
|
translation-required,
|
||||||
|
unreachable,
|
||||||
|
use-vim-comment,
|
||||||
|
wrong-tabs-instead-of-spaces,
|
||||||
|
xml-syntax-error,
|
||||||
|
attribute-string-redundant,
|
||||||
|
character-not-valid-in-resource-link,
|
||||||
|
consider-merging-classes-inherited,
|
||||||
|
context-overridden,
|
||||||
|
create-user-wo-reset-password,
|
||||||
|
dangerous-filter-wo-user,
|
||||||
|
dangerous-qweb-replace-wo-priority,
|
||||||
|
deprecated-data-xml-node,
|
||||||
|
deprecated-openerp-xml-node,
|
||||||
|
duplicate-po-message-definition,
|
||||||
|
except-pass,
|
||||||
|
file-not-used,
|
||||||
|
invalid-commit,
|
||||||
|
manifest-maintainers-list,
|
||||||
|
missing-newline-extrafiles,
|
||||||
|
missing-readme,
|
||||||
|
missing-return,
|
||||||
|
odoo-addons-relative-import,
|
||||||
|
old-api7-method-defined,
|
||||||
|
po-msgstr-variables,
|
||||||
|
po-syntax-error,
|
||||||
|
renamed-field-parameter,
|
||||||
|
resource-not-exist,
|
||||||
|
str-format-used,
|
||||||
|
test-folder-imported,
|
||||||
|
translation-contains-variable,
|
||||||
|
translation-positional-used,
|
||||||
|
unnecessary-utf8-coding-comment,
|
||||||
|
website-manifest-key-not-valid-uri,
|
||||||
|
xml-attribute-translatable,
|
||||||
|
xml-deprecated-qweb-directive,
|
||||||
|
xml-deprecated-tree-attribute,
|
||||||
|
external-request-timeout,
|
||||||
|
# messages that do not cause the lint step to fail
|
||||||
|
consider-merging-classes-inherited,
|
||||||
|
create-user-wo-reset-password,
|
||||||
|
dangerous-filter-wo-user,
|
||||||
|
deprecated-module,
|
||||||
|
file-not-used,
|
||||||
|
invalid-commit,
|
||||||
|
missing-manifest-dependency,
|
||||||
|
missing-newline-extrafiles,
|
||||||
|
missing-readme,
|
||||||
|
no-utf8-coding-comment,
|
||||||
|
odoo-addons-relative-import,
|
||||||
|
old-api7-method-defined,
|
||||||
|
redefined-builtin,
|
||||||
|
too-complex,
|
||||||
|
unnecessary-utf8-coding-comment
|
||||||
|
|
||||||
|
|
||||||
|
[REPORTS]
|
||||||
|
msg-template={path}:{line}: [{msg_id}({symbol}), {obj}] {msg}
|
||||||
|
output-format=colorized
|
||||||
|
reports=no
|
||||||
98
.pylintrc-mandatory
Normal file
98
.pylintrc-mandatory
Normal file
@@ -0,0 +1,98 @@
|
|||||||
|
|
||||||
|
[MASTER]
|
||||||
|
load-plugins=pylint_odoo
|
||||||
|
score=n
|
||||||
|
|
||||||
|
[ODOOLINT]
|
||||||
|
readme-template-url="https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst"
|
||||||
|
manifest-required-authors=Elabore
|
||||||
|
manifest-required-keys=license
|
||||||
|
manifest-deprecated-keys=description,active
|
||||||
|
license-allowed=AGPL-3,GPL-2,GPL-2 or any later version,GPL-3,GPL-3 or any later version,LGPL-3
|
||||||
|
valid-odoo-versions=16.0
|
||||||
|
|
||||||
|
[MESSAGES CONTROL]
|
||||||
|
disable=all
|
||||||
|
|
||||||
|
enable=anomalous-backslash-in-string,
|
||||||
|
api-one-deprecated,
|
||||||
|
api-one-multi-together,
|
||||||
|
assignment-from-none,
|
||||||
|
attribute-deprecated,
|
||||||
|
class-camelcase,
|
||||||
|
dangerous-default-value,
|
||||||
|
dangerous-view-replace-wo-priority,
|
||||||
|
development-status-allowed,
|
||||||
|
duplicate-id-csv,
|
||||||
|
duplicate-key,
|
||||||
|
duplicate-xml-fields,
|
||||||
|
duplicate-xml-record-id,
|
||||||
|
eval-referenced,
|
||||||
|
eval-used,
|
||||||
|
incoherent-interpreter-exec-perm,
|
||||||
|
license-allowed,
|
||||||
|
manifest-author-string,
|
||||||
|
manifest-deprecated-key,
|
||||||
|
manifest-required-author,
|
||||||
|
manifest-required-key,
|
||||||
|
manifest-version-format,
|
||||||
|
method-compute,
|
||||||
|
method-inverse,
|
||||||
|
method-required-super,
|
||||||
|
method-search,
|
||||||
|
openerp-exception-warning,
|
||||||
|
pointless-statement,
|
||||||
|
pointless-string-statement,
|
||||||
|
print-used,
|
||||||
|
redundant-keyword-arg,
|
||||||
|
redundant-modulename-xml,
|
||||||
|
reimported,
|
||||||
|
relative-import,
|
||||||
|
return-in-init,
|
||||||
|
rst-syntax-error,
|
||||||
|
sql-injection,
|
||||||
|
too-few-format-args,
|
||||||
|
translation-field,
|
||||||
|
translation-required,
|
||||||
|
unreachable,
|
||||||
|
use-vim-comment,
|
||||||
|
wrong-tabs-instead-of-spaces,
|
||||||
|
xml-syntax-error,
|
||||||
|
attribute-string-redundant,
|
||||||
|
character-not-valid-in-resource-link,
|
||||||
|
consider-merging-classes-inherited,
|
||||||
|
context-overridden,
|
||||||
|
create-user-wo-reset-password,
|
||||||
|
dangerous-filter-wo-user,
|
||||||
|
dangerous-qweb-replace-wo-priority,
|
||||||
|
deprecated-data-xml-node,
|
||||||
|
deprecated-openerp-xml-node,
|
||||||
|
duplicate-po-message-definition,
|
||||||
|
except-pass,
|
||||||
|
file-not-used,
|
||||||
|
invalid-commit,
|
||||||
|
manifest-maintainers-list,
|
||||||
|
missing-newline-extrafiles,
|
||||||
|
missing-readme,
|
||||||
|
missing-return,
|
||||||
|
odoo-addons-relative-import,
|
||||||
|
old-api7-method-defined,
|
||||||
|
po-msgstr-variables,
|
||||||
|
po-syntax-error,
|
||||||
|
renamed-field-parameter,
|
||||||
|
resource-not-exist,
|
||||||
|
str-format-used,
|
||||||
|
test-folder-imported,
|
||||||
|
translation-contains-variable,
|
||||||
|
translation-positional-used,
|
||||||
|
unnecessary-utf8-coding-comment,
|
||||||
|
website-manifest-key-not-valid-uri,
|
||||||
|
xml-attribute-translatable,
|
||||||
|
xml-deprecated-qweb-directive,
|
||||||
|
xml-deprecated-tree-attribute,
|
||||||
|
external-request-timeout
|
||||||
|
|
||||||
|
[REPORTS]
|
||||||
|
msg-template={path}:{line}: [{msg_id}({symbol}), {obj}] {msg}
|
||||||
|
output-format=colorized
|
||||||
|
reports=no
|
||||||
31
.ruff.toml
Normal file
31
.ruff.toml
Normal file
@@ -0,0 +1,31 @@
|
|||||||
|
|
||||||
|
target-version = "py310"
|
||||||
|
fix = true
|
||||||
|
|
||||||
|
[lint]
|
||||||
|
extend-select = [
|
||||||
|
"B",
|
||||||
|
"C90",
|
||||||
|
"E501", # line too long (default 88)
|
||||||
|
"I", # isort
|
||||||
|
"UP", # pyupgrade
|
||||||
|
]
|
||||||
|
extend-safe-fixes = ["UP008"]
|
||||||
|
exclude = ["setup/*"]
|
||||||
|
|
||||||
|
[format]
|
||||||
|
exclude = ["setup/*"]
|
||||||
|
|
||||||
|
[lint.per-file-ignores]
|
||||||
|
"__init__.py" = ["F401", "I001"] # ignore unused and unsorted imports in __init__.py
|
||||||
|
"__manifest__.py" = ["B018"] # useless expression
|
||||||
|
|
||||||
|
[lint.isort]
|
||||||
|
section-order = ["future", "standard-library", "third-party", "odoo", "odoo-addons", "first-party", "local-folder"]
|
||||||
|
|
||||||
|
[lint.isort.sections]
|
||||||
|
"odoo" = ["odoo"]
|
||||||
|
"odoo-addons" = ["odoo.addons"]
|
||||||
|
|
||||||
|
[lint.mccabe]
|
||||||
|
max-complexity = 16
|
||||||
1
sale_restricted_qty_website_sale/__init__.py
Normal file
1
sale_restricted_qty_website_sale/__init__.py
Normal file
@@ -0,0 +1 @@
|
|||||||
|
from . import models
|
||||||
35
sale_restricted_qty_website_sale/__manifest__.py
Normal file
35
sale_restricted_qty_website_sale/__manifest__.py
Normal file
@@ -0,0 +1,35 @@
|
|||||||
|
# Copyright 2026 Elabore ()
|
||||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
|
||||||
|
|
||||||
|
{
|
||||||
|
"name": "sale_restricted_qty_website_sale",
|
||||||
|
"version": "16.0.1.0.0",
|
||||||
|
"author": "Elabore",
|
||||||
|
"website": "https://elabore.coop",
|
||||||
|
"maintainer": "Elabore",
|
||||||
|
"license": "AGPL-3",
|
||||||
|
"category": "Sales",
|
||||||
|
"summary": "Improve UX handling on e-commerce product page",
|
||||||
|
# any module necessary for this one to work correctly
|
||||||
|
"depends": [
|
||||||
|
"sale_restricted_qty","website_sale",
|
||||||
|
],
|
||||||
|
"qweb": [],
|
||||||
|
"external_dependencies": {
|
||||||
|
"python": [],
|
||||||
|
},
|
||||||
|
# always loaded
|
||||||
|
"data": [
|
||||||
|
"views/restricted_qty_website_sale_view.xml"
|
||||||
|
],
|
||||||
|
"assets": {
|
||||||
|
"web.assets_frontend": [
|
||||||
|
"sale_restricted_qty_website_sale/static/src/js/add_to_cart.js",
|
||||||
|
],
|
||||||
|
},
|
||||||
|
"installable": True,
|
||||||
|
# Install this module automatically if all dependency have been previously
|
||||||
|
# and independently installed. Used for synergetic or glue modules.
|
||||||
|
"auto_install": True,
|
||||||
|
"application": False,
|
||||||
|
}
|
||||||
80
sale_restricted_qty_website_sale/i18n/fr.po
Normal file
80
sale_restricted_qty_website_sale/i18n/fr.po
Normal file
@@ -0,0 +1,80 @@
|
|||||||
|
# Translation of Odoo Server.
|
||||||
|
# This file contains the translation of the following modules:
|
||||||
|
# * elabore_website_sale_restricted_qty
|
||||||
|
#
|
||||||
|
msgid ""
|
||||||
|
msgstr ""
|
||||||
|
"Project-Id-Version: Odoo Server 16.0\n"
|
||||||
|
"Report-Msgid-Bugs-To: \n"
|
||||||
|
"PO-Revision-Date: 2026-08-18 00:00+0000\n"
|
||||||
|
"Last-Translator: Élabore <https://elabore.coop>\n"
|
||||||
|
"Language-Team: \n"
|
||||||
|
"Language: fr\n"
|
||||||
|
"MIME-Version: 1.0\n"
|
||||||
|
"Content-Type: text/plain; charset=UTF-8\n"
|
||||||
|
"Content-Transfer-Encoding: 8bit\n"
|
||||||
|
"Plural-Forms: nplurals=2; plural=(n > 1);\n"
|
||||||
|
|
||||||
|
#. module: elabore_website_sale_restricted_qty
|
||||||
|
#. odoo-python
|
||||||
|
#: code:addons/elabore_website_sale_restricted_qty/models/sale_order_line.py:0
|
||||||
|
#, python-format
|
||||||
|
msgid ""
|
||||||
|
"The selected quantity is not available for sale:\n"
|
||||||
|
"\n"
|
||||||
|
"%s\n"
|
||||||
|
"\n"
|
||||||
|
"Please adjust the quantity."
|
||||||
|
msgstr ""
|
||||||
|
"La quantité choisie n'est pas disponible à la vente :\n"
|
||||||
|
"\n"
|
||||||
|
"%s\n"
|
||||||
|
"\n"
|
||||||
|
"Merci d'ajuster la quantité."
|
||||||
|
|
||||||
|
#. module: elabore_website_sale_restricted_qty
|
||||||
|
#. odoo-python
|
||||||
|
#: code:addons/elabore_website_sale_restricted_qty/models/sale_order_line.py:0
|
||||||
|
#, python-format
|
||||||
|
msgid "%(product)s: minimum order quantity is %(qty)s."
|
||||||
|
msgstr "%(product)s : quantité minimum de commande %(qty)s."
|
||||||
|
|
||||||
|
#. module: elabore_website_sale_restricted_qty
|
||||||
|
#. odoo-python
|
||||||
|
#: code:addons/elabore_website_sale_restricted_qty/models/sale_order_line.py:0
|
||||||
|
#, python-format
|
||||||
|
msgid "%(product)s: maximum order quantity is %(qty)s."
|
||||||
|
msgstr "%(product)s : quantité maximum de commande %(qty)s."
|
||||||
|
|
||||||
|
#. module: elabore_website_sale_restricted_qty
|
||||||
|
#. odoo-python
|
||||||
|
#: code:addons/elabore_website_sale_restricted_qty/models/sale_order_line.py:0
|
||||||
|
#, python-format
|
||||||
|
msgid "%(product)s: must be ordered in multiples of %(qty)s."
|
||||||
|
msgstr "%(product)s : à commander par multiples de %(qty)s."
|
||||||
|
|
||||||
|
#. module: elabore_website_sale_restricted_qty
|
||||||
|
#. odoo-python
|
||||||
|
#: code:addons/elabore_website_sale_restricted_qty/models/product_restricted_qty_mixin.py:0
|
||||||
|
#, python-format
|
||||||
|
msgid "minimum order: %(qty)s %(uom)s"
|
||||||
|
msgstr "commande minimum : %(qty)s %(uom)s"
|
||||||
|
|
||||||
|
#. module: elabore_website_sale_restricted_qty
|
||||||
|
#. odoo-python
|
||||||
|
#: code:addons/elabore_website_sale_restricted_qty/models/product_restricted_qty_mixin.py:0
|
||||||
|
#, python-format
|
||||||
|
msgid "in multiples of %(qty)s %(uom)s"
|
||||||
|
msgstr "par multiples de %(qty)s %(uom)s"
|
||||||
|
|
||||||
|
#. module: elabore_website_sale_restricted_qty
|
||||||
|
#. odoo-python
|
||||||
|
#: code:addons/elabore_website_sale_restricted_qty/models/product_restricted_qty_mixin.py:0
|
||||||
|
#, python-format
|
||||||
|
msgid "maximum: %(qty)s %(uom)s"
|
||||||
|
msgstr "maximum : %(qty)s %(uom)s"
|
||||||
|
|
||||||
|
#. module: elabore_website_sale_restricted_qty
|
||||||
|
#: model:ir.ui.view,name:elabore_website_sale_restricted_qty.product_restricted_qty_info
|
||||||
|
msgid "Restricted Quantity Information"
|
||||||
|
msgstr "Information sur les quantités restreintes"
|
||||||
2
sale_restricted_qty_website_sale/models/__init__.py
Normal file
2
sale_restricted_qty_website_sale/models/__init__.py
Normal file
@@ -0,0 +1,2 @@
|
|||||||
|
from . import product_restricted_qty_mixin
|
||||||
|
from . import sale_order_line
|
||||||
@@ -0,0 +1,59 @@
|
|||||||
|
# Copyright 2026 Élabore (https://elabore.coop)
|
||||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
|
||||||
|
|
||||||
|
from odoo import _, models
|
||||||
|
from odoo.tools import float_is_zero
|
||||||
|
from odoo.tools.misc import formatLang
|
||||||
|
|
||||||
|
|
||||||
|
class ProductRestrictedQtyMixin(models.AbstractModel):
|
||||||
|
"""Extending the OCA abstract mixin makes these helpers available at once
|
||||||
|
on product.product, product.template and product.category."""
|
||||||
|
|
||||||
|
_inherit = "product.restricted.qty.mixin"
|
||||||
|
|
||||||
|
def _format_restricted_qty(self, qty):
|
||||||
|
"""Short form: 6 rather than 6.0, and a locale-aware separator
|
||||||
|
for decimals."""
|
||||||
|
if float_is_zero(qty - int(qty), precision_digits=3):
|
||||||
|
return str(int(qty))
|
||||||
|
return formatLang(self.env, qty)
|
||||||
|
|
||||||
|
def _get_restricted_qty_hint(self):
|
||||||
|
"""Sentence shown to the customer. Empty when nothing restricts him.
|
||||||
|
|
||||||
|
OCA semantics: `force_sale_min_qty` / `force_sale_max_qty` set to Yes
|
||||||
|
make the bound purely indicative -- the constraint does not block, so
|
||||||
|
we announce nothing rather than promising a rule that does not exist.
|
||||||
|
"""
|
||||||
|
self.ensure_one()
|
||||||
|
uom = self.uom_id.name if "uom_id" in self._fields else ""
|
||||||
|
parts = []
|
||||||
|
if self.sale_min_qty and not self.force_sale_min_qty:
|
||||||
|
parts.append(
|
||||||
|
_(
|
||||||
|
"minimum order: %(qty)s %(uom)s",
|
||||||
|
qty=self._format_restricted_qty(self.sale_min_qty),
|
||||||
|
uom=uom,
|
||||||
|
)
|
||||||
|
)
|
||||||
|
if self.sale_multiple_qty:
|
||||||
|
parts.append(
|
||||||
|
_(
|
||||||
|
"in multiples of %(qty)s %(uom)s",
|
||||||
|
qty=self._format_restricted_qty(self.sale_multiple_qty),
|
||||||
|
uom=uom,
|
||||||
|
)
|
||||||
|
)
|
||||||
|
if self.sale_max_qty and not self.force_sale_max_qty:
|
||||||
|
parts.append(
|
||||||
|
_(
|
||||||
|
"maximum: %(qty)s %(uom)s",
|
||||||
|
qty=self._format_restricted_qty(self.sale_max_qty),
|
||||||
|
uom=uom,
|
||||||
|
)
|
||||||
|
)
|
||||||
|
if not parts:
|
||||||
|
return ""
|
||||||
|
hint = " · ".join(parts)
|
||||||
|
return hint[0].upper() + hint[1:]
|
||||||
62
sale_restricted_qty_website_sale/models/sale_order_line.py
Normal file
62
sale_restricted_qty_website_sale/models/sale_order_line.py
Normal file
@@ -0,0 +1,62 @@
|
|||||||
|
from odoo import _, api, models
|
||||||
|
from odoo.exceptions import ValidationError
|
||||||
|
|
||||||
|
|
||||||
|
class SaleOrderLine(models.Model):
|
||||||
|
_inherit = "sale.order.line"
|
||||||
|
|
||||||
|
@api.constrains(
|
||||||
|
"product_uom_qty", "sale_min_qty", "sale_max_qty", "sale_multiple_qty"
|
||||||
|
)
|
||||||
|
def check_constraint_restricted_qty(self):
|
||||||
|
"""Rewrite the error message raised by `sale_restricted_qty`.
|
||||||
|
|
||||||
|
The upstream message is written for a salesperson: it dumps the
|
||||||
|
offending products and explains how to tick "force min quantity" on
|
||||||
|
the product form. Unreadable for an eCommerce customer.
|
||||||
|
|
||||||
|
Detection is left to the OCA module -- the `is_qty_*` fields stay the
|
||||||
|
single source of truth. Only the wording is replaced, so an upstream
|
||||||
|
change to the rules is followed without breaking anything.
|
||||||
|
"""
|
||||||
|
try:
|
||||||
|
return super().check_constraint_restricted_qty()
|
||||||
|
except ValidationError:
|
||||||
|
raise ValidationError(self._get_restricted_qty_error_message()) from None
|
||||||
|
|
||||||
|
def _get_restricted_qty_error_message(self):
|
||||||
|
"""Customer-facing message, one line per unmet restriction."""
|
||||||
|
details = []
|
||||||
|
for line in self:
|
||||||
|
product = line.product_id
|
||||||
|
if not product.force_sale_min_qty and line.is_qty_less_min_qty:
|
||||||
|
details.append(
|
||||||
|
_(
|
||||||
|
"%(product)s: minimum order quantity is %(qty)s.",
|
||||||
|
product=product.display_name,
|
||||||
|
qty=product._format_restricted_qty(line.sale_min_qty),
|
||||||
|
)
|
||||||
|
)
|
||||||
|
if not product.force_sale_max_qty and line.is_qty_bigger_max_qty:
|
||||||
|
details.append(
|
||||||
|
_(
|
||||||
|
"%(product)s: maximum order quantity is %(qty)s.",
|
||||||
|
product=product.display_name,
|
||||||
|
qty=product._format_restricted_qty(line.sale_max_qty),
|
||||||
|
)
|
||||||
|
)
|
||||||
|
if line.is_qty_not_multiple_qty:
|
||||||
|
details.append(
|
||||||
|
_(
|
||||||
|
"%(product)s: must be ordered in multiples of %(qty)s.",
|
||||||
|
product=product.display_name,
|
||||||
|
qty=product._format_restricted_qty(line.sale_multiple_qty),
|
||||||
|
)
|
||||||
|
)
|
||||||
|
|
||||||
|
return _(
|
||||||
|
"The selected quantity is not available for sale:\n\n"
|
||||||
|
"%s\n\n"
|
||||||
|
"Please adjust the quantity.",
|
||||||
|
"\n".join(details),
|
||||||
|
)
|
||||||
@@ -0,0 +1,45 @@
|
|||||||
|
odoo.define("sale_restricted_qty_website_sale.add_to_cart", function (require) {
|
||||||
|
"use strict";
|
||||||
|
|
||||||
|
const publicWidget = require("web.public.widget");
|
||||||
|
const wSaleUtils = require("website_sale.utils");
|
||||||
|
const WebsiteSale = require("website_sale.website_sale").WebsiteSale;
|
||||||
|
|
||||||
|
WebsiteSale.include({
|
||||||
|
/**
|
||||||
|
*
|
||||||
|
* @override
|
||||||
|
*/
|
||||||
|
_addToCartInPage: function () {
|
||||||
|
return this._super.apply(this, arguments).catch((error) => {
|
||||||
|
const message = this._getRestrictedQtyMessage(error);
|
||||||
|
if (!message) {
|
||||||
|
// Not a quantity restriction: let the real error surface.
|
||||||
|
return Promise.reject(error);
|
||||||
|
}
|
||||||
|
if (error && error.event && error.event.preventDefault) {
|
||||||
|
// Legacy convention: tells the crash manager the error is
|
||||||
|
// handled, so it does not open its own dialog on top.
|
||||||
|
error.event.preventDefault();
|
||||||
|
}
|
||||||
|
wSaleUtils.showWarning(message);
|
||||||
|
});
|
||||||
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @private
|
||||||
|
* @param {Object} error rejection from the legacy RPC
|
||||||
|
* @returns {string|false} the message, or false to re-throw
|
||||||
|
*/
|
||||||
|
_getRestrictedQtyMessage: function (error) {
|
||||||
|
const data =
|
||||||
|
(error && error.message && error.message.data) ||
|
||||||
|
(error && error.data);
|
||||||
|
if (!data || data.name !== "odoo.exceptions.ValidationError") {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
return data.message;
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
});
|
||||||
1
sale_restricted_qty_website_sale/tests/__init__.py
Normal file
1
sale_restricted_qty_website_sale/tests/__init__.py
Normal file
@@ -0,0 +1 @@
|
|||||||
|
from . import test_restricted_qty_website_sale
|
||||||
@@ -0,0 +1,89 @@
|
|||||||
|
from lxml import html
|
||||||
|
|
||||||
|
from odoo.exceptions import ValidationError
|
||||||
|
from odoo.tests import HttpCase, tagged
|
||||||
|
|
||||||
|
from odoo.addons.http_routing.models.ir_http import slug
|
||||||
|
|
||||||
|
|
||||||
|
@tagged("post_install", "-at_install")
|
||||||
|
class TestRestrictedQtyInfo(HttpCase):
|
||||||
|
@classmethod
|
||||||
|
def setUpClass(cls):
|
||||||
|
super().setUpClass()
|
||||||
|
cls.partner = cls.env["res.partner"].create({"name": "eCommerce customer"})
|
||||||
|
cls.product_min = cls._create_product(
|
||||||
|
"Product min 6", manual_sale_min_qty=6.0
|
||||||
|
)
|
||||||
|
cls.product_free = cls._create_product("Unrestricted product")
|
||||||
|
# force = Yes -> purely indicative bound, nothing to tell the customer
|
||||||
|
cls.product_indicative = cls._create_product(
|
||||||
|
"Product min 6 indicative",
|
||||||
|
manual_sale_min_qty=6.0,
|
||||||
|
manual_force_sale_min_qty="force",
|
||||||
|
)
|
||||||
|
|
||||||
|
@classmethod
|
||||||
|
def _create_product(cls, name, **restrictions):
|
||||||
|
values = {
|
||||||
|
"name": name,
|
||||||
|
"type": "consu",
|
||||||
|
"list_price": 10.0,
|
||||||
|
"sale_ok": True,
|
||||||
|
"is_published": True,
|
||||||
|
}
|
||||||
|
values.update(restrictions)
|
||||||
|
return cls.env["product.template"].create(values).product_variant_id
|
||||||
|
|
||||||
|
# ------------------------------------------------------------------
|
||||||
|
# Error message
|
||||||
|
# ------------------------------------------------------------------
|
||||||
|
|
||||||
|
def _create_line(self, product, qty):
|
||||||
|
order = self.env["sale.order"].create({"partner_id": self.partner.id})
|
||||||
|
self.env["sale.order.line"].create(
|
||||||
|
{
|
||||||
|
"order_id": order.id,
|
||||||
|
"product_id": product.id,
|
||||||
|
"product_uom_qty": qty,
|
||||||
|
}
|
||||||
|
)
|
||||||
|
# @api.constrains are evaluated on flush, not on create().
|
||||||
|
self.env.flush_all()
|
||||||
|
return order
|
||||||
|
|
||||||
|
def test_error_message_is_customer_facing(self):
|
||||||
|
with self.assertRaises(ValidationError) as error:
|
||||||
|
self._create_line(self.product_min, 1)
|
||||||
|
|
||||||
|
message = str(error.exception)
|
||||||
|
self.assertIn("Product min 6", message)
|
||||||
|
self.assertIn("6", message)
|
||||||
|
self.assertNotIn("force min", message, "Upstream jargon still present")
|
||||||
|
|
||||||
|
def test_valid_quantity_raises_nothing(self):
|
||||||
|
self._create_line(self.product_min, 6)
|
||||||
|
|
||||||
|
def test_indicative_minimum_still_does_not_block(self):
|
||||||
|
self._create_line(self.product_indicative, 1)
|
||||||
|
|
||||||
|
# ------------------------------------------------------------------
|
||||||
|
# Product page display
|
||||||
|
# ------------------------------------------------------------------
|
||||||
|
|
||||||
|
def _hint_on_page(self, product):
|
||||||
|
response = self.url_open("/shop/%s" % slug(product.product_tmpl_id))
|
||||||
|
self.assertEqual(response.status_code, 200)
|
||||||
|
tree = html.fromstring(response.content)
|
||||||
|
return tree.xpath("//div[contains(@class, 'o_wsale_qty_restriction')]")
|
||||||
|
|
||||||
|
def test_hint_is_displayed(self):
|
||||||
|
hint = self._hint_on_page(self.product_min)
|
||||||
|
self.assertEqual(len(hint), 1)
|
||||||
|
self.assertIn("6", hint[0].text_content())
|
||||||
|
|
||||||
|
def test_no_hint_without_restriction(self):
|
||||||
|
self.assertFalse(self._hint_on_page(self.product_free))
|
||||||
|
|
||||||
|
def test_no_hint_for_an_indicative_minimum(self):
|
||||||
|
self.assertFalse(self._hint_on_page(self.product_indicative))
|
||||||
@@ -0,0 +1,19 @@
|
|||||||
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
|
<odoo>
|
||||||
|
|
||||||
|
<!-- Product page: tell the customer about the orderable quantities,
|
||||||
|
right below the quantity selector and the add-to-cart button. -->
|
||||||
|
<template id="product_restricted_qty_info"
|
||||||
|
inherit_id="website_sale.product_quantity"
|
||||||
|
name="Restricted Quantity Information">
|
||||||
|
<xpath expr="//div[@id='add_to_cart_wrap']" position="after">
|
||||||
|
<t t-set="qty_hint" t-value="product._get_restricted_qty_hint()"/>
|
||||||
|
<div t-if="qty_hint"
|
||||||
|
class="o_wsale_qty_restriction alert alert-info d-flex align-items-center w-100 mt-2 mb-0 py-2 px-3">
|
||||||
|
<i class="fa fa-info-circle me-2" role="img" aria-label="Information"/>
|
||||||
|
<span t-out="qty_hint"/>
|
||||||
|
</div>
|
||||||
|
</xpath>
|
||||||
|
</template>
|
||||||
|
|
||||||
|
</odoo>
|
||||||
@@ -3,7 +3,7 @@
|
|||||||
|
|
||||||
{
|
{
|
||||||
"name": "sale_usability_misc",
|
"name": "sale_usability_misc",
|
||||||
"version": "16.0.1.0.0",
|
"version": "16.0.1.0.1",
|
||||||
"author": "Elabore",
|
"author": "Elabore",
|
||||||
"website": "https://elabore.coop",
|
"website": "https://elabore.coop",
|
||||||
"maintainer": "Elabore",
|
"maintainer": "Elabore",
|
||||||
@@ -12,7 +12,7 @@
|
|||||||
"summary": "Various improves for Sales app",
|
"summary": "Various improves for Sales app",
|
||||||
# any module necessary for this one to work correctly
|
# any module necessary for this one to work correctly
|
||||||
"depends": [
|
"depends": [
|
||||||
"base","sale","sales_team",
|
"base","sale","sales_team","sale_crm",
|
||||||
],
|
],
|
||||||
"qweb": [],
|
"qweb": [],
|
||||||
"external_dependencies": {
|
"external_dependencies": {
|
||||||
|
|||||||
@@ -1,4 +1,4 @@
|
|||||||
<?xml version="1.0" encoding="utf-8" ?>
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
<odoo>
|
<odoo>
|
||||||
|
|
||||||
<record id="view_order_form_inherit" model="ir.ui.view">
|
<record id="view_order_form_inherit" model="ir.ui.view">
|
||||||
@@ -6,10 +6,25 @@
|
|||||||
<field name="model">sale.order</field>
|
<field name="model">sale.order</field>
|
||||||
<field name="inherit_id" ref="sale.view_order_form" />
|
<field name="inherit_id" ref="sale.view_order_form" />
|
||||||
<field name="arch" type="xml">
|
<field name="arch" type="xml">
|
||||||
<xpath expr="//group[@name='order_details']/div[@class='o_td_label' and @groups='base.group_no_one']" position="attributes">
|
<xpath
|
||||||
|
expr="//group[@name='order_details']/div[@class='o_td_label' and @groups='base.group_no_one']"
|
||||||
|
position="attributes">
|
||||||
<attribute name="groups">sales_team.group_sale_manager</attribute>
|
<attribute name="groups">sales_team.group_sale_manager</attribute>
|
||||||
</xpath>
|
</xpath>
|
||||||
<xpath expr="//group[@name='order_details']/field[@name='date_order' and @groups='base.group_no_one']" position="attributes">
|
<xpath
|
||||||
|
expr="//group[@name='order_details']/field[@name='date_order' and @groups='base.group_no_one']"
|
||||||
|
position="attributes">
|
||||||
|
<attribute name="groups">sales_team.group_sale_manager</attribute>
|
||||||
|
</xpath>
|
||||||
|
</field>
|
||||||
|
</record>
|
||||||
|
|
||||||
|
<record id="sale_opportunity_not_dev_inherit" model="ir.ui.view">
|
||||||
|
<field name="name">sale.opportunity.not.dev.inherit</field>
|
||||||
|
<field name="model">sale.order</field>
|
||||||
|
<field name="inherit_id" ref="sale_crm.sale_view_inherit123" />
|
||||||
|
<field name="arch" type="xml">
|
||||||
|
<xpath expr="//field[@name='opportunity_id']" position="attributes">
|
||||||
<attribute name="groups">sales_team.group_sale_manager</attribute>
|
<attribute name="groups">sales_team.group_sale_manager</attribute>
|
||||||
</xpath>
|
</xpath>
|
||||||
</field>
|
</field>
|
||||||
|
|||||||
Reference in New Issue
Block a user