Skip to content

Commit ea250d0

Browse files
committed
[IMP] util/records: Fix edit_view docstring
`edit_view` yields an xmlid view according to the following logic: If the view has `noupdate=False`, we assume that it is a standard view that'll be updated by the ORM later on, so by default we don't yield it when calling `edit_view`. We may force yield such a view when setting 'skip_if_not_noupdate` to False. This behaviour was initially described correctly in the docstring, but the values of `noupdate` got mixed up in the new docstring in aaa1f0f . This commit reverts the values in the docstring to the correct ones. closes #119 Signed-off-by: Christophe Simonis (chs) <[email protected]>
1 parent c2cd2b7 commit ea250d0

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/util/records.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,8 +185,8 @@ def edit_view(cr, xmlid=None, view_id=None, skip_if_not_noupdate=True, active="a
185185
186186
When the view is identified by `view_id`, the arch is always yielded if the view
187187
exists, with disregard to any `noupdate` flag it may have associated. When `xmlid` is
188-
set, if the view `noupdate` flag is `True` then the arch will not be yielded *unless*
189-
`skip_if_not_noupdate` is set to `False`. If `noupdate` is `False`, the view will be
188+
set, if the view `noupdate` flag is `False` then the arch will not be yielded *unless*
189+
`skip_if_not_noupdate` is set to `False`. If `noupdate` is `True`, the view will be
190190
yielded for edit.
191191
192192
If the `active` argument is `True` or `False`, the `active` flag of the view will be set

0 commit comments

Comments
 (0)