[MPMD-369] System encoding conflicts with XML encoding in CpdViolationCheckMojo

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.20.0
• Fix Version/s: 3.21.0
• Assignee: Elliotte Rusty Harold
• Reporter: Elliotte Rusty Harold
• Created: 05/Apr/23 14:19
• Updated: 10/May/23 11:16
• Resolved: 08/Apr/23 17:40

Description

```
@Override
protected List<Duplication> getErrorDetails( File cpdFile )
throws XmlPullParserException, IOException
{
try ( FileReader fileReader = new FileReader( cpdFile ) )

{ CpdXpp3Reader reader = new CpdXpp3Reader(); CpdErrorDetail details = reader.read( fileReader, false ); return details.getDuplications(); }

}
```

It passes or fails depending on the system encoding. Instead it should use an inputstream rather than a reader and rely on the XML parser to detect the encoding. This breaks integration tests on some but not all platforms, again depending on whether the system encoding is or is not compatible with ISO-8859-1

Issue Links

• links to: GitHub Pull Request #122