Check for before destorying CSharpierProcessPipeMultipleFiles.java (#682

)

* Check for before destorying CSharpierProcessPipeMultipleFiles.java

* Releasing 1.2.7 to fix NRE

Co-authored-by: Bela VanderVoort <twobitbela@gmail.com>

Src/CSharpier.Rider/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 # csharpier-rider Changelog
 
+## [1.2.7]
+- Fix Null Reference Exception from CSharpierProcessPipeMultipleFiles if it fails to start process
+
 ## [1.2.6]
 - Remove plugin until property so plugin doesn't need to be updated for each new rider version
 

Src/CSharpier.Rider/README.md

@@ -22,3 +22,14 @@ Please report any [issues](https://github.com/belav/csharpier/issues)
 ---
 Plugin based on the [IntelliJ Platform Plugin Template][template].
 
+## Troubleshooting
+
+### Viewing CSharpier Logs
+- Use the action "Show Log in Explorer"
+- Look for entries for "CSharpierLogger"
+
+### Enable Debug Logging for CSharpier
+- Use the action "Debug Log Settings"
+- Add entry for "#com.intellij.csharpier.CSharpierLogger"
+- Restart Rider
+

Src/CSharpier.Rider/gradle.properties

@@ -4,7 +4,7 @@
 pluginGroup = com.intellij.csharpier
 pluginName = csharpier
 # SemVer format -> https://semver.org
-pluginVersion = 1.2.6
+pluginVersion = 1.2.7
 
 # See https://plugins.jetbrains.com/docs/intellij/build-number-ranges.html
 # for insight into build numbers and IntelliJ Platform versions.

Src/CSharpier.Rider/src/main/java/com/intellij/csharpier/CSharpierProcessPipeMultipleFiles.java

@@ -81,6 +81,8 @@ public String formatFile(String content, String filePath) {
 
  @Override
  public void dispose() {
- this.process.destroy();
+ if (this.process != null) {
+ this.process.destroy();
+ }
  }
 }