Hello,

When I install this module on D7 I can't really use it because when I click edit on any content type I get internal server error.

Comments

deaftone’s picture

Title: can't edit content type » can't edit content type (D7)
lelizondo’s picture

Can you post your apache / php logs please for when this error occurs.

osopolar’s picture

Not sure but I guess I have the same problem. When I edit the content type I get:

( ! ) Fatal error: Call to undefined function ctools_dependent_process() in /path/to/drupal/includes/form.inc on line 1724

Without checking for ctools you set '#process' => array('ctools_dependent_process') in settings_node.inc: _nodeformsettings_settings_form()

  $form['nodeformsettings']['nfs_cancel']['nfs_cancel_behaviour'] = array(
    '#title' => t("Cancel Button Behaviour"),
    '#type' => 'select',
    '#options' => array(
      0 => t("Back one page using Javascript"),
      1 => t("Using previous destination"),
    ),
    '#default_value' => isset($settings['nfs_cancel']['nfs_cancel_behaviour']) ? $settings['nfs_cancel']['nfs_cancel_behaviour'] : $defaults['nfs_cancel']['nfs_cancel_behaviour'],
    '#dependency' => array('radio:nfs_cancel[nfs_cancel_status]' => array(0)),
    '#process' => array('ctools_dependent_process'),
  );

BTW: I had installed ctools before, but I uninstalled it later ... now I installed it again, as a workaround for this issue.

lelizondo’s picture

Title: can't edit content type (D7) » Make this module not Ctools dependent
Category: bug » feature

Well, this is not a bug then, more like a Feature request to make this module not dependent on Ctools. Anyway, at least in my sites, Ctools is becoming a huge dependency since now Views and many others are Ctools dependent.

osopolar’s picture

At the moment it's a bug, because the module is not working as expected. You may solve this by declaring this module explicitly ctools dependent. And another task would actually be: "Make this module not Ctools dependent". But you decide, you are the maintainer.

lelizondo’s picture

Status: Active » Fixed

Ctools is needed in a very small configuration thing. Is not really worth it to make it a dependency. If Ctools is not installed, the only thing that is going to happen is that a small setting will not be dependent on another and it will not show/hide.

I just committed and pushed to 7.x-2.x

lelizondo’s picture

Also commited and pushed to 6.x-2.x

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