Review: Needs Information > this.searchview[view.controller.searchable === false || > this.searchview.hidden ? 'hide' : 'show']();
Cette ligne (214 dans le diff preview de launchpad) me semble bizarre: si la vue est pas searchable on va appeler `this.searchview[true]()`… * Dans `do_searchview_search` il y a un commentaire qui traine, et juste histoire d'être certain je pense que `last_search` devrait être set avant de dispatcher, sinon on va perdre la recherche si la vue a un bug (ou alors il faut setTimeout/_.defer le dispatch vers la vue) * On schedule le fait de modifier domaine et contexte sur le dataset (histoire que les vues puissent faire un read_slice direct sans se poser de question) après? Je pense que ce serait quand même une bonne idée, en plus de les filer à la fonction de recherche. -- https://code.launchpad.net/~openerp-dev/openerp-web/trunk-search2-fme/+merge/77519 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openerp-web/trunk-search2-fme. _______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp
