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.
This module needs to be updated to run on the drupal 6 branch.
Comment | File | Size | Author |
---|---|---|---|
#7 | poormanscron.module.rej_.diff | 2.51 KB | chb |
#4 | poormanscron.drupal6_2.patch | 4.18 KB | RobLoach |
#1 | poormanscron.drupal6.patch | 3.92 KB | RobLoach |
Comments
Comment #1
RobLoachThere's still something wrong with the menu, not sure what though. It's reporting
Invalid argument supplied for foreach()
, so we must be doing something wrong here....Comment #2
netbjarne CreditAttribution: netbjarne commentedI'm just hanging on here - been poormanscron fan since 4.something.
Bjarne
Comment #3
RobLoachIf anyone figures out what's wrong with the menu in that patch, please let me know. I'm curious as to what I'm doing wrong.
Comment #4
RobLoachIt seemed the issue was with my cache, it's working now. Updated patch with UNIX end lines and a documentation fix. Good to be reviewed by you guys.
Comment #5
RobLoachDoes this branch need a maintainer? Because I'd be willing to help out with that...
Comment #6
litwol CreditAttribution: litwol commentedi think it does.
Comment #7
chb CreditAttribution: chb commentedJust tried to apply the patch to cvs from 05.02.2008, something gone wrong:
Comment #8
RobLoachOh, sorry... This is leading off of the DRUPAL-5 branch, because that is the one that was most recently updated.
Comment #9
chb CreditAttribution: chb commentedThanks for pointing that out. Now it looks like as if the patch is already applied, so I just need to bump the Version number to 6.x. I get
I'm using Drupal-6 rc3, maybe something else has changed.
Comment #10
RobLoachI've fixed up HEAD to include the hook_menu fixes from my patch, some watchdog API changes in Drupal 6, and poormanscron.admin.inc, which hooks into the menu system. Do you think we should make the DRUPAL-6--1 branch?
Comment #11
chb CreditAttribution: chb commentedI just installed it so far it is working, thanks :-). I really would like to have a drupal-6 branch but I can't contribute much code to it...
Comment #12
ax CreditAttribution: ax commentedas Dries created the DRUPAL-6 branch for core, meaning that HEAD isn't necessarily compatible with Drupal 6 anymore, i just created the corresponding DRUPAL-6--1 CVS branch and the development snapshot release for this module. please do all Drupal 6 development on this branch now.
Comment #13
RobLoachThanks, ax!
Comment #14
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.