"Override the page template" cannot be disabled

I suppose this is the way:

Making Panels Everywhere aware themes     
=====================================     
     
To make a theme PE aware, all that really matters is to provide a default site    
template that matches what the theme's page.tpl.php should be. To do this,     
create a site template in your site. Export the handler via the bulk export     
mechanism. Edit your .info file to contain these lines:    
     
; We provide default page manager pages for our site template     
api[page_manager][pages_default][version] = 1     
api[page_manager][pages_default][path] = pages    
     
The bulk export will give you a .pages_default.inc file -- just place that in      
the 'pages' directory. Your new site template should be immediately available.    
     
It's a very good idea to add a 'selection criteria' so that this template will    
only activate when your theme is the active theme.     
    
You can also give your theme a hybrid mode where it will be smart and use its    
normal page.tpl.php if the site_template is not in use, and use a stripped     
down page.tpl.php if it is. Place the following code in your theme (chances     
are you already have a preprocess page).     
      
  function MYTHEME_preprocess_page(&$vars) {     
    if (!empty($vars['panels_everywhere_site_template'])) {     
      $vars['template_file'] = 'page-panels-everywhere';    
    }     
  }    
     
Then copy panels_everywhere/theme/page.tpl.php to     
page-panels-everywhere.tpl.php in your theme. Once this is done, your theme    
will play nice with Panels Everywhere even if the option to take over the    
page template is not enabled.    
     
There is rather a lot of information available via this variable if you like.     
The actual template used will be in $vars['panels_everywhere_site_template']['handler']    
and the contexts will be in $vars['panels_everywhere_site_template']['contexts'].    

Just tried, and no way out. I'm using last -dev release of Ctools and Panels Everywhere, and I've flushed all caches (including theme registry). If I select "Override the page template" I got my site_template working (but not rendering correctly as my theme is supposed to work with its own page.tpl.php).
If I go back to Panels Everywhere 6.x-1.1 everything goes ok (or simply changing the new panels_everywhere.module for the old one)...

I'd have to debug this (well, I'm more a designer than a programmer, but I'll try). I've tried applying all the patches included in -dev version to the stable release and no problem with them, but the patch in the issue http://drupal.org/node/872260 generates the bug I'm reporting here. So, having a look around I guess the problem comes from here (Am I wrong - ? -),

http://drupal.org/node/872260#comment-3327300

Here's a new version of the patch. I found that PanelsEverywhere was being activated by merely turning on the site template. With this version, you need to explicitly enable the "Override page template" option (or do it per-theme).

Well, I'd like to use PE without enabling the "Override page template" option (as it was in stable release - to have more control on my page.tpl.php -), it's still possible?

I'd like to suggest to go back from

if (!panels_everywhere_should_override_theme()  

to

 if (variable_get('panels_everywhere_site_template_enabled', FALSE)) --- as it is in last stable release   

in panels_everywhere.module - to allow activate PE without enabling the "Override page template" for the ones who'd like to use their own page.tpl.php -

Here's a patch I've tested in my local environment, I'm just replacing a couple of lines with the old ones
Does it make sense to you? Don't know if I'm doing right!? (or the module will be supposed to work only if "Override page template" is enabled...?)

thanks for testing

I've tested the patch and it seems ok! I'll test more during the day and I'll report if I find any bug.
Thanks for improving the patch. It would be great to have the "Enable Panels Everywhere site template on a per-theme basis" option available even if the "Override the page template" checkbox is not selected.

Here's a patch (to be tested)

Sure! Thanks for testing, I've posted the patch here: http://drupal.org/node/872260#comment-3856656

@mcpuddin
I've tested deeper the patch at #10:
if "Override the page template" checkbox is not selected it's all ok, but try

- selecting "Override the page template"
- selecting "Enable Panels Everywhere site template on a per-theme basis" and your PE-compatible theme
- change theme to a PE-no-compatible theme (e.g., Garland)

Going back to front-end (where I've activated Garland) I get a PHP error - it seems like Panels Everywhere site template is enabled for all themes
I don't know if this is something specific for my local environment, so it'll be great if someone can try by himself and report
Maybe a fix is to remove completely the patch posted by NigelCunningham @ http://drupal.org/node/872260#comment-3341716 and discuss the integration of the option to enable PE on a per-theme basis in the same issue (http://drupal.org/node/872260), as mcpuddin was suggesting

.

Hi James,
you're right!, I've tried again and everything seems fine, I think it was something specific for my local environment

Hope more people can test the patch @ #10, so that it'll be possible to change the status to RTBC

Hmmmm, well, maybe it will be better to have more opinions about the patch ( ... unluckily I'm not an "experienced developer" )

Reviewed & Tested by the Community ["RTBC"]
The patch has received a thorough review and test by one or more experienced developers, and they have deemed it as ready to be committed to the project.
One should not set their own patch to this status. Patches need to go through the review process. Some exceptions may be made if another developer voices their support, or when the patch is utterly trivial (e.g. fixing a spelling mistake in a comment). (...)

http://drupal.org/node/156119