Adalennis opened a new pull request, #9132: URL: https://github.com/apache/hop/pull/9132
Addresses #8970 Seventh batch: a Result section on the pages of JavaScript, Repeat, End repeat, XSD validator, XSLT, Ping, Telnet and Web service available. Every row comes from a hop-run test against the current code (54 workflows, plus the child workflows run by Repeat). What the tests show: - JavaScript: a false script gives 0 errors; only a script that can't run (syntax or runtime error) gives 1 error. A value that is not a boolean is converted the JavaScript way (0 is false, a non-empty text is true). The `errors` variable in the script still holds the previous action's error count. - Repeat: adds the result files, rows and metrics of the last run only (a child run 3 times, adding 2 rows and a different file each time, passes on 2 rows and 1 file). When the stop variable already has its value, the file isn't run and the previous result is passed on unchanged, also when it was false. The value is compared without case. An empty File to repeat or a non-numeric delay stops the workflow instead of giving a false result. - End repeat: always true with 0 errors, also outside a Repeat. - Ping, Telnet, Web service available: false with 1 error on failure and when the host or URL is empty. Web service available is false on an HTTP error status such as 404. Also fixed: - XSD validator: the page said any DOCTYPE is refused. Only a DOCTYPE that points to an external DTD, or an external entity, is refused; a DOCTYPE with only internal declarations is accepted. - XSLT: "Fail" makes the action false with 1 error, not the workflow fail; "Do Nothing" leaves the file and is true. Labels: Output Filename, Transformer factory, Do Nothing, Add output filename to result. - Ping: classic ping is true when the ping command ends with exit code 0 (on Linux and macOS: when at least one packet gets a reply), not "when no packets are lost". Labels: Action name, Nr packets to send. - Repeat labels: File to repeat (.hpl or .hwf), Keep variable values after executions, and four labels without the trailing "?". Telnet: Timeout (ms). JavaScript: a typo. Found while testing, documented as a NOTE on the XSLT page: - Since 57bd0dcdab only XML File name, XSL File name, Output Filename and If file exists are saved. Get filenames from previous result, Transformer factory, Add output filename to result and the Advanced tab have no effect once the workflow is saved and opened again, or run with hop-run. - Create new with unique name builds the new name but writes to the original file, so the existing file is overwritten. - When the XSL file doesn't exist, the error in the log names the XML file instead (not in the docs). ------------------------ Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily: - [x] Run `mvn clean install apache-rat:check` to make sure basic checks pass. A more thorough check will be performed on your pull request automatically. - [x] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using `git rebase -i`. - [x] Mention the appropriate issue in your description (for example: `addresses #123`), if applicable. To make clear that you license your contribution under the [Apache License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0) you have to acknowledge this by using the following check-box. - [x] I hereby declare this contribution to be licensed under the [Apache License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0) - [ ] In any other case, please file an [Apache Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
