Created on 2022-05-09.15:46:18 by hodeinavarro, last changed 3 days ago by roundup-bot.
New changeset 99924f9dd683 by Cédric Krier in branch 'default': Test for sub-domain instead of not leaf in domain parser https://hg.tryton.org/tryton-env/rev/99924f9dd683
New changeset 07fec0d45308 by Cédric Krier in branch 'default': Add test for domain completion ending with or https://hg.tryton.org/tryton/rev/07fec0d45308
New changeset 8e38c5eebb96 by Cédric Krier in branch 'default': Test for sub-domain instead of not leaf in domain parser https://hg.tryton.org/sao/rev/8e38c5eebb96
Here is review411171004 that solves the problem in a generic way.
Go into any "FILTERS" text box and type a field with a value then as soon as you end the keyword "or" or "and" it will raise the mentioned exceptions.
Ex. go into Parties and type: "Name: Albright or"
Could you please provide a scenario to reproduce your problem?
Thank you for raising the bug and provide a patch for it.
P.S: Please assign the issue to yourself if you plan to work on it ;-)
ending_clause method evaluates incorrectly the
is_leaf by returning
last_element is not an instance of
Array which is the reason a loop is created.
Tested on demo websites from 5.0 to 6.4, persists in all versions.
Firefox (crashes, so it's not noticed unless checking the console)
Uncaught InternalError: too much recursion
Chrome/Edge (crashes, so it's not noticed unless checking the console)
Uncaught RangeError: Maximum call stack size exceeded
Safari gets the window hang up and CPU utilization spikes up to 100%.
p.s: Thanks to Sergi that helped me to catch the bug and evaluate a possible fix quicker than if I sought to do it alone
|2022-05-19 21:59:01||roundup-bot||set||messages: + msg76723|
|2022-05-19 21:58:53||roundup-bot||set||messages: + msg76721|
nosy: + roundup-bot
status: testing -> resolved
|2022-05-17 17:44:50||reviewbot||set||messages: + msg76702|
|2022-05-17 17:40:52||ced||set||assignedto: hodeinavarro -> ced|
messages: + msg76701
reviews: 431091003 -> 431091003,411171004
status: in-progress -> testing
|2022-05-12 01:15:42||ced||link||issue11477 superseder|
|2022-05-11 23:17:45||hodeinavarro||set||messages: + msg76600|
nosy: + ced
|2022-05-11 21:25:08||reviewbot||set||messages: + msg76596|
|2022-05-10 09:49:01||reviewbot||set||messages: + msg76575|
Showing 10 items. Show all history (warning: this could be VERY long)