Did you send me the site files for this project already for another ticket?

Yeah, if you can do that, a zip file with .sql file would help me debug it easier.

Thanks, I’ll try to look at it tonight.

The zip file isn’t opening for me, seems incomplete. I’ve downloaded it 3 times.

Ok, I am getting really, really strange results. It looks like when when trying to view a draft the site_pages array has this value “/about-us/rich-heritage” but the open version is found at “/about/rich-heritage” - so adding ?publisher_status=draft to “/about/rich-heritage” doesn’t find any entry, b/c a draft does not exist, so things go haywire (its loading the includes/_header template recursively).

So you’re trying to view a draft of an entry that EE can’t find. If the draft is intended to have a different URL, then I’m not sure how to work around this issue. If a different URL was not the intention, then you might need to take a look at the parent page and this page in question to fix the URL.

If a different URL was intended, then I might have to do a little more work, and you might have to use the {exp:publisher:toolbar} to see the draft, instead of just adding the parameter to the URL.

can you do a test for me, on line 555 of ext.publisher.php add

return $results;

should be added just before this:

$lang_id = FALSE;

See if that does anything.

That actually points me in the right direction, thanks for that report.

Tim, you may want to make these minor tweaks. Turns out the custom field save action was changed a bit in 2.6.

diff --git a/third_party/publisher/acc.publisher.php b/third_party/publisher/acc.publisher.php
index 45db644f95259c93647355fd914171b02c258c9b..7204a5f85ca63758fad41e739bf043b26130283f 100644
--- a/third_party/publisher/acc.publisher.php
+++ b/third_party/publisher/acc.publisher.php
@@ -177,12 +177,6 @@ class Publisher_acc {
         $this->sections['Pending Approvals'] = ee()->load->view('accessory', $vars, TRUE);
 
         // Handle the diff template management
-        if(ee()->input->post('field_id'))
-        {
-            ee()->load->library('Publisher/Publisher_diff');
-            ee()->publisher_diff->save_field_setting();
-        }
-
         if(isset(ee()->cp) AND ee()->input->get('D') == 'cp' AND ee()->input->get('M') == 'field_edit')
         {
             ee()->load->library('Publisher/Publisher_diff');

diff --git a/third_party/publisher/libraries/Publisher/Publisher_session.php b/third_party/publisher/libraries/Publisher/Publisher_session.php
index 50980ed6e88dea5b10fa7d6a4560367e72ab094a..f301ed0429142b0d524bc7e9e0eac1699ba98402 100644
--- a/third_party/publisher/libraries/Publisher/Publisher_session.php
+++ b/third_party/publisher/libraries/Publisher/Publisher_session.php
@@ -192,6 +192,13 @@ class Publisher_session
             {
                 $this->set(ee()->publisher_setting->get('cookie_name') .'_cp', $lang_id);
             }
+
+            // If saving a custom field, save possible diff settings.
+            if(ee()->input->get_post('M') == 'field_update' && ee()->input->get_post('field_id'))
+            {
+                ee()->load->library('Publisher/Publisher_diff');
+                ee()->publisher_diff->save_field_setting();
+            }
         }
         elseif (REQ == 'PAGE' OR REQ == 'ACTION')
         {

Going to close things, sounds like its working for you, and I have this included in the next release.