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]: Fix the traceback when click on the location and warehouse fil… #31316

Closed

Conversation

@C3POdoo C3POdoo added the RD research & development, internal work label Feb 21, 2019
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Feb 21, 2019
@ppr-odoo ppr-odoo force-pushed the saas-12.1-filter-traceback-fix-ppr branch from 15e1bd7 to b7cc33c Compare February 21, 2019 10:58
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Feb 21, 2019
@ppr-odoo ppr-odoo force-pushed the saas-12.1-filter-traceback-fix-ppr branch from b7cc33c to 2d45534 Compare February 21, 2019 12:11
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Feb 21, 2019
@ppr-odoo ppr-odoo force-pushed the saas-12.1-filter-traceback-fix-ppr branch from 2d45534 to ed9d644 Compare February 22, 2019 12:54
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Feb 22, 2019
@mgeubelle
Copy link
Contributor

Linked to #30368 where self in context="{'field_id': self}" is now a list of value (either integer for id, either string for search terms).

@Whenrow Whenrow force-pushed the saas-12.1-filter-traceback-fix-ppr branch 3 times, most recently from 91fc9ae to e8e478c Compare May 6, 2019 14:27
domain += [('company_id', '=', self.env.context['force_company'])]
location_ids = self.env['stock.location'].search(domain).ids

def _search_ids(model, value, force_company_id):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

values

domain += [('company_id', '=', self.env.context['force_company'])]
location_ids = self.env['stock.location'].search(domain).ids

def _search_ids(model, value, force_company_id):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add a comment: dummy field uset in the search view will always return a list of ids or a list of string

@Whenrow Whenrow force-pushed the saas-12.1-filter-traceback-fix-ppr branch 4 times, most recently from a7da4ca to 7e39727 Compare May 8, 2019 11:47
@sle-odoo
Copy link
Contributor

robodoo r+

@robodoo
Copy link
Contributor

robodoo commented May 10, 2019

'ci/runbot' failed on this reviewed PR.

@robodoo
Copy link
Contributor

robodoo commented May 10, 2019

'ci/runbot' failed on this reviewed PR.

1 similar comment
@robodoo
Copy link
Contributor

robodoo commented May 10, 2019

'ci/runbot' failed on this reviewed PR.

Purpose: filter product by location of warehouse failed due to the
control panel refactoring (odoo#29106). This commit fix the way the
product quantities are computed when location/warehouse into filter
context.

Task:1941639

Co-Authored-By: Martin Geubelle <mge@odoo.com>
@Whenrow Whenrow force-pushed the saas-12.1-filter-traceback-fix-ppr branch from 7e39727 to 5a2bcf8 Compare May 10, 2019 08:07
@robodoo robodoo removed the r+ 👌 label May 10, 2019
@sle-odoo
Copy link
Contributor

robodoo r+

robodoo pushed a commit that referenced this pull request May 10, 2019
Purpose: filter product by location of warehouse failed due to the
control panel refactoring (#29106). This commit fix the way the
product quantities are computed when location/warehouse into filter
context.

Task:1941639

Co-Authored-By: Martin Geubelle <mge@odoo.com>

closes #31316

Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses merging 👷 and removed r+ 👌 CI 🤖 Robodoo has seen passing statuses labels May 10, 2019
@robodoo
Copy link
Contributor

robodoo commented May 10, 2019

Staging failed: ci/runbot on 4fd16198707025667427575458e7379912fa7016 (view more at http://runbot.odoo.com/runbot/build/516586)

@KangOl
Copy link
Contributor

KangOl commented May 10, 2019

@robodoo retry

@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses r+ 👌 and removed error 🙅 labels May 10, 2019
@robodoo
Copy link
Contributor

robodoo commented May 10, 2019

Merged, thanks!

@robodoo robodoo closed this May 10, 2019
@KangOl KangOl deleted the saas-12.1-filter-traceback-fix-ppr branch May 10, 2019 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants