Sorry - a bit of syntax punctuation error on the above change to the search.inc.php - one of the quotes should have been a double quote.

this has been corrected here -

@DobePhat
Many thx for reporting this issue here.

I too upgraded News (excellent module) but hadn't noticed the expired article problem.

News has always considered 'Archive' as a collection of all articles. My definition of archive is a collection of old (expired) articles. This version of News considers archive as a collection of anything BUT expired articles. It's about as logical as a Citroen!

@Skenow
Many thx for the fixes.

I am particularly pleased because, if like me you only want the archive to contain expired articles, you can slightly modify your suggestion for news/class/class.newsstory.php to:


Search doesn't work for me though. I'm not sure if News ever searched the archive?

Man, I have been playing with too many files - I forgot the site I was looking at didn't want expired articles to be included in the search results, but did want them in the archive.

So, to include expired articles in the search results, use this statement -



I have verified this works!

Quote:
Indeed it does!

Quote:
I agree. Thx to this thread at least you can search for 'old' news, but it would be good if the search results indicated an article is from the archive.

Thanks!
Didnt realize it was just such a simple adjustment.

Good to know, I gather this is still kind of a hack for the news modules now, since the default behvior is to not have them show up in a search right?

Thanksagain though, now the older/expired articles are not just viewable but showing up in our site search.