Code Smell: 'ProcessZeroEntitiesFirst' calls 'All' but does not use the value the method returns. Linq methods are known to not have side effects. Use the result in a conditional statement, assign the result to a variable, or pass it as an argument to another method

Capgemini / xrm-datamigration

Export and import data for Microsoft Dataverse. Supports JSON and CSV.

MIT License

25 stars 11 forks source link

Code Smell: 'ProcessZeroEntitiesFirst' calls 'All' but does not use the value the method returns. Linq methods are known to not have side effects. Use the result in a conditional statement, assign the result to a variable, or pass it as an argument to another method #93

Open tdashworth opened 2 years ago

tdashworth commented 2 years ago

'ProcessZeroEntitiesFirst' calls 'All' but does not use the value the method returns. Linq methods are known to not have side effects. Use the result in a conditional statement, assign the result to a variable, or pass it as an argument to another method.

https://sonarcloud.io/project/issues?resolved=false&rules=external_roslyn%3ACA1806&severities=MAJOR&id=xrm-datamigration&open=AXlmWBvHKjdVQvrIoNHe