Hi,

Following #1293588: Using the body of the site content type, I decided to use comments on some node types in aegir, like "site".

But I was blocked when comments where disabled on a node and I was not able to activate them back using comment settings usually in the form "node/1234/edit".

ergonlogic informed me this is due to the function hosting_form_alter:
http://api.aegirproject.org/api/Hostmaster/modules--hosting--hosting.mod...

Because comments can be used in aegir, I suggest we give the option to modify their settings by removing this line:

unset($form['comment_settings']);

Are you agree?
Do you want me to provide you a patch?

Comments

anarcat’s picture

Status: Active » Postponed (maintainer needs more info)

I don't see why we should allow per-node settings like this, really. It will just clutter the interface. We can, however, enable the comments on a sitewide basis, through admin/content/node-type/site...

Would that be sufficient for you?

j0nathan’s picture

Status: Postponed (maintainer needs more info) » Active

That will not resolve the issue I faced when comments are already disabled on a given node and I have to enable it back on that node only. I face the issue where comments are disabled per-node, not sitewide.

ergonlogic’s picture

Status: Active » Closed (won't fix)

Comments can be re-enabled simply enough with something like:

function site_comments_form_alter($form, $form_state, $form_id) {
  if ($form_id == 'site_node_form') {
    $form['comment_settings']['#access'] = TRUE;
  }
}

This might be a good candidate for a contrib module, as I can see some utility to using comments as a log of sorts.