Skip to content
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

Lint - Uses of a deprecated module 'openerp.osv' warning when using openerp.osv.expression #306

Closed
yvaucher opened this issue Jan 14, 2016 · 9 comments

Comments

Projects
None yet
7 participants
@yvaucher

This comment has been minimized.

Copy link
Member Author

commented Jan 14, 2016

@guewen

This comment has been minimized.

Copy link
Member

commented Jan 14, 2016

👍 for the removal

@gurneyalex

This comment has been minimized.

Copy link
Member

commented Jan 14, 2016

note that you can also use from openerp.models import expression (but I'm really not a fan of transitive imports...)

@Yajo

This comment has been minimized.

Copy link
Member

commented Jan 14, 2016

👎 You just have to change for what @gurneyalex said and pass the lint.

@moylop260

This comment has been minimized.

Copy link
Contributor

commented Jan 14, 2016

Please, let us know if expresion from models works

@dreispt

This comment has been minimized.

Copy link
Member

commented Jan 16, 2016

AFAIK osv is supposed to be deprecated.
If there is a viable alternative, I don't why we should use it.

@StefanRijnhart

This comment has been minimized.

Copy link
Member

commented Jun 24, 2016

Can't the test be made more sophisticated so as to exclude expression?

@moylop260

This comment has been minimized.

Copy link
Contributor

commented Jun 24, 2016

@StefanRijnhart
Can't the test be made more sophisticated so as to exclude expression?

let me try it with augmentations

@moylop260

This comment has been minimized.

Copy link
Contributor

commented Jun 24, 2016

FYI the fix OCA/pylint-odoo#40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.