Currently the per-node type variables are named 'pingback_[nodetype]'. This can create undesired conflicts as users could create a content type 'Mode' which would then override the existing pingback_mode variable.

My recommendation is to either merge all the node type variables into one pingback_node_types variable with an array of node types, or rename the variables to pingback_node_type_[nodetype].

Also discovered if users rename a node type's machine name, the pingback variable does not get renamed.

Comments

AlexisWilke’s picture

I think that this is a critical error, don't you think?