| T O P I C R E V I E W |
| leatherlips |
Posted - 24 October 2005 : 21:35:37
With my other post on how to only allow moderators and admins to use the reply with quote function, I was wondering if I could do the same with images. The reply with quote alteration was pretty simple, would it be just as simple with the images?
I'd like to only allow moderators and admins to be able to post images. However, I can't find where this code is. It doesn't seem to appear in the topic.asp file. Where would I look?
Would it be as simple as adding the parts below in green around the correct section of code?
If mlev >= 3 then
Section of code here
End If
Thanks.
< Moved to MOD Add-On Forum (W/O Code) by Shaggy />< |
| 15 L A T E S T R E P L I E S (Newest First) |
| rkp |
Posted - 06 August 2007 : 11:09:26
does anyone have this working?< |
| AnonJr |
Posted - 03 November 2005 : 10:22:15
Now that I think about it, I believe that there is a second place in topic.asp too....< |
| Shaggy |
Posted - 03 November 2005 : 10:05:47
And don't forget the other files I mentioned above, as well. If you don't amend all occurences of FormatStr in your files, you're going to get some errors.
< |
| AnonJr |
Posted - 03 November 2005 : 09:30:30
I've not had a chance to test this, but I think this will give you what you want:
Change in inc_func_common.asp:
On (or about) Line 114 change:function FormatStr(fString)
to:function FormatStr(fString, bAllowImages)
On (or about) Line 126 change:if strIMGInPosts = "1" then
to:if strIMGInPosts = "1" And bAllowImages then
You will need to find all instances where FormatStr is called and add either True or False depending on if you want images or not.
For instance, in topic.asp around line 663 you will need to change
if qsSearchTerms <> "" then
Response.Write SearchHiLite(formatStr(Reply_Content))
else
Response.Write formatStr(Reply_Content)
end if
to
if qsSearchTerms <> "" then
If mLev >= 3 Then
Response.Write SearchHiLite(formatStr(Reply_Content,True))
Else
Response.Write SearchHiLite(formatStr(Reply_Content,False))
End If
else
If mLev >= 3 Then
Response.Write formatStr(Reply_Content,True)
Else
Response.Write formatStr(Reply_Content,False)
End If
end if
Let me know how it goes. (or if I goofed  )< |
| AnonJr |
Posted - 03 November 2005 : 08:38:58
Great minds and all that  let me get some coffee and see what can be put together...< |
| Shaggy |
Posted - 03 November 2005 : 05:30:22
We're thinking the same thing, dude 
< |
| AnonJr |
Posted - 02 November 2005 : 13:25:38
I was thinking more along the lines of an extra argument passed along in the call...
FormatStr([...normalArgs...],blnAllowImages)
And then check if blnAllowImages is true or not to allow images. It would be more work up front, but it would ultimately make the function more flexible by giving more control over where images are allowed and where they aren't.< |
| Shaggy |
Posted - 02 November 2005 : 12:35:54
I wouldn't say there's a need for a field (I assume you mean database field), Anon, unless you want to make it an admin option. Although, you could use an additional boolean argument when calling the FormatStr function which would specify whether or not to call ReplaceImageTags. Would be easier, in the long run, than passing M_LEVEL.
Other files where FormatStr is used, off the top of my head: pop_preview, pop_previe_sig, pop_profile, default, default_group, pop_printer_friendly.
< |
| leatherlips |
Posted - 02 November 2005 : 12:11:24
I enjoy reading through your posts Shaggy and AnonJr, but my head is spinning....boolean, FormatStr()to ReplaceImageTags()...I have no idea what you guys are talking about.
I'm glad you guys know what you are doing!
The reason I would like to get this to work is because my forum is being used for a middle school. The teachers are the mods. I'd like to be able to allow the teachers to post images that may be used for examples for the questions they are discussing. I just don't want the middle school students (normal users) to be able to post...you never know what they'll put up. < |
| AnonJr |
Posted - 02 November 2005 : 11:09:01
Ah. I wonder if it would be worth adding a boolean field for allowing images... that way you could be more particular in where images are allowed and where they are not... or do you think it would be more work than it would be worth?< |
| Shaggy |
Posted - 02 November 2005 : 10:26:19
Actually, you'd only need to pass it through to FormatStr and then use it to check whether or not to call ReplaceImageTags. Problem is, FormatStr is used elsewhere besides topic.asp.
< |
| AnonJr |
Posted - 02 November 2005 : 09:32:36
Hmm... I'll look at it as I wait for the class to finish ...< |
| Shaggy |
Posted - 02 November 2005 : 09:28:44
It'd be tricky, you'd have to pass the member level of the poster down through FormatStr() to ReplaceImageTags().
< |
| AnonJr |
Posted - 02 November 2005 : 09:18:12
Ah. Where would it go then?< |
| Shaggy |
Posted - 02 November 2005 : 05:48:09
That wouldn't work as ReplaceImageTags() is called by the FormatStr() function which a post is passed through when it is pulled from the database, not inserted. Using the above method would mean that only moderators and admins would see images posted by any member of your forums.
< |
