-
Notifications
You must be signed in to change notification settings - Fork 10k
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
Get the field name from its parent when it doesn't have one when collecting fields (bug 1864136) #17271
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a small typo in the commit message: doens't
should be doesn't
.
After addressing the comments, please run all tests.
/botio integrationtest |
From: Bot.io (Linux m4)ReceivedCommand cmd_integrationtest from @calixteman received. Current queue size: 0 Live output at: http://54.241.84.105:8877/32d859d93fa2b9e/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_integrationtest from @calixteman received. Current queue size: 0 Live output at: http://54.193.163.58:8877/7550afa2c84da55/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/32d859d93fa2b9e/output.txt Total script time: 6.58 mins
|
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/7550afa2c84da55/output.txt Total script time: 19.03 mins
|
The test failures was because of some fields with no name but with a parent which has one, but the parent was visited during the Fields collection. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me, with one comment; thank you!
…ecting fields (bug 1864136) Some fields, somewhere under the Fields entry in Acroform, could have no name (in T) but with a parent which has a name but which isn't somewhere under Fields. As a side-effect, this patch prevents infinite loops because of potential cycles under Fields.
/botio integrationtest |
From: Bot.io (Linux m4)ReceivedCommand cmd_integrationtest from @calixteman received. Current queue size: 0 Live output at: http://54.241.84.105:8877/d24307623ba531b/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_integrationtest from @calixteman received. Current queue size: 0 Live output at: http://54.193.163.58:8877/68401a519497981/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.241.84.105:8877/d24307623ba531b/output.txt Total script time: 5.58 mins
|
From: Bot.io (Windows)SuccessFull output at http://54.193.163.58:8877/68401a519497981/output.txt Total script time: 18.03 mins
|
Some fields, somewhere under the Fields entry in Acroform, could have no name (in T)
but with a parent which has a name but which isn't somewhere under Fields.
As a side-effect, this patch prevents infinite loops because of potential cycles
under Fields.