Description: ------------ When using the checkstyle-format and writing to a report-file, each run no errors are found a new report is nested inside the existing report, resulting in invalid XML. I ran into this problem while using Jenkins, which fails to parse the XML report for all projects that lack codestyle-errors. Test script: --------------- Run this command; phpcs --report=checkstyle --report-checkstyle=/tmp/phpcs/report /tmp/phpcs/example.php Where '/tmp/phpcs/example.php' is completely empty or otherwise contains no errors for the sniffs that are used. The first run the report file is ok. Each consecutive run duplicate nodes are added, corrupting the XML structure. This only occurs when not a single error or warning is found, checkstyle formatting is used, and a previous report file already exists. Expected result: ---------------- /tmp/phpcs/report: <?xml version="1.0" encoding="UTF-8"?> <checkstyle version="1.5.4"> </checkstyle> Actual result: -------------- /tmp/phpcs/report: <?xml version="1.0" encoding="UTF-8"?> <checkstyle version="1.5.4"> <?xml version="1.0" encoding="UTF-8"?> <checkstyle version="1.5.4"> </checkstyle> </checkstyle>

This was a general error for all report types, but most notable in the ones that output some content even when nothing is found. Fix is committed here: https://github.com/squizlabs/PHP_CodeSniffer/commit/933bd158d79247993bdf6e547c 55fcdcbefa6b3e