Forum posts to 'bugs (temp)'

Re: Module Forum 0.1.2 needs MYSQL5

Helmut Zörrer — Wed, 07 May 2008 19:37:41 +1200

Hi Ingo,
Sorry to say, i only did a 'switch-back' to old functionality.
this is no 'patch'.
this means, patching with this code, would lead to
loosing the new abilitys, the new forum version has...

g
Helmut

Posted to: Module Forum 0.1.2 needs MYSQL5

Re: Module Forum 0.1.2 needs MYSQL5

Ingo Schommer — Wed, 07 May 2008 10:02:54 +1200

hey zyko - great to hear you made this functionality mysql4 compatible - can you provide your changes as a patch? http://open.silverstripe.com/#Submittingpatches
cheers

Posted to: Module Forum 0.1.2 needs MYSQL5

Re: Module Forum 0.1.2 needs MYSQL5

Helmut Zörrer — Wed, 07 May 2008 02:27:41 +1200

YEAH, to find something in Silverstripe forums
is sometimes hard.
Would be fine if there was only ONE result shown for a 'thread' if one post of the thread meets the query. and not any post of the thread, that meets the search criteria...

BTW: if anyone is interested:
I did a replacement of Forum.php function Topics()
with 'the old version' of this function and things are ok for now.
[code]
function Topics() {
if(Member::currentUser()==$this->Moderator()) {
return DataObject::get("Post", "ForumID = $this->ID and ParentID = 0 and (Status = 'Moderated' or Status = 'Awaiting')");
}
return DataObject::get("Post", "ForumID = $this->ID and ParentID = 0 and Status = 'Moderated'");
}
[/code]
the new function
[code]
function Topics() {
if(Member::currentUser()==$this->Moderator() && is_numeric($this->ID)) {
$statusFilter = "(`Post`.Status IN ('Moderated', 'Awaiting')";
} else {
$statusFilter = "`Post`.Status = 'Moderated'";
}

if(isset($_GET['start']) && is_numeric($_GET['start'])) $limit = Convert::raw2sql($_GET['start']) . ", 30";
else $limit = 30;

return DataObject::get("Post", "`Post`.ForumID = $this->ID and `Post`.ParentID = 0 and $statusFilter", "max(PostList.Created) DESC",
"INNER JOIN `Post` AS PostList ON PostList.TopicID = `Post`.TopicID", $limit
);
}
[/code]
seems to be better in showing the latest reply on articles in front of the thread, but it worked without this in former versions.
And maybe someone can do this correct for older my-sql versions in future versions...

Posted to: Module Forum 0.1.2 needs MYSQL5

Re: Module Forum 0.1.2 needs MYSQL5

Nick — Wed, 07 May 2008 02:00:58 +1200

I think sections would be a lot easier to search as well. So hard to find information at the moment.

Posted to: Module Forum 0.1.2 needs MYSQL5

Re: Module Forum 0.1.2 needs MYSQL5

Ingo Schommer — Sat, 29 Mar 2008 16:27:27 +1300

modules don't necessarily need to have matching requirements to the core, in the end its just a missing "minimum requirements" note for the forum module.

good point with your search for a "forum" keyword, this would be more efficient when limited to a specific "modules" forum - although we're trying to avoid a too fragmented forum structure. what do other people think?

Posted to: Module Forum 0.1.2 needs MYSQL5

Module Forum 0.1.2 needs MYSQL5

Helmut Zörrer — Fri, 21 Mar 2008 23:37:41 +1300

I did an Update from Forum 0.1.1 to the newer version.
I get Errors on MYSQL 4.1.22
that tell something abot 'invalid GROUP statements'.

The requiremnets for Silverstripe say MYSQL 4.1
so there should be no MYSQL5 specific code i think...

I didn't look at the details much more, am a novice in silverstripe.
maybe someone can check this?

BTW: would be fine to have seperate forums to discuss about things concerning
modules or themes.
Try to find something with a query about 'forum', and you see what i mean

*g
Helmut

Posted to: Module Forum 0.1.2 needs MYSQL5