PostApproval plugin released!

jspautsch

This discussion is related to the PostApproval addon.

Finally released my PostApproval plugin to the Addons page here at vanillaforums.org, I was waiting until I could figure out how to make it work without manual changes to the core code, and the solution was a lot easier than I thought. The plugin includes the modified files for the comments, discussions, and vanillasearch models. It's not the greatest solution but at least it works and shouldn't permanently bork your forums if something goes wrong.

Let me know how it performs and if you notice any glaring bugs. I've released it as a beta for now as I didn't have time to do a lot of testing.

422

Will check this out today, good onya Jon

Mathias

Was looking for this. Thank you!

Mupetz

Nice plugin. Well done @jspautsch

konpaevents

After i installed the plug in my board crashed...now i can't restore it back to normal....Can someone please help me out

konpaevents.com/newsboard.html#/

Thanks

jspautsch

What version of Vanilla do you use?

konpaevents

2-0-18-1

konpaevents

Now i removed the postapproval-plugin - i got this error message > Fatal error: Call to undefined method Gdn::applicationmanager() in /home/konpaeve/public_html/newsboard/index.php on line 48>

jspautsch

You removed the plugin files without disabling it via the dashboard first you mean?

UnderDog

... and now he's crossposting... @konpaevents first read this thread: http://vanillaforums.org/discussion/17954/food-for-thought-forum-etiquette Set it as your homepage. For every action you take on this forum, first read the etiquette thread and then see if you still want to take that action.

Now for your solution. How did you disable the PostApproval plugin?

Consider re-installing your forum, making a backup, installing the plugins 1 by 1 and making a backup after you've installed each plugin.

jspautsch

The original error you got was probably a conflict with another plugin, and the second sounds like you deleted the PostApproval plugin files without first disabling the plugin.

Anonymoose

Very nice!

Lincoln

@jspautsch Nice plugin. I highly recommend using CommentModel_FunctionToOverride_Create()-style methods in your main plugin file though rather than packaging modified copies of the Comment, Discussions, and Search models. That's going to make this a headache to maintain. I can't deploy it with that broad of overrides in it.

jspautsch

Lincoln said: @jspautsch Nice plugin. I highly recommend using CommentModel_FunctionToOverride_Create()-style methods in your main plugin file

You can do that? Sweet! Update inc.

jspautsch

Hmmm, although, won't that only work if the function doesn't already exist? I seem to remember that being a limitation...

GlennAllen

I like the concept. However, I don't really want to hold all new posts for approval; I just want to hold the initial posts from new members--those who have yet to prove themselves as "trusted" (to mostly block spammers and the like); call it an introductory/probationary period. Once a member makes valid posts, he/she can become "trusted" and posts would no longer be held (unless some currently trusted member starts making "problem" posts and gets him-/herself set back to "untrusted"). Is something like this a planned option? Thanks.

hbf

jspautsch said: Hmmm, although, won't that only work if the function doesn't already exist? I seem to remember that being a limitation...

override will do just that, override any other function with that name. if you don't declare override then it needs to be a new function name.

hbf

GlennAllen said: I like the concept. However, I don't really want to hold all new posts for approval; I just want to hold the initial posts from new members--those who have yet to prove themselves as "trusted" (to mostly block spammers and the like); call it an introductory/probationary period. Once a member makes valid posts, he/she can become "trusted" and posts would no longer be held (unless some currently trusted member starts making "problem" posts and gets him-/herself set back to "untrusted"). Is something like this a planned option? Thanks.

i don't use this plugin, but it should be role based, so you just create a probationary membership role, and use this to capture all of the posts from users with that role.

jspautsch

hbf said:

jspautsch said: Hmmm, although, won't that only work if the function doesn't already exist? I seem to remember that being a limitation...

override will do just that, override any other function with that name. if you don't declare override then it needs to be a new function name.

Where you do declare override? The syntax Lincoln mentioned was Model_Function_Create()

neothemachine

GlennAllen's suggestion is also what I was looking for. If this addon would support that, then it's perfect :)