Publisher is EE6 compatible, but it does not support the Fluid field. Please do not contact asking when support will be available.

If you purchased an add-on from expressionengine.com, be sure to visit boldminded.com/claim to add the license to your account here on boldminded.com.

Ticket: Structure created pages in ignored channel does not get the site_pages set correctly

Status Resolved
Add-on / Version Publisher Lite 3.1.5
Severity
EE Version 5.3.2

Ron Hickson

May 20, 2020

When we have Structure-created pages (which are also an ignored channel in Publisher) the URL on the front-end after saving is an illegal offset.  When looking through what was occurring it looks like the pages in ignored channels are not added to the publisher_site_pages table.  I’m not sure if there are other ramifications to the following change but moving the call of the “saveIgnored” method to the after_channel_entry_save (instead of the before_channel_entry_save) appeared to solve the issue since an entryId exists at that point for setting the array.  Are there other foreseeable issues this would cause?

#1

BoldMinded (Brian)

May 20, 2020

This should have been fixed in the 3.1.3 release on March 6th. Can you take a screenshot of your code change?

[Added] Saving an entry in an ignored channel that is also a Pages or Structure managed channel should correctly save the page data

#2

Ron Hickson

May 20, 2020

Added to the ticket.

#3

BoldMinded (Brian)

May 20, 2020

Offhand I can’t think of what side effects might be. I’ll have to take a closer look at it. Thanks for mentioning this. For now I guess if it’s working for both ignored and not Ignored channels then keep it as is.

#4

BoldMinded (Brian)

May 20, 2020

OK, I think you can keep it the way you have it. The before_channel_entry_save hook doesn’t really do that much, so the code move makes sense. I’ll include this fix in the next release. Thanks for pointing it out!

Login to reply