developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

#451 closed bug (fixed)

Xul Overlays : event should contain the full template selector syntax

Reported by: Julien Owned by: laurentj
Priority: normal Milestone: Jelix 1.0.2
Component: jelix:core response Version: 1.0.1
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed:
Hosting Provider: Php version:

Description

When doing something like this :

$window = $this->getResponse('xul');
$window->bodyTpl = 'mainxul';
$window->fetchOverlays = true;
return $window;

in testb~default:index, then whe have :

echo $event->getParam('tpl');
// prints 'mainxul'

but i think we should know the module of the template too, so we can write complete test like (extract from Jelix manual) :

class testaListener extends jEventListener{
 
   function onFetchXulOverlay($event){
      if($event->getParam('tpl') == 'testb~mainxul'){
            $event->Add('testa~xul:overlay1');
        }
   }
}

Attachments (1)

451-jResponseXul-fetchOverlays-fullTemplateSelector.patch (1.2 KB) - added by Julien 12 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 Changed 12 years ago by laurentj

  • Milestone set to Jelix 1.0.2
  • Severity changed from normal to minor
  • Status changed from new to assigned

comment:2 Changed 12 years ago by laurentj

  • Resolution set to fixed
  • Status changed from assigned to closed

review+

Patch landed in the trunk and 1.0.x branch. Thanks for this patch !

Note: See TracTickets for help on using tickets.