| Age | Commit message (Collapse) | Author |
|
closes odoo/odoo#58322
closes odoo/odoo#58434
X-original-commit: 7ceb73d324cd90038c80a5c7d65d9f43fd4b264a
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
|
|
Write guidelines on how to write secure code and avoid commons
mistakes.
Update outdated information and move existing security guidelines to
the same section.
closes odoo/odoo#58060
X-original-commit: 10184274fb6be1639d2047e863654ed5c7bef9f0
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
|
|
closes odoo/odoo#57451
X-original-commit: 3b4a0ab81a667c75ce11b8eec0cbf2469bc76f61
Signed-off-by: Richard Mathot (rim) <rim@openerp.com>
|
|
You can see on example below the updated line that it's module_name and not model_name
<!-- security -->
<record id="module_name_group_user" model="res.groups">
...
</record>
closes odoo/odoo#54756
X-original-commit: 56de03befa16eeaaec211b5fc08116ce778b52b7
Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
|
|
Mimic the syntax of logging to use placeholders in translatable
message.
The main advantage is to be able to fallback on the source term if the
translation can not be formatted properly.
It is very common to have errors in translations with missing
placeholders or badly translated (ie. the source
"%(subject)s" -> "%(sujet)s").
Instead of blocking the execution of the code, fallback on source
string.
Task-id: 1853119
|
|
X-original-commit: c83e999c6a8b16f78dfd9f944936439a53d2b52f
|
|
Unclosed tag
closes odoo/odoo#37744
X-original-commit: a0612f5a17e08e13362034203995944756a7c0be
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
closes odoo/odoo#35404
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
|
|
Those two decorators are removed/deprecated since recent commits but some references remained in the documentation.
api.multi is deprecated since https://github.com/odoo/odoo/commit/cc62447a44c2cf4e2edaf8a1f6d94520e799ddd9
api.one was removed with https://github.com/odoo/odoo/commit/407f1f6cdb87a59bbad9f853078d7345cdf9f60c
closes odoo/odoo#35261
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
|
|
Multi is the default api for methods, it is not necessary to explicitly
decorate methods with it, adds clutter and most people use it because
they see that the rest of the code uses it.
Done with `find . -type f -name '*.py' | xargs sed -i '/@api.multi/d'`
|
|
Adapt doc to d7cd97a9b
closes odoo/odoo#33921
Signed-off-by: Jérome Maes (jem) <jem@openerp.com>
|
|
Use task-#xxx is wrong since it will refer to an unexising or wrong hash
of github.
closes odoo/odoo#33121
Signed-off-by: Jérémy Kersten (jke) <jke@openerp.com>
|
|
Task-1817988
closes odoo/odoo#28714
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
|
|
* rewording of XML naming section
* remove the convention of suffixing inherited XML ids
with `_inherit_my_module, as the information is in the
complete XML id name (prefixed by module name). This will
avoid too long XML ids.
* extend convention for the name of groups, views, ...
Task-1817988
|
|
Rewording of "action" naming section.
Task-1817988
|
|
This commit
* removes deprecated guidelines for static file and assets struture;
* fixes some typo and mistakes;
* correctly links JS guidelines section;
* adds a running example to ease understanding;
* adds missing section about security files;
* adds text blocks for each directory to show directly the
expected result of each section;
* perform some rewording;
Thanks to @pedrobaeza for its review.
Task-1817988
|
|
Fix rst-formating for numbered list under heading "Inherited XML".
closes odoo/odoo#30043
|
|
If the code is well written from the start, won't get so many hook requests
closes odoo/odoo#27570
|
|
The documentation should be an example of good practices
closes odoo/odoo#27262
|
|
Commit cf853a785ba4b533ddb2e8a717c9ffa6429e7b81 removed all yml tests
and the yml import engine from Odoo forever, however PyYAML remains a
dependency even though it's not used anymore.
This commit removes any reference to this lib that could be found.
closes odoo/odoo#27563
|
|
Sometime, we need to get selection values from
a method. Here are the guidelines about how
to correctly name method and place it in the
code.
|
|
Odoo evolves and its coding guidelines too.
This commit :
- put some news guidelines, case to take
into account when making new dev
- update some guidelines that are difficult
to applied in the real life
- remove deprecated stuff (RML report, ...)
- remove references to old
api mainly, since all new dev must be done in
new api
Task #1817988
|
|
The same content is already there 2 lines above.
|
|
|
|
The state of the JS documentation was quite sad, not updated in a long
time. With this commit, we rework a lot of the JS doc. Most notably,
we add a section 'Javascript Cheatsheet' (help on some common
customization tasks) and rewrite the 'Javascript Reference' totally.
Note that the JS reference page also contains a reference on all field
widgets.
This work is just the beginning, it is probably not complete, but at
least we will have a foundation.
|
|
Github will automatically close the report is you use the format
Closes #number
No need to a leading #
cf https://help.github.com/articles/closing-issues-using-keywords/
Add the meaning of [I18N] tag
This commit is an example of a violation of the "one commit, one fix" principle
|
|
Purpose of this commit is to clean a bit the wording and to clarify how to
handle guidelines between stable and development versions.
|
|
Purpose of this commit is to explain with more details how to write commit
messages. Content of the Git section is also updated with last decisions
and conventions.
|
|
There is only one API, long live Odoo's API!
Closes #20928
|
|
|
|
|
|
|
|
Some bits posterior or leftover from fffaf735f58f1ac57781c095eb552de6d1fcdc47
See also parent commit.
|
|
Because now is no longer required
Closes #18144
|
|
Typo
Closes #14652
|
|
|
|
|
|
- As of v10, manifest files should be named `__manifest__.py`
- For backwards-compatibility, __openerp__.py manifest files
will still be supported for the time being
- Limited refactoring, to add support for the 2 different
naming conventions
- All textual references to __openerp_.py updated in
documentation and examples
|
|
- add python idioms
- odoo specific guidelines (for ORM, translations, ...) inspired from old guidelines v6.0
- some other best pratices
|
|
- correct typos
- clarify some points, according to the community comments
- precise the xml notation : record tag vs act_window/menuitem/... tags
- add organization of report, wizard, data/demo, ... (directories and naming conventions)
- new conventions for 'symbols'
- ...
Thanks to @rim-odoo for its grammar extremist review, and @tde-banana-odoo for our long debates about wizard conventions.
|
|
To prevent huge chunk of modifications for styling reasons (e.g. mass PEP8
commits)
|
|
According to PEP-263, the -*- is just a variant "using formats recognized by
popular editors".
|
|
Can still be mentionned in documentation but should not be used in example.
|
|
Some rephrasing to slightly improve the docs.
Closes #11652
|
|
|
|
Backport of most of #6827 by J Bradshaw with additional
modifications (and some reverts) during review.
closes #6827
|
|
Pretty much completely rewritten theme with custom HTML translator and a
few parts of the old theme extracted to their own extensions.
Banner images thought not to be that huge after all, and not worth the
hassle of them living in a different repository.
co-authored with @stefanorigano
|
|
the guidelines. Discussions and conventions are not frozen.
|