Odoo Changelog — 2019-06-08

Saturday, June 8, 2019

3 changes

Improvements(3)

Enhancements to existing features

[IMP] core: log traceback in one group

Coremastercore

Test error tracebacks are now logged as a single grouped entry instead of many separate lines. This makes runbot logs easier to read, search, and copy when diagnosing failed tests, while removing less useful test docstring text from short descriptions.

Original PR description

Traceback coming from tests are currently logged
line by line. This will implies that runbot will have one
ir_logging entry per line which is not practical. More than
that, the log prefix can make the traceback less readable
because of line returns and difficult to copy paste.

This commit simply remove this feature. After discussion with odo
and chs, we will also remove the docstring from test shortDescription
since most of the time this information is not clear and can be accessed
in source code if needed.

#testing#logging#runbot

[IMP] odoo: ignore unsupported args from controller

PR #33962

Coremastertest_websiteodoo

Website and web controller pages now ignore unexpected extra URL parameters instead of failing. This makes public pages more resilient when visitors arrive through marketing links, debug links, or other URLs that add extra tracking details.

Original PR description

Before this commit, call a controller defined as: 

```python
@http.route('/route', type='http', auth='public')
def controller_func(self, foo):
    do_it()
```

and called with url like /route?foo=1&bar=2

will crash with an exception:
`TypeError: controller_func() got an unexpected keyword argument 'bar'`

Now, we remove the extra parameters if the controller doesn't support it.
This case is not uncommon, you can easily arrive in this case with utm or
debug as extra parameter.



--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

#website#controllers#url-parameters#stability

[IMP] doc: adapt doc to new test asset and JS test file structure

PR #33921

Coremasterdoc

The documentation was updated to match the current structure for test assets and JavaScript test files. This helps development teams follow the latest project organization and reduces confusion when maintaining or adding tests.

Original PR description

Adapt doc to d7cd97a9b

#documentation#testing#javascript

Traceback coming from tests are currently logged line by line. This will implies that runbot will have one ir_logging entry per line which is not practical. More than that, the log prefix can make the traceback less readable because of line returns and difficult to copy paste. This commit simply remove this feature. After discussion with odo and chs, we will also remove the docstring from test shortDescription since most of the time this information is not clear and can be accessed in source code if needed.

Before this commit, call a controller defined as: ```python @http.route('/route', type='http', auth='public') def controller_func(self, foo): do_it() ``` and called with url like /route?foo=1&bar=2 will crash with an exception: `TypeError: controller_func() got an unexpected keyword argument 'bar'` Now, we remove the extra parameters if the controller doesn't support it. This case is not uncommon, you can easily arrive in this case with utm or debug as extra parameter. -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr