Results 1 to 8 of 8

Thread: Please provide a patch for bug 9291

  1. #1
    Proware's Avatar
    Proware is offline A Prolific Poster
    Join Date
    Mar 2006
    Location
    Sydney Australia
    Posts
    310

    Default Please provide a patch for bug 9291

    Please help! Help! HELP!

    Bug #9291 is really causing us serious grief and making our Sugar system unusable.

    We have 20 field agents who are screaming at us because of this bug!!

    Can anyone or the Sugar team please Please PLEASE provide a patch for this bug urgently.

    It is only marked as a Medium priority bug but it is HIGH HIGH HIGH priority for us.
    Cheers

    David Younger
    TSM - The Service Manager
    http://www.theservicemanager.com


    Operating system type and version: Windows 2003 Server
    » Sugar Suite version 4.5.1e Professional
    » Webserver type and version Microsoft-IIS/6.0
    » PHP version 5.2.6
    » MySQL server version 5.0.51a

  2. #2
    stevec is offline Sugar Community Member
    Join Date
    Oct 2005
    Location
    London
    Posts
    1,100

    Default Re: Please provide a patch for bug 9291

    I concur - just noticed it too - we cannot roll out 4.5 with this present. It also affects a contact's quickcreate for calls too.

    The previous versions defaulted with the account set as the parent - with contacts being linked via the calls_contacts relationship. I'll see if I can track this down myself and I'll lean on our re-seller to push this too.
    Last edited by stevec; 2006-10-05 at 10:18 AM.

  3. #3
    stevec is offline Sugar Community Member
    Join Date
    Oct 2005
    Location
    London
    Posts
    1,100

    Default Re: Please provide a patch for bug 9291

    For info - in EditView.php - it seems that the default application string for the parent type that it should default to is being set as Accounts. However, it is being overridden by the variable "$_REQUEST['parent_type']" which could be getting set in the detail view or maybe the quickcreate script.
    Last edited by stevec; 2006-10-05 at 10:19 AM.

  4. #4
    stevec is offline Sugar Community Member
    Join Date
    Oct 2005
    Location
    London
    Posts
    1,100

    Default FIX - Re: Please provide a patch for bug 9291

    Hi,

    I've got a fix for it - it's the form created for the quickcreate ajax form setting the parent type and id as the contact (via the return module definition). You can set this to be the account by setting it to the parent type and id proper.

    Edit the two files:
    modules/Calls/tpls/QuickCreate.tpl and
    modules/Meetings/tpls/QuickCreate.tpl
    and find the sections in both:

    Code:
    <input type="hidden" name="parent_id" value="{$REQUEST.return_id}">
    <input type="hidden" name="parent_type" value="{$REQUEST.return_module}">
    and replace with:
    Code:
    <input type="hidden" name="parent_id" value="{$REQUEST.parent_id}">
    <input type="hidden" name="parent_type" value="{$REQUEST.parent_type}">
    PLEASE TEST THIS!

    By changing the definition for the Calls and Meetings quickcreate - it also effects creating calls and meetings against other possible types. E.g. creating calls against leads and bugs etc. I've tested against the situations we might use and it appears to work OK. HOWEVER, get it tested in your environment to make sure it doesn't set the parent incorrectly. If it does, then it might be simpler to set the two $_REQUEST variables within the appropriate EditView.php files within various conditions to make sure it only occurs when creating calls and meetings for contacts.
    Last edited by stevec; 2006-10-05 at 10:21 AM.

  5. #5
    Proware's Avatar
    Proware is offline A Prolific Poster
    Join Date
    Mar 2006
    Location
    Sydney Australia
    Posts
    310

    Default Re: Please provide a patch for bug 9291

    You are a true champion.

    As we no longer use leads or targets or other modules since they do not work the way we want them to, working fine for contacts is fine for me!!

    My team of sales representatives and demanding sales manager thank you over and over.

    There is a god!!
    Last edited by Proware; 2006-10-05 at 11:35 PM.
    Cheers

    David Younger
    TSM - The Service Manager
    http://www.theservicemanager.com


    Operating system type and version: Windows 2003 Server
    » Sugar Suite version 4.5.1e Professional
    » Webserver type and version Microsoft-IIS/6.0
    » PHP version 5.2.6
    » MySQL server version 5.0.51a

  6. #6
    clint's Avatar
    clint is offline Sugar Team Member | Forums Lead Moderator
    Join Date
    Aug 2004
    Location
    Silicon Valley
    Posts
    2,120

    Default Re: Please provide a patch for bug 9291

    Looking in the bug tracker, I see that this bug is marked as fixed. I'm going to check with the engineer right now and see if his fix is more extensive than what SteveC posted. If so, I'll get a hot fix posted here in this thread.

    Clint
    Sugar Developer Zone - developer resources | Sugar University - user and admin training
    Sugar Docs - user and admin documentation |
    Sugar Bug Tracker - Enter or view bugs
    SugarForge- open source modules, themes, lang packs | SugarExchange - commercial extensions

    Clint Oram
    Chief Technology Officer and Co-founder
    SugarCRM

  7. #7
    stevec is offline Sugar Community Member
    Join Date
    Oct 2005
    Location
    London
    Posts
    1,100

    Default Re: Please provide a patch for bug 9291

    Hi..
    Good news and bad news.

    The Sugar guys fixed bug 9291 for calls - the fix looks like the one I did + additional changes for a 'meridian' issue.

    However, they forgot to do the same for meetings too.

    Until that's addressed, you'll need to do the above patch by hand for the meetings quick create code.

  8. #8
    clint's Avatar
    clint is offline Sugar Team Member | Forums Lead Moderator
    Join Date
    Aug 2004
    Location
    Silicon Valley
    Posts
    2,120

    Default Re: Please provide a patch for bug 9291

    Uff. Created bug 9427 to track this for meetings.

    Clint
    Sugar Developer Zone - developer resources | Sugar University - user and admin training
    Sugar Docs - user and admin documentation |
    Sugar Bug Tracker - Enter or view bugs
    SugarForge- open source modules, themes, lang packs | SugarExchange - commercial extensions

    Clint Oram
    Chief Technology Officer and Co-founder
    SugarCRM

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •