The cron query leaves an extra row in the watchdog table. See http://drupal.org/node/241467.

CommentFileSizeAuthor
dblog_cron.patch732 bytessolotandem
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kbahey’s picture

kbahey CreditAttribution: kbahey commented

Why did you post a separate issue for this one line patch?

Please include it in the original issue.

Having said that, +1 on this, it is a simple enough change.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Version: 6.x-dev » 7.x-dev

The two issues point to separate projects and components (one for SimpleTest, the other for Drupal Core / DBLog), which explains why two separate issues were required.

Having said that, this one-liner seems ready to go in. A back-port for 6.x will be required.

Dries’s picture

Dries CreditAttribution: Dries commented
Version: 7.x-dev » 6.x-dev
Status: Needs review » Fixed

Committed to CVS HEAD. Thanks solotandem. Moving this down to DRUPAL-6 so Gabor can consider to commit this as well.

gpk’s picture

gpk CreditAttribution: gpk commented
Status: Fixed » Reviewed & tested by the community

While this is fixed in 7.x I think it is still RTBC for 6.x.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Version: 7.x-dev » 6.x-dev
Status: Reviewed & tested by the community » Fixed

Committed to 6.x. Thanks.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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