Do not save draft for empty forms and delete empty drafts during cron run [#1460360]

Delete empty drafts during cron run
If the data, title and body field is null or empty the draft should be deleted, even if the delete draft option is turned off. These are junks and not needed. Moreover, having a button in the settings page to do the same would be great!

Thanks for the great module!

Comment	File	Size	Author
#6	delete-empty-1460360.patch	3.86 KB	udvranto

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

udvranto CreditAttribution: udvranto commented 28 February 2012 at 20:39

Removed

Comment #2

udvranto CreditAttribution: udvranto commented 28 February 2012 at 17:34

2. Delete empty drafts during cron run
Adding the following line in draft.module at the end of function draft_cron will do it. It will be great if anyone can confirm.

  // Remove empty drafts
  db_query("DELETE FROM {drafts} WHERE title IS NULL AND body IS NULL AND data IS NULL");

Comment #3

udvranto CreditAttribution: udvranto commented 28 February 2012 at 19:06

2. Delete empty drafts during cron run
I added an option as well.

draft.admin.inc

  $form['draft']['draft_cron_remove_timeout'] = array(
    '#type' => 'textfield',
    '#title' => t('Amount of time (Days) a draft can stay in the system without being updated'),
    '#default_value' => variable_get('draft_cron_remove_timeout', 1),
    '#description' => t('When a draft is originally created if the cron timeout is never set to 1 then it has never been updated so we need to allow these to remain for a little amount of time before they are removed'),
    '#size' => 6,
  );
  $form['draft']['draft_cron_remove_empty'] = array(
    '#type' => 'checkbox',
    '#title' => t('Remove empty drafts even if the cron removal is disabled'),
    '#default_value' => variable_get('draft_cron_remove_empty', 0),
    '#description' => t('If checked, cron will remove empty drafts even if the cron removal is disabled'),
  );

draft.module at the end of function draft_cron

    db_query("DELETE FROM {drafts} WHERE cron_remove = 0 AND updated <= %d", $updated);
  }
  // Remove empty drafts
  if (variable_get('draft_cron_remove_empty', 0) == 1)
    db_query("DELETE FROM {drafts} WHERE title IS NULL AND body IS NULL AND data IS NULL");
}