Hi,
I migrated nodes from D5 to D6. In the D6 version of my nodes I all the time get this error:

warning: Missing argument 2 for _nodecomment_node_add_access() in /sites/all/modules/nodecomment/nodecomment.module on line 905.

How can I get rid of this error please?

Thanks a lot in advance for your reply!
greetings, Martijn

Comments

crea’s picture

crea commented
Status: Active » Postponed (maintainer needs more info)

Please post detailed instructions to reproduce

crea’s picture

crea commented
Status: Postponed (maintainer needs more info) » Closed (won't fix)

Well, tbh, I can't support data migrations. I never used D5 version, and D6 1.x branch was totally different.
So, unless you provide a patch fixing this stuff, I think it will remain as is.

summit’s picture

summit commented

hI Andrey, s orry, I do not have time until after the summer.
I will investigate it more then hopefully. Thanks for the update.
greetings, Martijn

broncomania’s picture

broncomania commented

I get the same error without migration. I just switch from normal drupal forum comments to node comments. I did this on two forums. The first works as expected nice without any problems but the second forum throws now this error. I am really wondering because I made the same steps but maybe it is caused by other modules. What will be the best way to solve this issue?

broncomania’s picture

broncomania commented
Status: Closed (won't fix) » Active
broncomania’s picture

broncomania commented

Look here this solves the problem for me http://drupal.org/node/1249196#comment-5313854

dunx’s picture

dunx commented

I get this error when I installed the edge-usage-case module domain_content_types

Two parameters expected in _nodecomment_node_add_access($op,$type) and the only variable that's being passed in is my content type. Not idea whether this is an error in nodecomment or domain_content_types, but thought I'd post it here.

I'll follow up if I find anything else out.

UPDATE: Sounds a bit drastic, but I just changed the function to accept just one parameter; $type! Everything still seems to be working as expected... so far.

crea’s picture

crea commented
Status: Active » Closed (won't fix)

2.x is unlikely to get any support at this point. Closing all related issues.