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

[FIX] stock_account: fix traceback when try to install stock_account module #21107

Conversation

nla-odoo
Copy link

@nla-odoo nla-odoo commented Nov 21, 2017

'without_demo' param of tools.config always set value False when user make DB from UI even if the user has made DB without demo data.

Issue: https://www.odoo.com/web#id=783157&view_type=form&model=project.issue&action=963&active_id=681&menu_id=4720

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

@nla-odoo nla-odoo changed the title [FIX] stock_account: fix traceback when try to install stock_account … [FIX] stock_account: fix traceback when try to install stock_account module Nov 21, 2017
@C3POdoo C3POdoo added the RD research & development, internal work label Nov 21, 2017
@tde-banana-odoo
Copy link
Contributor

Can't we use tools.config['demo'] instead ?

…module.

		-'without_demo' param of tools.config always set value False when user make db from UI even if user have made db without demo data.
@mba-odoo mba-odoo force-pushed the master-stock_account-fix-traceback-issue-nla branch from 5999c18 to 1a4ae36 Compare December 28, 2017 10:46
@mba-odoo
Copy link
Contributor

@tde-banana-odoo , No, we will not get tools.config['demo'] or tools.config['without_demo'] while creating Db with UI. Thanks

@tde-banana-odoo
Copy link
Contributor

LGTM then.

@tde-banana-odoo
Copy link
Contributor

Isn't it applicable to saas 11.1 ? I see original commit is before the freeze.

@mba-odoo
Copy link
Contributor

@tde-banana-odoo , It's inside saas-11.1 https://github.com/odoo/odoo/blob/saas-11.1/addons/stock_account/__init__.py So can we fix in that? Thanks

@tde-banana-odoo
Copy link
Contributor

Hello,

Yes you can make a PR targetting 11.1 Thanks !

@nla-odoo
Copy link
Author

@tde-banana-odoo , Here is the PR for 11.1 - #21899
Thanks.

@tde-banana-odoo
Copy link
Contributor

Done in saas 11.1 at c268aa5 . Forward port will solve the issue in master. Thanks !

@tde-banana-odoo tde-banana-odoo deleted the master-stock_account-fix-traceback-issue-nla branch December 29, 2017 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants