umbraco/Umbraco-CMS — pull request #4 by dpen2000 (2013-06-12 15:33:10+00:00)
Title: Fix Error on saving User Permissions

This fixes the bug here:
http://issues.umbraco.org/issue/U4-2320

My extended commit comment:
Nodes can be null but NodeIds property always returns a collection and this will return Ids from Nodes if _nodeIds are null and Nodes is not.

A bit more detail: DeletePermissions in Permission.cs calls the constructor of UserPermissions with an array of NodeIds and so when InvalidateCacheForPermissionsChange is called with one of these objects only NodeIds are available, not Nodes.