Merge branch 'hotfix-2.2/MNT-17550' into 'release/V2.2.0.x'

MNT-17550: Add in null checks to prevent NPE and allow correct error codes to bubble up.

See merge request !114
This commit is contained in:
David Webster
2017-03-09 13:16:14 +00:00
2 changed files with 15 additions and 3 deletions

View File

@@ -195,6 +195,12 @@
<classifier>config</classifier> <classifier>config</classifier>
</dependency> </dependency>
<!-- Test dependencies --> <!-- Test dependencies -->
<dependency>
<groupId>org.testng</groupId>
<artifactId>testng</artifactId>
<version>6.10</version>
<scope>test</scope>
</dependency>
<dependency> <dependency>
<groupId>junit</groupId> <groupId>junit</groupId>
<artifactId>junit</artifactId> <artifactId>junit</artifactId>

View File

@@ -33,8 +33,11 @@ public class ExtendedFileFolderServiceImpl extends FileFolderServiceImpl
finally finally
{ {
recordService.enablePropertyEditableCheck(); recordService.enablePropertyEditableCheck();
if (result != null)
{
recordService.disablePropertyEditableCheck(result.getNodeRef()); recordService.disablePropertyEditableCheck(result.getNodeRef());
} }
}
return result; return result;
} }
@@ -52,8 +55,11 @@ public class ExtendedFileFolderServiceImpl extends FileFolderServiceImpl
finally finally
{ {
recordService.enablePropertyEditableCheck(); recordService.enablePropertyEditableCheck();
if (result != null)
{
recordService.disablePropertyEditableCheck(result.getNodeRef()); recordService.disablePropertyEditableCheck(result.getNodeRef());
} }
}
return result; return result;
} }