if ($noofrows) {
    for ($i = 0; $i < $noofrows; $i++) {
        $attachmentIds[] = vtws_getId($WsEntityId,$adb->query_result($result, $i, 'attachmentsid'));
    }
}

@uma.s I still don't get this? Surely: ```php if ($noofrows) { for ($i = 0; $i < $noofrows; $i++) { $attachmentIds[] = vtws_getId($WsEntityId,$adb->query_result($result, $i, 'attachmentsid')); } } ``` Is all that is needed? Why treat a total of 1 row differently from >1 row?

@lord_alan Right! Changes made [Ref](https://code.vtiger.com/vtiger/vtigercrm/merge_requests/614/diffs)

:+1: