The D6 version has been evolving more slowly than the D7 version, and has started to diverge (removal of the customizable collection name in watchdog).

IMHO it makes sense to just backport the current version to have a more usable base than our original D6 code.

CommentFileSizeAuthor
#1 backport_d7-1174142-1.patch48.39 KBfgm

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Needs review
StatusFileSize
new backport_d7-1174142-1.patch48.39 KB

Accordingly, here is a complete backport from mongodb and mongodb_watchdog from yesterday's dev versions, plus the patches I submitted in the meantime for the D7 version.

The simpletest does even not crash, although it is far from passing: reading it shows that it is essentially still the test for the dblog module, so much work remains to be done on it anyway.

If the 6.x version is to remain in its original, more limiting, version, this patch could be the basis for a 6.2 branch.

crea’s picture

crea commented

I vote for this. The more D6 version differs from the D7 one, the harder it will be to backport fixes.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Needs review » Fixed

OK, since nobody else commented and both chx and the butckst0p agreed offline, I just committed this version so we can be more in line with D7.

Status: Fixed » Closed (fixed)

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