[Phpslash-trackers] [ phpslash-Patches-590927 ] tested newer better cronmail.php3

[<no...@so...>]

Patches item #590927, was opened at 2002-08-04 20:37
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=310566&aid=590927&group_id=10566

Category: None
Group: None
>Status: Closed
Resolution: Accepted
Priority: 5
Submitted By: Luis M (lems1)
Assigned to: Nobody/Anonymous (nobody)
Summary: tested newer better cronmail.php3

Initial Comment:
Hello all,

I just finished checking out the latest phpslash-ft and
got surprised to see that my cronmail.php3 has been
incorporated. I got surprised also on how bad that code
was :-)
I have been using this version of the cronmail.php3
script for quite a while on latinomixed.com. It seems
to work perfectly.

Features:
- let's you use 'fortunes' to get quotes and send those
together in the regular emails (can be commented out
with no loss of functionality to the script :-) )
- better explanation on how to setup the cron job right
on top of the script.... needed for newbies and for
inclusion in the documentation
- cleaner script (cleanup a lot of odd stuff ... vim is
great ain't it?)


Bug fixes:
- Missing day got fixed. Not sure if Blake intended to
miss Tuesday, but for me that's just another regular
day. So I included Tuesday and moved the other days by
15 minutes in the schedule
- removed (commented out) the HTML part of the script
(have never used HTML newsletter format and don't think
I will; if somebody wants to test that out and add it
in a way that it can be turned off, go ahead and I will
sync mine).
- script does not continue sendmail emails if there is
no news to send! Fixed those ugly blank messages when
there is no news to send to the users... hey, some
sites are busier than others
- fixed the links sent in the message. they were
showing without the forward slash before the
"article.php3" script... thus, if you clicked on it,
nothing would happen... ugly bug! bad bug!

The attached file (tar.gz) has both a patch (.diff)
that you can apply to the actual version, and a
complete version of it (if don't feel like patching).

Enjoy


----------------------------------------------------------------------

Comment By: Joe Stewart (joestewart)
Date: 2002-08-13 09:44

Message:
Logged In: YES 
user_id=77269

This patch has been applied to the CVS.  Please update your
copy of PHPSlash, or wait for the next release.

----------------------------------------------------------------------

Comment By: Joe Stewart (joestewart)
Date: 2002-08-13 09:43

Message:
Logged In: YES 
user_id=77269

Committed with the only changes of putting the contents of
the alter_mailinglist.sql and header.txt in the header of
the script.

Hope this was OK.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=310566&aid=590927&group_id=10566