Jumoo / uSync.Complete.Issues

Public Issue tracker and roadmap for uSync.Complete
https://jumoo.co.uk/usync/complete/
2 stars 1 forks source link

Unpublish not unpublishing on target #125

Closed IRooc closed 3 years ago

IRooc commented 3 years ago

Describe the bug We have two servers, staging and production. Publishing items on staging and publish to server production works fine. But when we unpublish on staging and call "publish to production". The dialog mentions that the Published boolean will be set to false (for the default culture) but on the target (production) the changelog is "Save" and "Publish" instead of unpublish.

Staging log: image

Production log: image

Version (please complete the following information):

To Reproduce Steps to reproduce the behavior: We have a 'vary by culture' doctype setup but currently only with 1 default culture en-GB

  1. Go to staging server create content for default language
  2. Click on publish to to deploy this change to production
  3. On staging click 'unpublish' to unpublish the item
  4. On staging click on publish to to deploy this change to production
  5. On production the item is still published

Expected behavior The item is also unpublished on production

Screenshots Change log on staging before publish image

Report on staging after publish image

KevinJump commented 3 years ago

Looking into this now.

IRooc commented 3 years ago

Just did a PULL from staging to our local dev environment and it pulled the draft but published the item as well.

KevinJump commented 3 years ago

Looks like its a Umbraco version thing :( - works on our base install (which is quite old v8.1) - on fresh v8.15 we do see thls issue)

having a look to see if we can find out why.

patrickdemooij9 commented 3 years ago

I wonder if it might be related to this issue: https://github.com/umbraco/Umbraco-CMS/issues/10695

KevinJump commented 3 years ago

yes - Looks something like that :(

line : 106 of the content serializer

           published.Add(new XElement("Published", item.IsCulturePublished(culture),
                    new XAttribute("Culture", culture)));

is returning true and false on the same site depending on how its called 😕

image

Will see if i can identify why - and report into issue if its the same thing

KevinJump commented 3 years ago

I think its not the same issue, but it is debatably a umbraco bug.

if you have a variant with a single culture and you unpublish the single culture. then the publishedCultures value of the content item still contains the culture although the overall item.published value = false.

I am guessing when you completly unpublish an item by unpublishing all its languages - that doesn't wipe the publishedCultures value but it does set the master unpublished value.

we can work around this in uSync. and if i can work out where this happens in Umbraco.Core i will log an issue on the main repo. because i think this is confusing.

KevinJump commented 3 years ago

Workaround for this issue, in uSync - will be part of next uSync release.