_simplenews_scheduler_new_edition() does everything for creating a new edition of a newsletter... except actually recording it in the {simplenews_scheduler_editions} table!

This makes it rather awkward to test the module, as using _simplenews_scheduler_new_edition() doesn't connect the new node up with the original.

Comments

joachim’s picture

Status: Active » Needs review
StatusFileSize
new1.55 KB

Here's a patch.

joachim’s picture

Status: Needs review » Fixed
StatusFileSize
new1.45 KB

Committed this rerolled patch.

joachim’s picture

Status: Fixed » Patch (to be ported)

Possibly needs porting to D7.

joachim’s picture

Version: 6.x-2.x-dev » 7.x-1.x-dev
Status: Patch (to be ported) » Closed (duplicate)

This seems to be taken care of here in D7:

  // Insert edition data.
  $values = array(
    'eid' => $edition_node->nid,
    'pid' => $template_node->nid,
    'date_issued' => $edition_time,
  );
  db_insert('simplenews_scheduler_editions')
    ->fields($values)
    ->execute();
berdir’s picture

Status: Closed (duplicate) » Patch (to be ported)

Will check.

berdir’s picture

Status: Patch (to be ported) » Closed (duplicate)

Oh, double post.