The link created to 'settings' in the dblog when the internal lightweight cron is run still has the D6 path. See attached screen shots. It should be admin/config/content/scheduler not admin/settings/scheduler

Patch coming up.

Jonathan

CommentFileSizeAuthor
#1 _1358656_2.scheduler.dblog_setting_link.patch718 bytesjonathan1055
dblog settings link correct.jpg107.99 KBjonathan1055
dblog settings link wrong.jpg126.23 KBjonathan1055
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 commented
Status: Active » Needs review
FileSize
_1358656_2.scheduler.dblog_setting_link.patch718 bytes

The change is in _scheduler_run_cron().
Patch against latest dev of 29th Nov.

jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 commented

Hi,
Would anyone like to confirm this simple patch? Then it can committed. It is a little frustrating not to be able to use the link in the dblog table entry, but this patch fixes it.
Jonathan

kiranjala’s picture

kiranjala CreditAttribution: kiranjala commented

Hello Jonathan,

I tested this patch on a Drupal 7.10 installation and the settings link now shows up correctly on the dblogs page. Scheduler runs fine. This patch fixes any new links that will show up but not existing entries in dblog which is not a problem at all but some thing not to be confused for and also an FYI. Thanks for the patch.

Kiran

jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 commented
Status: Needs review » Reviewed & tested by the community

Thanks Kiran for your feedback.
I've set the status to RTBC - are you ok with this?

Jonathan
[edit: set status to RTBC]

rickmanelius’s picture

rickmanelius CreditAttribution: rickmanelius commented
Status: Reviewed & tested by the community » Closed (fixed)

Thanks @jonathan1055. This was committed!
http://drupalcode.org/project/scheduler.git/commit/d079cb4

jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 commented
Status: Closed (fixed) » Fixed

Thanks Rick. Tested this and I confirm the patch was committed and the dblog link is now correct.
Leaving as 'fixed' and will then be closed automatically in two weeks.

We have the first patch committed after a years gap - Scheduler is moving again!

Jonathan

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.