FUDforum
Fast Uncompromising Discussions. FUDforum will get your users talking.

Home » FUDforum Development » Bug Reports » NNTP Sync minor quirks
Show: Today's Messages :: Polls :: Message Navigator
Switch to threaded view of this topic Create a new topic Submit Reply
NNTP Sync minor quirks [message #159768] Mon, 22 June 2009 01:46 Go to next message
jcarter is currently offline  jcarter   United States
Messages: 25
Registered: August 2006
Karma: 0
Junior Member
I recently upgraded to the latest version and have found several minor issues:

Version: 2.8.1

File: FUDForumData/include/nntp.inc

Around line 425:

if ( !strlen($msg_post->subject) ) {

The problem is that $msg_post->subject can be null which halts the script. I replaced it with:

if ( ($msg_post->subject == null) || !strlen($msg_post->subject) ) {


---------------------------------
Around line 455:

iconv_mime_encode("Subject:", $subject,

The problem is that the iconv_mime_encode function expects a keyword, and adds the colon, resulting in double colons and mangled subject thread titles. I changed it to :

iconv_mime_encode("Subject", $subject,

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

I posted these to {patches}.

Re: NNTP Sync minor quirks [message #159786 is a reply to message #159768] Wed, 24 June 2009 13:11 Go to previous message
naudefj is currently offline  naudefj   South Africa
Messages: 3771
Registered: December 2004
Karma: 28
Senior Member
Administrator
Core Developer
Thanks for the report and patch (see topic http://fudforum.org/forum/index.php?t=msg&goto=159785).
  Switch to threaded view of this topic Create a new topic Submit Reply
Previous Topic: Attachments in newsgroups aren't decoded
Next Topic: why i cannot add a new Category of Forum
Goto Forum:
  

-=] Back to Top [=-
[ Syndicate this forum (XML) ] [ RSS ]

Current Time: Sat Nov 30 01:37:14 GMT 2024

Total time taken to generate the page: 0.01951 seconds