CRM-18445 don't 'Assign Case Role' to client

[kenwest]

This PR removes an unnecessary test that Case Id is set. Observing how the function is called, the Case Id will always be set, so the condition is always true and the else part is never executed.

Even if Case Id was not set, this test is not the appropriate response.

---

• CRM-18445: In 4.7 the activity 'Assign Case Role' is assigned to the client

[colemanw]

Jenkins, retest this please.

[colemanw]

I've reviewed the code in question and agree that this is the proper fix. There is no need to check isset for a property that is always set. I haven't user-tested this myself but I trust Ken's assessment. Chances of regressions are low because the patch is limited to the 1 form exhibiting the bug.