Skip to content

[5.0] New task plugin for check in items#37842

Merged
HLeithner merged 74 commits intojoomla:5.0-devfrom
rdeutz:new_task_example_checkin
Aug 26, 2023
Merged

[5.0] New task plugin for check in items#37842
HLeithner merged 74 commits intojoomla:5.0-devfrom
rdeutz:new_task_example_checkin

Conversation

@rdeutz
Copy link
Copy Markdown
Contributor

@rdeutz rdeutz commented May 21, 2022

New feature

Thanks to @brianteeman for the idea.

Summary

New task plugin to check in all checked out items.

Testing Instructions

  • Apply patch
  • Check out some items
  • Create a new task in system/Scheduled Tasks
  • Select Global Checkin
  • Set Execution Rule to Manual Execution
  • Set a Title
  • Save
  • You should see something like this

grafik

  • Click on Run Task
  • Go Back to the checked out items, they should be checked in

You can also set a delay time in hours, then only items that are checkout before actual - delay time will be checked in.

Documentation Changes Required

Documentation needs to be updated

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.2-dev labels May 21, 2022
@rdeutz rdeutz changed the title [4.2]Create a new example plugin [4.2] New example task plugin for check in items May 21, 2022
@HLeithner
Copy link
Copy Markdown
Member

Nice plugin but blindly checking in all it's seems wrong to me. Can we get a parameter which allows us to configure an expire timeframe?

Something lime "Minutes to expire" default 60?

@richard67
Copy link
Copy Markdown
Member

Nice plugin but blindly checking in all it's seems wrong to me. Can we get a parameter which allows us to configure an expire timeframe?

Something lime "Minutes to expire" default 60?

I just wanted to say the same.

@brianteeman
Copy link
Copy Markdown
Contributor

Nice plugin but blindly checking in all it's seems wrong to me. Can we get a parameter which allows us to configure an expire timeframe?

Something lime "Minutes to expire" default 60?

I just wanted to say the same.

Except the core component does it blindly as well

@laoneo
Copy link
Copy Markdown
Member

laoneo commented May 21, 2022

Can you please use the modern way with a service provider.

@richard67
Copy link
Copy Markdown
Member

Nice plugin but blindly checking in all it's seems wrong to me. Can we get a parameter which allows us to configure an expire timeframe?
Something lime "Minutes to expire" default 60?

I just wanted to say the same.

Except the core component does it blindly as well

The core component is change controlled by a human while a scheduler task runs unattended.

@rdeutz
Copy link
Copy Markdown
Contributor Author

rdeutz commented May 21, 2022

Changed so that you can set a delay time.

rdeutz and others added 6 commits May 22, 2022 14:25
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
Co-authored-by: Konstantin Kolos <k0nstantinkolos88@gmail.com>
Co-authored-by: Konstantin Kolos <k0nstantinkolos88@gmail.com>
Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
@rdeutz rdeutz marked this pull request as draft May 22, 2022 13:13
@rdeutz
Copy link
Copy Markdown
Contributor Author

rdeutz commented May 22, 2022

needs a bit more time fighting with the service provider

@HLeithner HLeithner changed the title New example task plugin for check in items [5.0] New task plugin for check in items Jul 27, 2023
@richard67
Copy link
Copy Markdown
Member

@rdeutz It needs to rename the update SQL scripts again because meanwhile we have some with a higher schema version (base name) in the repo so yours would not run on an update. I suggest to rename them to "5.0.0-2023-08-27.sql" because there are other PR's proposed which will already use "5.0.0-2023-08-22.sql" to "5.0.0-2023-08-26.sql".

@rdeutz
Copy link
Copy Markdown
Contributor Author

rdeutz commented Aug 26, 2023

@richard67 done, with a bit luck some tests today and we can merge this

@richard67
Copy link
Copy Markdown
Member

@richard67 done, with a bit luck some tests today and we can merge this

@rdeutz Thanks.

@webgras
Copy link
Copy Markdown
Contributor

webgras commented Aug 26, 2023

I have tested this item ✅ successfully on 6b3e80e

also check articles on two different times

tested on J 5.0.0-beta1-dev


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

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 26, 2023

I have tested this item ✅ successfully on 6b3e80e


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

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 26, 2023

some kind of cache issue
btw i'm fully confortable with this pr
waiting for the last mile
i'll redo my test

rdeutz and others added 4 commits August 26, 2023 14:35
Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
@richard67
Copy link
Copy Markdown
Member

@alikon @webgras Could you test again with the latest changes? Thanks in advance.

Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
@heelc29
Copy link
Copy Markdown
Contributor

heelc29 commented Aug 26, 2023

I have tested this item ✅ successfully on 6b3e80e


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

1 similar comment
@webgras
Copy link
Copy Markdown
Contributor

webgras commented Aug 26, 2023

I have tested this item ✅ successfully on 6b3e80e


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

@HLeithner HLeithner merged commit 0a5d604 into joomla:5.0-dev Aug 26, 2023
@HLeithner
Copy link
Copy Markdown
Member

finally thanks all

@HLeithner HLeithner added this to the Joomla! 5.0 milestone Aug 26, 2023
@rdeutz
Copy link
Copy Markdown
Contributor Author

rdeutz commented Aug 26, 2023

Thanks to all who helped to get this done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Language Change This is for Translators Maintainers Checked Used if the PR is conceptional useful

Projects

None yet

Development

Successfully merging this pull request may close these issues.