Expand Minimize

SPC059009: Consider verifying that the user has sufficient permissions when calling ListItem.Update()

Before updating a ListItem object, consider making sure that the user has permissions to perform that action.

CheckId SPC059009
TypeName CheckHasListItemPermissionsBeforeUpdate
Severity Warning
Type Assembly

Bad Practice

using (ClientContext context = new ClientContext("http://yoursite"))
{
    List list = context.Web.Lists.GetByTitle("MyCustomerList");
    ListItem item = list.GetItemById(1);
    item["Title"] = "My New Customer Title LOL";
    item.Update();
    context.ExecuteQuery();
}

Good Practice
using (ClientContext context = new ClientContext("http://yoursite"))
{
    List list = context.Web.Lists.GetByTitle("MyCustomerList");
    context.Load(list, l => l.EffectiveBasePermissions);
    context.ExecuteQuery();

    if (list.EffectiveBasePermissions.Has(PermissionKind.EditListItems))
    {
        // Do stuff with list.
        ListItem item = list.GetItemById(1);
        item["Title"] = "My New Customer Title";
        item.Update();
        context.ExecuteQuery();
    }
}

To suppress this violation in managed code add the following attribute to the method which contains the instruction (available since SPCAF version v5.2). Learn more about SuppressMessage here.

// Important: Ensure to have #define CODE_ANALYSIS at the beginning of your .cs file
[SuppressMessage("SPCAF.Rules.ManagedCSOM.CSOMBestPracticesGroup", "SPC059009:CheckHasListItemPermissionsBeforeUpdate", Justification = "Provide reason for suppression here")]
Disclaimer: The views and opinions expressed in this documentation and in SPCAF do not necessarily reflect the opinions and recommendations of Microsoft or any member of Microsoft. SPCAF and RENCORE are registered trademarks of Rencore. All other trademarks, service marks, collective marks, copyrights, registered names, and marks used or cited by this documentation are the property of their respective owners.