Skip to content

Conversation

@kitepascal
Copy link
Contributor

Pull Request for Issue #37351 .

Summary of Changes

Testing Instructions

Save as Copy

Actual result BEFORE applying this Pull Request

Article is published.

Expected result AFTER applying this Pull Request

Article shoud be unpublished like in J3 before.

Documentation Changes Required

None.

Fixes #37351

kitepascal and others added 10 commits March 5, 2022 01:41
Making things clearer :)
Making things clearer
Removed br spaces and adjusted the link href to the language specific doc.
Inserted /Special:MyLanguage/ also into the link anchor. Was unsure about that before.
Like modules and menu items, also articles should be unpublished on save2copy.
@toivo
Copy link
Contributor

toivo commented Mar 23, 2022

I have tested this item ✅ successfully on b17c1c5

Tested successfully in 4.1.1-rc2-dev of 23 March using PHP 8.0.15


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37354.

@Shubhamverma2796
Copy link
Contributor

I have tested this item ✅ successfully on b17c1c5


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37354.

@brianteeman
Copy link
Contributor

I am a little unsure that this is the correct fix. It would be useful to find out when/where the j3 behaviour was changed. Was there a reason for it, was anything else changed at the same time. There may be more needed than just this one line.

@richard67
Copy link
Member

I am a little unsure that this is the correct fix. It would be useful to find out when/where the j3 behaviour was changed. Was there a reason for it, was anything else changed at the same time. There may be more needed than just this one line.

Yes, I have the same feeling, but I'm at work so can't investigate. I hope someone else can step in if necessary.

@brianteeman
Copy link
Contributor

Well there is no rush to merge it ;)

Thinking aloud but it might be something to do with the creation of multilingual associated articles

@chmst chmst added the Updates Requested Indicates that this pull request needs an update from the author and should not be tested. label Mar 23, 2022
@kitepascal
Copy link
Contributor Author

it might be something to do with the creation of multilingual associated articles

Created multilingual associated articles are still published initially, just tested that.
Also if you create an associated translation of an unpublished article.

@laoneo
Copy link
Member

laoneo commented Mar 23, 2022

Perhaps @bembelimen can shed some light in here as it can relate to the workflow changes.

@chmst chmst removed the Updates Requested Indicates that this pull request needs an update from the author and should not be tested. label Mar 23, 2022
@richard67
Copy link
Member

Perhaps @bembelimen can shed some light in here as it can relate to the workflow changes.

I think that could be the point. When workflow is enabled, it should use the default stage, which might be different. Maybe we should look up in the code how it's done for batch copying?

@bembelimen
Copy link
Contributor

Workflow didn't change the behaviour... I wanted to do so, but there was too much restistant and for me it was not worth the time to fight for it...so I think copy should always save unpublished, but that is just my opinion.

@brianteeman
Copy link
Contributor

Well something changed it - weird

@HLeithner
Copy link
Member

This pull request has automatically rebased to 4.2-dev.

@joomla-bot
Copy link
Contributor

This pull requests has been automatically converted to the PSR-12 coding standard.

@Quy
Copy link
Contributor

Quy commented Feb 4, 2023

Closing in favor of #38810 as it has more in-depth discussion. Thank you for your contribution.

@Quy Quy closed this Feb 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PBF Pizza, Bugs and Fun

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[J 4.1.x] Save as copy saves article published