Posted by JamesAn on June 12, 2009 at 3:05am
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | node.module |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
#361277: Remove the 'post settings' admin screen and relocate contents dispersed the settings found in "Post settings" (admin/content/node-settings) to other forms and moved admin/content/node-settings/rebuild to admin/reports/status/rebuild. Yet, there remains residual references to admin/content/node-settings and admin/content/node-settings/rebuild that need to be removed or changed.
Also, node_configure() should be removed since it builds the now-defunct "Post settings" form, as well as its submit callback function, node_configure_access_submit().
Comments
#1
Ok, of the 3 settings in "Post settings" form, two were moved to the "Content type" config forms and one was moved to the "Site information" form.
This patch modifies the 4 references to admin/content/node-settings and admin/content/node-settings/rebuild in node_help(). It also removes node_configure and node_configure_access_submit, which were orphaned by #361277: Remove the 'post settings' admin screen and relocate contents, and changes the cancel path in the node access permissions rebuild confirm form from admin/content/node-settings, which no longer exists, to admin/reports/status.
#2
The last submitted patch failed testing.
#3
Patch looks great to me. I was just wasting time working on that node_configure function for another issue when I realized nothing even calls it anymore...
#4
The last submitted patch failed testing.
#5
It looks like half of this already went in. The remaining bits are here.
#6
Committed to CVS HEAD. Thanks!
#7
Automatically closed -- issue fixed for 2 weeks with no activity.