Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
aggregator.module
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
23 Aug 2008 at 09:05 UTC
Updated:
2 Jan 2014 at 23:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
mustafau commentedCritical because aggregator is not working without this.
Comment #2
ztyx commentedI also experienced this bug. Drupal crashed when running cron (see attached file for screenshot).
I can also confirm that this patch fix the problem.
Comment #3
Anonymous (not verified) commentedHmm... I think it is a duplicate of #201427: Field 'description' doesn't have a default value query?
Comment #4
ztyx commentedNo, it's not. The error message they are talking about in #201427: Field 'description' doesn't have a default value query is very different from the one we are getting. Infact we are getting a crash while they seem to be getting an warning. This bug makes Drupal unusable while the other one seem to still make it usable. Also, this bug is triggered when cron is run, and theirs is triggered when adding a feed.
Comment #5
damien tournoud commentedThis error appeared when we switched to MySQL strict mode.
The patch itself is good.
No specific tests are needed: #243532: Catch notices, warnings, errors and fatal errors from the tested side catches that error.
Comment #6
dries commentedThere should be a SimpleTest for this, shouldn't there? It is odd that this isn't catched by the existing tests. Let's make sure to correct that.
Comment #7
Anonymous (not verified) commented@Dries: I thought the same thing but then I noticed Damien's last sentence in #5.
Comment #8
ztyx commentedAre there tests just doing a plain cron run? This error would have been caught in such a test.
Comment #9
chx commentedIs this still an issue? I am marking novice in hope someone checks whether it is. Fixing might not be a novice thing.
Comment #10
mustafau commentedThis was fixed by another patch.