Results 1 to 6 of 6

Thread: SugarFeed - see all records - related_module blank in SugarFeed->get_list_view_data

  1. #1
    eggsurplus's Avatar
    eggsurplus is offline Sugar Community Member
    Join Date
    Dec 2005
    Location
    Minnesota
    Posts
    2,343

    Default SugarFeed - see all records - related_module blank in SugarFeed->get_list_view_data

    When I output $data['RELATED_MODULE'] in get_list_view_data in SugarFeed.php it returns a blank value for any object like a case or contact which means that the security check isn't doing it's job at this line:

    if(ACLController::moduleSupportsACL($data['RELATED_MODULE']).......

    Any idea on how to get this to return the related info as it is expecting?

  2. #2
    eggsurplus's Avatar
    eggsurplus is offline Sugar Community Member
    Join Date
    Dec 2005
    Location
    Minnesota
    Posts
    2,343

    Default Re: SugarFeed - see all records - related_module blank in SugarFeed->get_list_view_da

    OK, not sure why this is taken out in CE but if you want the SugarFeeds to respect your roles so that users can't see data that they shouldn't see then add 'related_module' and 'related_id' into the $this->lvs->setup call in modules\SugarFeed\Dashlets\SugarFeedDashlet\SugarF eedDashlet.php so that it looks like:
    PHP Code:
                $this->lvs->setup($this->seedBean$this->displayTpl$where $lvsParams0$this->displayRows
                                  array(
    'name'
                                        
    'description'
                                        
    'date_entered'
                                        
    'created_by'
                                        
    'related_module'//egg: added back in so that acls work correctly
                                        
    'related_id'//egg: added back in so that acls work correctly

                                        
    'link_url'
                                        
    'link_type')); 

  3. #3
    eggsurplus's Avatar
    eggsurplus is offline Sugar Community Member
    Join Date
    Dec 2005
    Location
    Minnesota
    Posts
    2,343

    Default Re: SugarFeed - see all records - related_module blank in SugarFeed->get_list_view_da

    Submitted bug 27059 with proposed fix at http://www.sugarcrm.com/crm/?option=...g=Bug+saved%21

  4. #4
    SugarDev.net is offline Sugar Community Member
    Join Date
    Feb 2008
    Posts
    1,401

    Default Re: SugarFeed - see all records - related_module blank in SugarFeed->get_list_view_da

    Thnx for diving into this!
    Developers go here
    Businesses go there (Dutch)

    Modules:
    SugarDev.net Developer Tools | Config | Dutch Language Pack
    "Nothing gets fixed unless there is a bug"

  5. #5
    kinshibuya's Avatar
    kinshibuya is offline A Sugar Hero
    Join Date
    Jul 2008
    Location
    brasil
    Posts
    521

    Default Re: SugarFeed - see all records - related_module blank in SugarFeed->get_list_view_da

    Hi!
    i was cheking, the security groups module already comes with this code right?
    Priscila Kin Yamamoto Joranhezon
    Phone: +55 61 32010000
    priscila.joranhezon@nct.com.br
    NCT Informática Ltda

  6. #6
    eggsurplus's Avatar
    eggsurplus is offline Sugar Community Member
    Join Date
    Dec 2005
    Location
    Minnesota
    Posts
    2,343

    Default Re: SugarFeed - see all records - related_module blank in SugarFeed->get_list_view_da

    Yes it does. It looks like Sugar has addressed this issue in 5.2.0c so I no longer have to include it manually with each release.

Thread Information

Users Browsing this Thread

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

Similar Threads

  1. A blank (realy blank) module from module builder???
    By dogfuel in forum Developer Help
    Replies: 1
    Last Post: 2008-01-06, 01:15 PM
  2. Replies: 14
    Last Post: 2007-05-29, 11:09 PM

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
  •