kareha's rss feeds are not valid rss (7)

1 Name: hotaru!hoTarufiRE!!H0csnvz2 2005-01-30 00:25 ID:C6GtE3/N (Replies) [Del]

http://feedvalidator.org/check.cgi?url=http%3A%2F%2Fwakaba.c3.cx%2Fsup%2Findex.rss
the "Missing channel element: description" problem is easy to fix, but i'm not sure how to fix the "author must include an email address" problem...

2 Name: Squeeks!!zsavMW2Z 2005-01-30 02:31 ID:Heaven (Replies) [Del]

Easy!

If the link/email field is empty, or does not contain what looks like a valid email address, is just uses the value <data>@<boardname>.

Otherwise, if it detects a legitmate email address (HAH), it uses that.

Well, thats my cheap idea towards it anyway.

3 Name: Squeeks!!zsavMW2Z 2005-01-30 02:46 ID:Heaven (Replies) [Del]

My idea in >>2 is only for the RSS template, and not for the XHTML document. I'll try to quickly cook up some coding ideas to make my idea work.

4 Name: Squeeks!!zsavMW2Z 2005-01-30 03:16 ID:Heaven (Replies) [Del]

Attempt #1 at bad code. At least this might give you an idea to work with.

templates.pl (use constant RSS_TEMPLATE, line ~458)

Replace:

<author><var $author></author>

With:

<author><var $author> - <var $rss_email></author>

kareha.pl

This is just stuff to add in.

# Insert above line 308

if($email='';)
{
my $rss_email ='$TITLE @ $ENV{SERVER_NAME}';
}
elsif($email!='';)
{
my $rss_email ='&lt; $email &gt;';
}

# Insert after line 316

$rss_email=clean_string($rss_email);

5 Name: !WAHa.06x36 2005-01-30 08:11 ID:vUgA4LU8 (Replies) [Del]

"Author must include an email address"? What the hell kind of retarded requirement is THAT? No, really, what the HELL?

I added the lacking description, but I really don't feel like doing anything about the <author> element.

6 Name: Anonymous 2005-01-30 12:32 ID:Heaven (Replies) [Del]

just have it default to [email protected] or something.

7 Name: 6 2005-01-30 12:54 ID:Heaven (Replies) [Del]

Actually, no, you're right, it's retarded. Don't bother.

Name: Link:
Leave these fields empty (spam trap):
More options...
Verification: