Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Save button still active after page has been published by automation #65

Open
ramonbakker opened this issue May 20, 2022 · 3 comments
Open
Labels

Comments

@ramonbakker
Copy link

Q A
Bug? possible
New Feature? no
Bundle Version 2.0.2 (e75a862)
Sulu Version 2.3.7 (9c4e81d7c90fba211b8e2cc76b5393215c3ae17e)
Browser Version Firefox 100.0.1 (64-bit)

Actual Behavior

image

After saving a page as a draft, I create an automation. When the automation has been executed, the page is published according to the activity log and I can see the changes on the website. The problem is that the 'Save' button is not disabled when I go to the page in Sulu, only when I clear the APCu cache by restarting the Sulu Docker container. The first two options in the dropdown of the 'Save' button are inactive, but the third one with the 'Publish' text, isn't.

Expected Behavior

I expect that the 'Save' button becomes disabled after a page has been published by an automation.

Steps to Reproduce

Described in Actual Behavior.

@alexander-schranz
Copy link
Member

@ramonbakker Thx you for creating this issue. That is really strange the handler is very simple it just call the document publish method: https://github.com/sulu/SuluAutomationBundle/blob/2.x/Handler/DocumentPublishHandler.php

Cach eissue sounds strange, maybe we need to call the service resetter after every task. Do you use multiple languages and the page was published in multiple languages?

@ramonbakker
Copy link
Author

Thanks for your reaction. I use multiple languages and I tested this bundle on pages that are published in a single language, as well as in multiple languages. The problem seems to occur in both situations.

@alexander-schranz
Copy link
Member

Thx for the feedback.

@wachterjohannes do you know what could go wrong here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants