Status message to the user [#772492]

I have users who can't remember that they scheduled a node. A message confirming that they did so would be very nice.

        // Right before we save the node, we need to check if a "publish on" value has been set.
        // If it has been set, we want to make sure the node is unpublished since it will be published at a later date
        if (isset($node->publish_on) && $node->publish_on != '' && is_numeric($node->publish_on) && $node->publish_on > time()) {
          $node->status = 0;
          drupal_set_message(t('This post is unpublished and will be published @publish_time.', array('@publish_time' => date($date_format, $node->publish_on))), 'status', FALSE);
        }
        break;

That's at the end of hook_nodeapi('presave'...

Comment	File	Size	Author
#6	scheduler_module_1_50_2_37-0.txt	960 bytes	Eric-Alexander Schaefer

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 2 May 2010 at 18:58

Status:

Active

» Closed (won't fix)

I dont think we are going to include this.

Comment #2

NancyDru

she/her/hers

English

Boston

CreditAttribution: NancyDru commented 2 May 2010 at 23:54

Status:

Closed (won't fix)

» Active

How about an option? I've run into users all over the place who don't understand why their nodes aren't showing up right away. This simple change stopped most of our phone calls.

Comment #3

jonathan1055 CreditAttribution: jonathan1055 commented 8 June 2010 at 18:47

Hi Nancy,

I wanted the same thing, and achieved it by adding:

  if ($vars['unpublished'] && isset($vars['publish_on']) && $vars['publish_on']) {
    $vars['for_publishing'] = t('Created on @datetime for publishing at @publish_date',
                                array('@datetime' => format_date($vars['created'], 'custom', 'jS M Y'),
                                      '@publish_date' => format_date($vars['publish_on'], 'custom', 'H:i \o\n jS M Y')));
  }

into my theme's hook_preprocess_node(), and

  <?php if ($unpublished): ? >
    <div class="unpublished"><?php print t('Unpublished'); ? ></div>
    <div class="for_publishing"><?php print $for_publishing; ? ></div>
  <?php endif; ? >

into node.tpl.php.

I know you could have figured this out too, but just thought I'd share it, in case it is any help.

Jonathan

Comment #4

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 26 June 2010 at 20:50

I am curious. A user writes an article and enters a time for scheduled publishing and clicks "save". How can they forget between entering the time and klicking the save button? I don't get it.

Comment #5

NancyDru

she/her/hers

English

Boston

CreditAttribution: NancyDru commented 28 June 2010 at 14:16

How about they notice that some text in the body needs to be updated and by the time they get done, they have forgotten. My users do it all the time.

It can also happen when using Vertical Tabs, which I'm really getting to like, switching to another setting group, like comments, what they specified disappears; Yes, the tab says "scheduled for publishing", but they don't seem to notice that.

Comment #6

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 28 June 2010 at 17:32

Version:	6.x-1.6	» 6.x-1.x-dev
Status:	Active	» Needs review

File	Size
scheduler_module_1_50_2_37-0.txt	960 bytes

Here you go. Please review...

Comment #7

NancyDru

she/her/hers

English

Boston

CreditAttribution: NancyDru commented 29 June 2010 at 19:19

Looks good. I haven't tested yet.

BTW, here's what my own submit handler does:

  $msg = NULL;
  if (!empty($form_state['values']['publish_on'])) {
    $msg .= t('This post will be published on @publish_date.',
      array('@publish_date' => str_replace('-', 'at', format_date(strtotime($form_state['values']['publish_on']), 'large'))));
  }
  if (!empty($form_state['values']['unpublish_on'])) {
    $msg .= ($msg ? ' ' : NULL) . t('This post will be unpublished on @unpublish_date.',
      array('@unpublish_date' => str_replace('-', 'at', format_date(strtotime($form_state['values']['unpublish_on']), 'large'))));
  }
  if ($msg) {
    drupal_set_message($msg . ' ' . t('The time is approximate as Forrester checks the scheduling every 20 minutes.'));
  }

Comment #8

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 27 July 2010 at 19:20

Status:

Needs review

» Fixed

http://drupal.org/cvs?commit=398542

Comment #9

jonathan1055 CreditAttribution: jonathan1055 commented 28 July 2010 at 15:39

Nancy's code has the benefit of using Drupal's format_date() function, which takes care of the users selected timezone (if any) or the website's default timezone. The message which Eric's code creates just displays the time using date() which could confuse users in other timezones. Just pointing out and recording the difference, which could be addressed later if required.