Hi,

While working on a project of mine, I came across the requirement to "record" activity related to only certain selected nodetypes. The best way I figured out was to provide admin users a settings page where the required nodetypes can be selected. I have attached a patch for the same.

The patch essentially adds settings page for the nodeactivity module, and changes the hook_nodeapi() implementation to do a check if the activity for current nodetype is configured to be recorded in the database.

I have tested the code and it is working on my setup, hence marking the issue as PCNR though it could also be PCNW as am not very confident about the quality of the code :P

CommentFileSizeAuthor
nodeactivity.patch2.73 KBsumeet.pareek

Comments

jaydub’s picture

jaydub commented

This feature is already a part of the 5.4 and 6.1 branches of Activity. The settings page for any one particular Activity contrib module allows one to enable or disable activity 'tokens' or 'operations' which in the case of Node corresponds to node types and create/update/delete.

If you have to stick with 5.3 for a production site then you'll be fine with your patch but if you have time before you have to deploy the 5.4/6.1 branches are getting all of the new features and fixes at this point.

sumeet.pareek’s picture

sumeet.pareek commented

I am currently using the 5.3 version on the production site so will have to stick with my patch for sometime. But I will check out the 5.4 version and see if I can help with any of the pending tasks.
Will switch the production site to use the 5.4 version once it is stable and I get a hang of it :)

jaydub’s picture

jaydub commented
Status: Needs review » Fixed

This is addressed in the new branches.

Status: Fixed » Closed (fixed)

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