after_ui_frame getting called twice now in 6.2.1

**eggsurplus** · 2011-07-25, 09:40 PM

In 6.2.1 the after_ui_frame logic hook gets triggered twice now in SugarView.php->process(). Is this as designed? I'd think the below code was meant to be an if/else but not sure. Need to know so that I can find the appropriate work around. Thanks!

PHP Code:

         if ( !empty($this->module) ) {
            $GLOBALS['logic_hook']->call_custom_logic($this->module, 'after_ui_frame');
        }
        $GLOBALS['logic_hook']->call_custom_logic('', 'after_ui_frame');

**Rodolfo Lima Jesus** · 2011-08-22, 08:18 PM

Hi Jason.

We were testing security suite module in same version and we find same bug.
So we make this simple modification and, apparently, everything is working now.

PHP Code:

 if ( !empty($this->module) ) 
     $GLOBALS['logic_hook']->call_custom_logic($this->module, 'after_ui_frame');
else
     $GLOBALS['logic_hook']->call_custom_logic('', 'after_ui_frame');

Best regards.

**eggsurplus** · 2011-08-24, 01:04 AM

Rodolfo,

Yep, exactly. I bring this up since this is in the SugarCRM code base though. It just happens to affect the SecuritySuite functionality.

**collinlee** · 2011-08-24, 04:02 AM

Looks like that code came in from Bug 26880 (when the InsideView Connector was added). It does appear to be a product bug and the else statement should have been added.