Skip to content
Snippets Groups Projects

Leads Record model getSearchResult

Merged its4you requested to merge its4you/vtigercrm:Leads_Record_model_getSearchResult into master

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • @its4you Can you please let me known the case scenario, that lead for this fix. Why do we need to hard code the moduleName here?

  • Author Contributor

    @uma.s but $this->getModule() is not defined

  • Author Contributor

    @uma.s The main issue is that function getSearchResult is defined as static, but inside this function you call $this->getModule() which causes fatal error.

  • @its4you Thanks! for the test-case, will analyze on this case.

  • @its4you After Analyzing through the api call flow, I can confirm that there is no call made to Leads getSearchResult() api.

    When you perform global search, Call is made to Vtiger getSearchResult() api.

    Please do update me if you have come across the test-case, where call is been made to Leads getSearchResult() api.

  • Uma Status changed to merged

    Status changed to merged

  • Uma mentioned in commit 2b2eb21a

    mentioned in commit 2b2eb21a

  • @its4you Thanks! for the fix, I tested this through standalone script which throwed an error.

Please register or sign in to reply
Loading