Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
isset() returns true on success, not the value of the thing you're testing, so your edit of line 23 is unlikely to do what you intended. I'm also pretty sure that a debugged version of line 32 is needed, so commenting the bug away will probably also not work.
In any case, I was able to get it to work by redefining that function like so:
This at least got it to the point where I was getting modal forms when *creating* nodes of a type I specified via the ctools_automodal_admin module. However, I haven't been able to get it to work with editing the same nodes, or (what I really wanted) deleting them -- it's doing something funky with node permissions that's getting in the way of any sort of editing-related changes.
FWIW, this module is looking a little neglected in terms of update history, so I'm not sure what it's future is...
1. Main goal of this module is to provide UI interface for CTools Automodal module which it does. CTools Automodal module works only with menu item pathes which collected by hook_menu(). So it is possible to make all node page, node edit and node delete links regardless of node type to be automodal by putting
node/%node
node/%node/edit
node/%node/delete
into 'Extra modal links' field at 'admin/config/system/ctools_automodal_admin'. But there is no way to make link just for single node like 'node/5' (nor pathauto aliases for it) to become automodal. This is restriction of CTools Automodal module and we have no intention to change this in our module.
2. Checkboxes at 'admin/config/system/ctools_automodal_admin' only enable 'node/add/*' pages to be automodal. They do not affect node edit pages at all. As I mentioned you can make node edit page automodal by putting 'node/%node/edit' into 'Extra modal links' field at 'admin/config/system/ctools_automodal_admin'. We should clarify this in module description.
3. It seems source of notice in header is incorrect path entered into 'Extra modal links' field. We'll add some checks to make sure that every entered path has corresponding menu item for it. This should get rid of notice.
Comments
Comment #1
amocsy CreditAttribution: amocsy commentedhttp://localhost/mysite/content/test_page#overlay=admin/config/system/ct...
1. Clicked on a custom content to tick it.
2. Clicked on save
Of course after the Notice message in the topic, the page of that content type do not pop up in modal.
The path associated with that content type is like this:
http://localhost/mysite/home
And the page I tried to test is:
http://localhost/mysite/home/fooldal
Comment #2
jeremymcminn CreditAttribution: jeremymcminn commentedAlso getting this error message.
Comment #3
jeremymcminn CreditAttribution: jeremymcminn commentedChanged line 23 from
to
and commenting out line 32 removed the notices, however the edit link still doesn't open in a modal window.
Comment #4
jim_at_miramontes CreditAttribution: jim_at_miramontes commentedisset() returns true on success, not the value of the thing you're testing, so your edit of line 23 is unlikely to do what you intended. I'm also pretty sure that a debugged version of line 32 is needed, so commenting the bug away will probably also not work.
In any case, I was able to get it to work by redefining that function like so:
This at least got it to the point where I was getting modal forms when *creating* nodes of a type I specified via the ctools_automodal_admin module. However, I haven't been able to get it to work with editing the same nodes, or (what I really wanted) deleting them -- it's doing something funky with node permissions that's getting in the way of any sort of editing-related changes.
FWIW, this module is looking a little neglected in terms of update history, so I'm not sure what it's future is...
Comment #5
Leonid.Kuznetsov CreditAttribution: Leonid.Kuznetsov commented1. Main goal of this module is to provide UI interface for CTools Automodal module which it does. CTools Automodal module works only with menu item pathes which collected by hook_menu(). So it is possible to make all node page, node edit and node delete links regardless of node type to be automodal by putting
into 'Extra modal links' field at 'admin/config/system/ctools_automodal_admin'. But there is no way to make link just for single node like 'node/5' (nor pathauto aliases for it) to become automodal. This is restriction of CTools Automodal module and we have no intention to change this in our module.
2. Checkboxes at 'admin/config/system/ctools_automodal_admin' only enable 'node/add/*' pages to be automodal. They do not affect node edit pages at all. As I mentioned you can make node edit page automodal by putting 'node/%node/edit' into 'Extra modal links' field at 'admin/config/system/ctools_automodal_admin'. We should clarify this in module description.
3. It seems source of notice in header is incorrect path entered into 'Extra modal links' field. We'll add some checks to make sure that every entered path has corresponding menu item for it. This should get rid of notice.
Comment #6
Leonid.Kuznetsov CreditAttribution: Leonid.Kuznetsov commentedPlease upgrade module to a CTools Auto-modal Admin Interface 7.x-1.1 to get rid of notice.