Bug #147
SugarCRM lookup failed
| Status: | Closed | Start: | 06/23/2009 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assigned to: | tshif | % Done: | 100% |
|
| Category: | - | |||
| Target version: | Caller ID Superfecta v 2.1.X Maintenance Releases |
Description
Even if you set the search type to 2 in freepbx (0 is for account, 1 is for account +users, 2 is for account+users+contact)
only account is used for lookup, phone number in users or contact database are not looked up.
Patch: replace the two:
if($wsearch_type >=
by
if($param['SugarCRM']['Search_Type'] >=
Also in the pop up help window for the Search type, instead of 'I'm not sure of what it is for' replace it by:
0 is for accounts lookup only, 1 is for accounts + users lookup, 2 is for accounts + users + contacts lookup.
And by the way, it would have been more logical for me to have users as the last one instead of contact. I'm receiving more phone calls from my customers than from my employees...
History
Updated by patrick_elx over 2 years ago
- Target version set to Caller ID Superfecta v 2.1.X Maintenance Releases
- % Done changed from 0 to 80
The change of the code was made by jacobsj in rev 107.
Just still need to change line 25 with the proper comment:
0:account lookup, 1: account and users lookup, 2: account, users and contact lookup
Updated by patrick_elx over 2 years ago
- Assigned to set to patrick_elx
- % Done changed from 80 to 100
help string updated in rev 108
Updated by tshif over 2 years ago
- Status changed from New to Resolved
- Assigned to changed from patrick_elx to tshif
- % Done changed from 100 to 90
Awaiting QS Testing.
Updated by tshif over 2 years ago
- Status changed from Resolved to Closed
- % Done changed from 90 to 100
QS: Pass. Module performs as expected with this patch. We have no Sugar CRM to test against - but these changes do not create any new failures, so it has passed QS.