I think `if (data.group) data.group = data.group.id || getId(data.group)` would be more easier to read. Although I'de just expect `getId(data.group)` to take care of that in first place. Is there any reason it shouldn't ?