Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Any additional volunteers to port this "Job Scheduler" module from Drupal 7 to 8?
Related pages
- Drupal 8.0.0 will be released on November 19, 2015
- News about "Porting modules and themes to Drupal 8 "
- "Drupal Module Upgrader" module. Which can do some of this automatically for you, covering many API hooks.
- Documentation about "Converting 7.x modules to 8.x"
We would be happy to contribute testing patch, quality assurance, documentation, and agile project management services if needed
Comment | File | Size | Author |
---|---|---|---|
#24 | Drupal_8_port-2615256-24.patch | 1.75 KB | intrafusion |
| |||
#18 | job_scheduler-drupal8_port-2615256-18.patch | 9.78 KB | m.lebedev |
Comments
Comment #2
intrafusionComment #3
m.lebedev CreditAttribution: m.lebedev commentedI try to make a working version of the module for drupal 8.
The main functions work already.
Comment #4
m.lebedev CreditAttribution: m.lebedev commentedThis patch is true
Comment #6
intrafusionThanks for the patch, this has been applied to the dev branch
Comment #7
m.lebedev CreditAttribution: m.lebedev commentedGreat! I will add new changes here if there will be.
Comment #8
FrancewhoaThanks for your contributions m.lebedev and intrafusion :)
Comment #9
Francewhoa@Francewhoa :) Note to myself. Updated ticket attribution.
Comment #10
m.lebedev CreditAttribution: m.lebedev commentedChangelog:
- Removed QueueWorker;
- Fixed "periodic" value save;
- Fixed a period to dispatch scheduled jobs;
- Fixed an info for log;
- Added the ability to create queues and to set a worker callback.
Comment #12
intrafusionThe new patch has been applied to the dev branch
Comment #13
m.lebedev CreditAttribution: m.lebedev commentedChangelog:
- Fixed records updating in the database. Before: A new record was creating, if it did not exist.
Comment #15
intrafusionAgain, this patch has been applied to the dev branch
Comment #16
m.lebedev CreditAttribution: m.lebedev commentedChangelog:
- Logger adds information if there is one;
- Replaced the invoking queues. Added the derivative and the queue worker. Now it compatible with the queue_ui module;
- Changed API documentation for hooks.
And.. it's all. I a lot of time thinked about realize queues, but I decided to leave everything as it is (from D7). If you have any ideas how to improve the module, then I will be glad if help is provided. For example, refactoring and writing documentation.
I think this module is almost finished.
Thanks.
Comment #18
m.lebedev CreditAttribution: m.lebedev commentedApplied the codesniffer_fixes.patch
Comment #21
intrafusionOnce again thanks m.lebedev for your work on the port, if you think the module is almost finished then is it time to create an alpha/beta release?
I think any new features/improvements should be handled in separate issues so we can get this port out ASAP. The test bot will always fail as there is no test coverage and to my mind would be the only release blocker.
Comment #22
intrafusion@Francewhoa you initially stated:
If this is still on offer then I think now would be the time for this to kick in.
Comment #23
m.lebedev CreditAttribution: m.lebedev commentedI think that we can wait about 2 weeks and after that to create an alpha release.
I use this module in my project now. I will report here about errors of the module if they occur. If not, then release!
Comment #24
intrafusionI have added some very simple tests (taken from Drupal 8: Writing Your First Unit Test With PHPUnit) which will hopefully satisfy the testbot and can be added to as necessary.
Comment #25
andypostLooking at current interface
\Drupal\job_scheduler\JobSchedulerInterface
it require better naming for methods then it makes sense to implement unittest for\Drupal\job_scheduler\JobScheduler
Comment #26
m.lebedev CreditAttribution: m.lebedev commentedIs time to create an alpha release?
Comment #27
intrafusionAlpha release has been created and this issue marked as Fixed.
Any bugs can reported separately to this issue, thanks for all your help
Comment #29
xjm