From the Drupal Docs:

In addition, developers using submit functions to persist data to the database are encouraged to return meaningful data about the results of the operation (ids of newly created nodes, etc.) in the $form_state. This allows developers calling forms using drupal_execute() to properly handle to the results of a form's successful submission.

I've attached a very small and simple patch to help do that for the Custom Pages section. The new page id should be passed on, successfully, to the next piece of code seeing the $form_state.

Since it's so simple and (I'd think) non-controversial, I'm marking as RTBC.

Comments

Status: Reviewed & tested by the community » Needs work

The last submitted patch, nodewords_custom_pages.admin_.inc_.diff, failed testing.

dave reid’s picture

Status: Needs work » Fixed

Seems great to me. Tested that it saves correctly and committed to Git.
http://drupalcode.org/project/nodewords.git/commit/b978d51

Status: Fixed » Closed (fixed)

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

sbandyopadhyay’s picture

Thanks Dave!

damienmckenna’s picture

Status: Closed (fixed) » Needs work
Issue tags: +v6.x-1.12 blocker

Needs to be re-rolled for 6.x-1.x.

damienmckenna’s picture

Status: Needs work » Fixed
StatusFileSize
new423 bytes

Re-rolled and committed.

Status: Fixed » Closed (fixed)
Issue tags: -v6.x-1.12 blocker

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