Closed (fixed)
Project:
Content Profile
Version:
6.x-1.x-dev
Component:
User registration module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Sep 2008 at 22:56 UTC
Updated:
14 Jul 2012 at 23:25 UTC
Jump to comment: Most recent file
Comments
Comment #1
fagoI suppose the call to watchdog() is wrong.
Comment #2
kenorb commentedfollowed watchdog call should be converted to Drupal 6.x syntax
Comment #3
kenorb commentedcomment line with the watchdog() call, or replace with:
This should work on both version of Drupal (5 and 6).
Comment #4
niklp commentedCan you make a patch for that please...? :)
Comment #5
kenorb commentedIn attachment, it's for development version.
If it will not work, download whole file here: http://drupal.org/node/313341#comment-1050831 with some additional patches.
Comment #6
david lesieur commentedIt is not necessary to support both 5.x and 6.x — this module only targets 6.x.
Comment #7
kenorb commentedSometimes people wants to back port some modules to D 5.x, so they will see the different and they will have less mistakes and converting:)
Any way you can use only this line:
for Drupal 6.x only
Comment #8
niklp commentedIn which case... can you please post a patch that fixes the module for D6 only, because it's really irrelevant to tackle anything related to D5.
Comment #9
kenorb commentedPut the NULL argument before WATCHDOG_NOTICE argument. And that's it.
It will take less time than creating and applying the patch.
At the moment I've patched half of the module, so I can't generate it at the moment.
Comment #10
kenorb commentedAnother change
There shouldn't be any t() function
http://drupal.org/node/114774#watchdog_parameters
So it should looks like:
Comment #11
jgraham commentedthanks kenorb