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

Skip View validation for redefined and controlled fields_get #153

Merged
merged 2 commits into from
Jul 5, 2024

Conversation

polsala
Copy link
Member

@polsala polsala commented Jul 3, 2024

2024-07-03 15:31:22,027:INFO:destral.testing.results:ERROR: test_all_views (destral.testing.OOBaseTests)
Tests all views defined in the module.
2024-07-03 15:31:22,027:INFO:destral.testing.results:
2024-07-03 15:31:22,027:INFO:destral.testing.results:----------------------------------------------------------------------
2024-07-03 15:31:22,027:INFO:destral.testing.results:
2024-07-03 15:31:22,027:INFO:destral.testing.results:Traceback (most recent call last):
  File "/home/psala/Projects/destral/destral/testing.py", line 159, in test_all_views
    model.fields_view_get(txn.cursor, txn.user, view.id, view.type, version=view.version)
  File "/home/psala/Projects/erp/server/bin/addons/custom_search/custom_search.py", line 418, in fields_view_get
    view_id, view_type, context, toolbar, version)
  File "/home/psala/Projects/erp/server/bin/tools/misc.py", line 985, in wrapper
    result = func(*args, **kwargs)
  File "/home/psala/Projects/erp/server/bin/osv/orm.py", line 1922, in fields_view_get
    xarch, xfields = self.__view_look_dom_arch(cr, user, result['arch'], view_id, context=context)
  File "/home/psala/Projects/erp/server/bin/osv/orm.py", line 1626, in __view_look_dom_arch
    fields = self.fields_get(cr, user, fields_def.keys(), context)
  File "/home/psala/Projects/erp/server/bin/addons/custom_search/custom_search.py", line 382, in fields_get
    raise osv.except_osv(_('Error'), _('Search id not found'))
except_osv: warning -- Error

Search id not found

@polsala polsala requested a review from ecarreras July 3, 2024 13:35
@polsala polsala self-assigned this Jul 3, 2024
@polsala polsala added enhancement to be merged minor Minor auto version V_.X._ labels Jul 3, 2024
@polsala polsala enabled auto-merge July 3, 2024 13:38
@polsala polsala merged commit 13ed4ff into master Jul 5, 2024
@polsala polsala deleted the avoid_test_ceritain_views branch July 5, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement minor Minor auto version V_.X._ to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants