Closed (fixed)
Project:
Examples for Developers
Version:
8.x-1.x-dev
Component:
Cron Example
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
16 Dec 2012 at 01:39 UTC
Updated:
16 Feb 2014 at 06:21 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
marvil07 commentedHere a first functional version!
Please let me know what more needed here.
Comment #2
marvil07 commentedComment #3
mile23Thanks, marvil07.
Many references to Drupal 7.
Adopting the structure of a D8 module is great (with lib directory, etc.), but the documentation needs some editorial attention as well.
Comment #4
mile23You're welcome to submit more patches, but we're holding off on D8 a bit: #1861598: Examples for Drupal 8 needs relatively stable feature set
Comment #5
mile23Comment #6
marvil07 commentedminor re-roll with info file to yml, still needs real changes
Comment #7
wesleydv commentedRewrote the patch to use a form controller and new Drupal 8 function like config mgmt
Comment #8
tsphethean commentedhook_cron_queue_info should be hook_queue_info and the comment needs updating as this is a new hook in D8.
No full stop on contains
This comment is a little odd
Shouldn't these be got from $config->get()
no newline at end of file?
Comment #9
wesleydv commentedFixed issues raised by tsphethean
Comment #10
marvil07 commented@wesleydv: Thanks for the new patch implementing new APIs!
I will try to make a closer look as soon as I fix the tests for other submodules already added to 8.x-1.x, so your patch can be tested by testbot. I only could fix block_example today.
Comment #11
marvil07 commentedI added the last patch to 8.x-1.x, thanks!
Comment #13
mile23Note: This module is now very broken because it relies on variable_set().
#2197267: Remove variable_set()/variable_get() from cron_example