Forum Made Simple 0.7

[styson]

Attention: I see a few little bugs in current version
In my roadmap I have a new version this Sunday.

Excellent.  Is this new version going to incorporate my request or are you telling me to wait until the new version before I make changes? 

[alby]

Attention: I see a few little bugs in current version
In my roadmap I have a new version this Sunday.

Excellent.  Is this new version going to incorporate my request or are you telling me to wait until the new version before I make changes? 
[/quote]

For lack of time I haven't time for check methods in FEU (I must see changed in 1.2 for Multilingual version  ).
Or if you search in FEU for return a define property from id_user I try to apply in new version

Alby

[styson]

If you use GetUserProperties($frontenduserid), this will return false or an array of all the properites set for that userid.  The array will contain:

id - record id field
userid - FEU id. 
title - name of the property.  This will be the field to filter on.
data - value of the property.  This is what is to be displayed in stead of the username. 

[alby]

If you use GetUserProperties($frontenduserid), this will return false or an array of all the properites set for that userid.

Ok, in attach there is a new version but I rewrote many parts of module that I consider a pre0.8.
Please test this and report bugs here. I want release a new version in two days and report to admin project for include in forge.

CHANGED FROM Forum Made Simple 0.7:
- Add alternative color in CSS style
- Add topic per page (default 999)
- Add post per page (default 999)
- Add paginations for topic and post
- Add check user is allowed to access
- Add action.create_topic.php, action.create_post.php, action.edit_topic.php and action.delete.php
- Add action.admin_create_forum.php, action.admin_save_settings.php, action.admin_delete_forum.php
- Add property_feu param for other display info from FEU (if it's not set default is loginname). AT THIS MOMENT, NOT WORK IN PRETTY URL!

- Rewrote module actions for reduce memory footprint
- Changed Minimum CMS Version and added support for the 1.1 series
- Correct prettyurl for Search module

[REMOVED ATTACH - DOWNLOAD FROM FORGE]

Alby

[styson]

I will test away. Thank you!

[styson]

Download isn't working. 

[alby]

Download isn't working.

Forum not permitted attach zip file.
Save download and rename mpg to zip

Alby

[tamlyn]

Excellent stuff alby! I've added you as a project admin

However this bit of code in each action:

Code: Select all

if(! $this->CheckForumPermission('default'))
	return $this->FrontEndDisplayMessage($id, &$params, $returnid, $this->Lang('forum_access'));

raises a 'calltime pass-by-reference' warning on every page. I think you need to remove the ampersand before the $params variable.

Keep up the great work.

[alby]

Excellent stuff alby! I've added you as a project admin

Great! 

However this bit of code in each action:

Code: Select all

if(! $this->CheckForumPermission('default'))
	return $this->FrontEndDisplayMessage($id, &$params, $returnid, $this->Lang('forum_access'));

raises a 'calltime pass-by-reference' warning on every page. I think you need to remove the ampersand before the $params variable.

Thanks, I forgot to enable error_reporting
I do in this attach (I think last release in forum) but I have not a feedback from users (styson?) 


I have not some time now because I want release MLE 1.2 but I hope:
- Forum parameter for display this forum only
- Templates in DB
- Finest permission per forum (new topic, new post, ...)


[REMOVED ATTACH - DOWNLOAD FROM FORGE]

Alby

[styson]

Sorry for no feedback  I have to get my client's site setup on a virtual machine, upgrade to 1.1.3.1 then test out the release.  I'm working on it!!!! 

[styson]

Great Work Alby.    8)    I setup two pages on my test site, one with the forum module loaded with the property_feu set  the other one without it set.  I logged into the FEU, made a post using both forum pages, then reviewed the posts from both forum pages.  Results worked as expected.  The forum with the property set displayed tjhe property value not the FEU userid and the forum without the property set displayed the userid.   

Well Done mate!!!

[alby]

Great Work Alby.        I setup two pages on my test site, one with the forum module loaded with the property_feu set  the other one without it set.  I logged into the FEU, made a post using both forum pages, then reviewed the posts from both forum pages.  Results worked as expected.  The forum with the property set displayed tjhe property value not the FEU userid and the forum without the property set displayed the userid. 

Well Done mate!!!

Ok, if I have not error feedback, tomorrow publish 0.8 in forge

Alby

[qido]

This is a very nice module for CMS, keep up the good work.

Have set the new 0.8 up on a test site, works great.

1) Can I call multible properties from FEU?  Like {cms_module module="Forum" property_feu="surname,firstname"} Tried with different signs but no success.

2) If I reply/edit a post I get directed back to the overview of the different topics and not to the topic page I made a reply to or edited. This is probeably my own personal taste of where it should direct my to.

3) If I am in a topic/forum and I want to log out, FEU do not directly log me out (Have a front end user login box in buttom of my page). I have to press log out two times, the first time I get directed back to the index page of the forum, second time I am logged out. I use CMS 1.1.3.1, feu 1.2.

4) Can there be added the total amount of post and last post for each forum to the index overview page. (just a small request, then I do not need to change the original code  )

Again a very nice module for CMS,

qido

[alby]

Have set the new 0.8 up on a test site, works great.

Thanks

1) Can I call multible properties from FEU?  Like {cms_module module="Forum" property_feu="surname,firstname"} Tried with different signs but no success.

No, but I think this for 0.9
The problem is the impossible to use them with prettyurl.
URL must be (orrible): www.mysite.com/forum/topic/1/1/surname,firstname/43
Your opinion on this?

2) If I reply/edit a post I get directed back to the overview of the different topics and not to the topic page I made a reply to or edited. This is probeably my own personal taste of where it should direct my to.

I thought on this, the redirect is to forum level (list of topics).
IMHO:
- for reply is correct. The update topic is first of list, like in this forum
- for edit post (modify), this forum redirect to topic level (list of posts). I redirect to forum level for make easier
Your opinion on this also?

3) If I am in a topic/forum and I want to log out, FEU do not directly log me out (Have a front end user login box in buttom of my page). I have to press log out two times, the first time I get directed back to the index page of the forum, second time I am logged out. I use CMS 1.1.3.1, feu 1.2.

Strange tomorrow I test this.
I think to normal behavior of FEU. If you want to log out FEU redirect in same page, if in this page there is CustomContent (I have this) display Access denied message.
If you don't have CC, you must set a logout alias page in FEU.
But your opinion on this also?

4) Can there be added the total amount of post and last post for each forum to the index overview page. (just a small request, then I do not need to change the original code  )

Ok, write in my todo list 

Alby

[bterkuile]

about the property_feu="surname,firstname" setting and pretty urls. Isn't it an idea to store this setting in the database?
something like:

Code: Select all

$this->setParameter('property_feu',$params['property_feu']);

Plus all kinds of checks if the property is set etc.
Then it does not need to be passed as get parameter