From 7bcf38805cfa5a8841b2bec10435db766a60f9ff Mon Sep 17 00:00:00 2001 From: Simone Orsi Date: Wed, 31 Oct 2018 10:55:51 +0100 Subject: [PATCH] web_advanced_search: fix 1st domain node load --- web_advanced_search/README.rst | 45 +++++++++++ web_advanced_search/__manifest__.py | 2 +- web_advanced_search/readme/CONTRIBUTORS.rst | 1 + web_advanced_search/readme/HISTORY.rst | 40 ++++++++++ .../static/description/index.html | 76 +++++++++++++++---- .../static/src/js/web_advanced_search.js | 6 +- 6 files changed, 153 insertions(+), 17 deletions(-) create mode 100644 web_advanced_search/readme/HISTORY.rst diff --git a/web_advanced_search/README.rst b/web_advanced_search/README.rst index f9660708..5c7cdf09 100644 --- a/web_advanced_search/README.rst +++ b/web_advanced_search/README.rst @@ -75,6 +75,50 @@ Improvements to the search view in this addon: * Beautiful, human-readable, domain representation when adding an advanced filter +Changelog +========= + +11.0.1.0.2 (2018-10-31) +~~~~~~~~~~~~~~~~~~~~~~~ + +* Fix initialization of 1st domain node + + Sometime the dialog is not ready yet, like on EE version. + Hence when you inject the 1st domain node + the dialog must be already opened. + + [simahawk] + + +11.0.1.0.1 (2018-09-18) +~~~~~~~~~~~~~~~~~~~~~~~ + +* Fix `undefined` in x2m fields + + Before this patch, when searching with the "equals to" operator in any + x2many field, the searched parameter was always `undefined`. + + The problem was that the underlying field manager implementation was + treating those fields as x2many, while the widget used was the `one2many` + one. + + This patch simply mocks the underlying fake record to make think that + any relational field is always a `one2many`. This sets all pieces in + place and makes the field manager work as expected, and thus you can + search as expected too. + +* Make linter happy + + [Yajo] + + +11.0.1.0.0 (2018-07-20) +~~~~~~~~~~~~~~~~~~~~~~~ + +* Rename, refactor, migrate to v11 + + [Yajo] + Bug Tracker =========== @@ -102,6 +146,7 @@ Contributors * Jairo Llopis * Rami Alwafaie * Jose Mª Bernet +* Simone Orsi Maintainers ~~~~~~~~~~~ diff --git a/web_advanced_search/__manifest__.py b/web_advanced_search/__manifest__.py index 32cf5a28..b17b84a2 100644 --- a/web_advanced_search/__manifest__.py +++ b/web_advanced_search/__manifest__.py @@ -5,7 +5,7 @@ { "name": "Advanced search", - "version": "11.0.1.0.0", + "version": "11.0.1.0.2", "author": "Therp BV, " "Tecnativa, " "Odoo Community Association (OCA)", diff --git a/web_advanced_search/readme/CONTRIBUTORS.rst b/web_advanced_search/readme/CONTRIBUTORS.rst index 4920464c..ed3dc10c 100644 --- a/web_advanced_search/readme/CONTRIBUTORS.rst +++ b/web_advanced_search/readme/CONTRIBUTORS.rst @@ -3,3 +3,4 @@ * Jairo Llopis * Rami Alwafaie * Jose Mª Bernet +* Simone Orsi diff --git a/web_advanced_search/readme/HISTORY.rst b/web_advanced_search/readme/HISTORY.rst new file mode 100644 index 00000000..7b791ed9 --- /dev/null +++ b/web_advanced_search/readme/HISTORY.rst @@ -0,0 +1,40 @@ +11.0.1.0.2 (2018-10-31) +~~~~~~~~~~~~~~~~~~~~~~~ + +* Fix initialization of 1st domain node + + Sometime the dialog is not ready yet, like on EE version. + Hence when you inject the 1st domain node + the dialog must be already opened. + + [simahawk] + + +11.0.1.0.1 (2018-09-18) +~~~~~~~~~~~~~~~~~~~~~~~ + +* Fix `undefined` in x2m fields + + Before this patch, when searching with the "equals to" operator in any + x2many field, the searched parameter was always `undefined`. + + The problem was that the underlying field manager implementation was + treating those fields as x2many, while the widget used was the `one2many` + one. + + This patch simply mocks the underlying fake record to make think that + any relational field is always a `one2many`. This sets all pieces in + place and makes the field manager work as expected, and thus you can + search as expected too. + +* Make linter happy + + [Yajo] + + +11.0.1.0.0 (2018-07-20) +~~~~~~~~~~~~~~~~~~~~~~~ + +* Rename, refactor, migrate to v11 + + [Yajo] diff --git a/web_advanced_search/static/description/index.html b/web_advanced_search/static/description/index.html index 53cba64b..847a54b8 100644 --- a/web_advanced_search/static/description/index.html +++ b/web_advanced_search/static/description/index.html @@ -372,19 +372,25 @@ ul.auto-toc {

Table of contents

-

Usage

+

Usage

To use this module, you need to:

  • Open Filters in a search view
    • @@ -408,7 +414,7 @@ this indefinetely, so it is possible to search for moves belonging to a journal which has a user who is member of a certain group etc.

-

Known issues / Roadmap

+

Known issues / Roadmap

Improvements to the domain widget, not exclusively related to this addon:

  • Use relational widgets when filtering a relational field
    • @@ -425,8 +431,49 @@ advanced domain editor advanced filter
+
+

Changelog

+
+

11.0.1.0.2 (2018-10-31)

+
    +

  • Fix initialization of 1st domain node

    +

    Sometime the dialog is not ready yet, like on EE version. +Hence when you inject the 1st domain node +the dialog must be already opened.

    +

    [simahawk]

    +
    • +
+
+
+

11.0.1.0.1 (2018-09-18)

+
    +

  • Fix undefined in x2m fields

    +

    Before this patch, when searching with the “equals to” operator in any +x2many field, the searched parameter was always undefined.

    +

    The problem was that the underlying field manager implementation was +treating those fields as x2many, while the widget used was the one2many +one.

    +

    This patch simply mocks the underlying fake record to make think that +any relational field is always a one2many. This sets all pieces in +place and makes the field manager work as expected, and thus you can +search as expected too.

    +
    • +

  • Make linter happy

    +

    [Yajo]

    +
    • +
+
+
+

11.0.1.0.0 (2018-07-20)

+
    +

  • Rename, refactor, migrate to v11

    +

    [Yajo]

    +
    • +
+
+
-

Bug Tracker

+

Bug Tracker

Bugs are tracked on GitHub Issues. In case of trouble, please check there if your issue has already been reported. If you spotted it first, help us smashing it by providing a detailed and welcomed @@ -434,26 +481,27 @@ If you spotted it first, help us smashing it by providing a detailed and welcome

Do not contact contributors directly about support or help with technical issues.

-

Credits

+

Credits

-

Authors

+

Authors

  • Therp BV
  • Tecnativa
-

Maintainers

+

Maintainers

This module is maintained by the OCA.

Odoo Community Association

OCA, or the Odoo Community Association, is a nonprofit organization whose diff --git a/web_advanced_search/static/src/js/web_advanced_search.js b/web_advanced_search/static/src/js/web_advanced_search.js index b2e12c75..963986a4 100644 --- a/web_advanced_search/static/src/js/web_advanced_search.js +++ b/web_advanced_search/static/src/js/web_advanced_search.js @@ -106,8 +106,10 @@ odoo.define("web_advanced_search", function (require) { readonly: false, } ); - // Add 1st domain node by default - domain_selector_dialog.domainSelector._onAddFirstButtonClick(); + domain_selector_dialog.opened(function () { + // Add 1st domain node by default + domain_selector_dialog.domainSelector._onAddFirstButtonClick(); + }); return domain_selector_dialog.open(); },