Newsletter Plugin v3.02 - Additional Functionality

Try changing the 'where' line to: 

'where'    => "to_list = '" . coalesce(mysql_escape(@$_REQUEST['to_list']), '2') . "' AND " . $whereMessageNumIs,

Daryl:  After a bit more testing I have been able to display what I originally thought was the 1st message.... its not, its the template!  Nothing in my code should be displaying the template alone (see screen shot).  It still only displays 1 message, but shows nothing in dropdown.

Here is my code:

<?php include 'common/includes/php_header.inc'; ?>
<?php
  // error checking
  if (!@$GLOBALS['NEWSLETTER_BUILDER_PLUGIN']) { die("You must activate the newsletter plugin to see this page."); }

  // load newsletters settings and archives
  global $newsletterSettings;
  $newsletterSettings  = mysql_get('_nlb_settings', 1);
  $archivedNewsletters = mysql_select('_nlb_messages', "send IN ('all','archive') ORDER BY sent_date DESC");

  // load newsletter
    $whereMessageNumIs = 'TRUE';

if (@$_REQUEST['message_num']){
 $whereMessageNumIs = " num = '".mysql_escape($_REQUEST['message_num'])."'";
}

list($_nlb_messages, $_nlb_messagesMetaData) = getRecords(array(
  'tableName'   => '_nlb_messages',
  'loadUploads' => true,
  'allowSearch' => false,
  'where'    => "to_list = '" . coalesce(mysql_escape(@$_REQUEST['to_list']), '2') . "' AND " . $whereMessageNumIs,
//    'debugSql'            => true,
));

  // replace placeholders
  $archivedNewsletters = nlb_viewers_replacePlaceholders($archivedNewsletters);

?>


<table cellspacing="2" cellpadding="4">
 <tr>
  <td valign="top">

<?php if ($_nlb_messages): ?>
 <select name="message_num">
   <option value="">--</option>
   <?php foreach($_nlb_messages as $message): ?>
     <option value="<?php echo htmlencode($message['num']); ?>" <?php selectedIf($message['num'], @$_REQUEST['message_num']); ?>
       <?php echo htmlencode($message['subject']); ?>
     </option>
   <?php endforeach; ?>
 </select>
<?php endif; ?>
</td>
  <td valign="top">

    <b><?php echo htmlencode($message['subject']) ?></b><br/>
    <?php echo $message['html']; ?><br/>

  </td>

</tr>
</table>

Hi Ragi,

You're missing a ">" on your <option> tag:

<option value="<?php echo htmlencode($message['num']); ?>" <?php selectedIf($message['num'], @$_REQUEST['message_num']); ?>>

For displaying the messages, I'd check the values of $_nlb_messages first to confirm if it has any records in it and if the where clause that we added is correct by adding this code after the getRecords() function:

showme($_nlb_messages);

 And then I'd check if I'm looping through $_nlb_messages and display each records - pulling values from the correct array keys. The dropdown fields should take care of the filtering since you've already added the field and the where clause for '_nlb_messages'

Cheers,

Daryl:  Thanks, that worked out great!  I do have a couple of questions for you:

1.  I have hard coded the "Weekly Messages" list to show in the file weeklyMessage.php.  However, it always displays the first message in the list, not the latest.  How can I get it to always display the latest first?

2.  For some reason the "Reset" button doesn't seem to work.  On the browser (Chrome), the button border turns blue when I click it (and stays that way), but no reset occurs.  Any ideas?

Thanks!

Ragi

Hi Ragi,

> 1.  I have hard coded the "Weekly Messages" list to show in the file weeklyMessage.php.  However, it always displays the first message in the list, not the latest.  How can I get it to always display the latest first?

- You can use orderBy to sort the records, for example:

  list($_nlb_messagesRecords, $_nlb_messagesMetaData) = getRecords(array(
    'tableName'   => '_nlb_messages',
    'orderBy'     => 'sent_date DESC',
  ));

> 2.  For some reason the "Reset" button doesn't seem to work.  On the browser (Chrome), the button border turns blue when I click it (and stays that way), but no reset occurs.  Any ideas?

- Can you send me a link so I can take a look?

Thanks,

Hi Daryl

i should have thought of the orderBy!

you can se the reset on the http://706aircadets.ca/weeklyMessage.php

I see, are you tying to reset the selected record on the right side of the page and the selected value of the drop down field?

The reset button only sets the form values back to it's previous values. It will not "reset" the record loaded on the page.
One way to "reset" the loaded record on the page is to add a redirect to the same page without the "?message_num=20". For example:

<input type="reset" value="Reset" onclick="location.href = '?';">

Since your form method is "GET", removing the "message_num" in the URL will deselect any messages.

Thanks Daryl!