[MOD] Lock topic by topic creator

[Luuk]

Ik heb (mijn eerste echte) een mod gemaakt waarbij de starter van het topic zelf het topic kan sluiten en weer openen. Hiervoor komt een button te staan naast de edit button. Het nut ervan is dat mensen niet meer hoeven te zeggen van "hier kan een slotje op" als ze geholpen zijn, maar dat dan zelf kunnen doen.

Ik heb het zelf getests op mijn eigen pc, met een admin account en een gebruikers account.

Code: Selecteer alles

######################################################## 
## MOD Title:       Lock topic by topic creator 
## MOD Author:      Luuk < luukweerens@home.nl > Luuk Weerens 
## MOD Description: This mod creates a button like one from the mod cp for locking a topic. 
##                  Members get the oppertunity to lock their own topics. The admin can choose 
##                  in what forum. 
## MOD Version:     1.0.1 
## 
## Installation Level: Easy 
## Installation Time:  10 Minutes 
## 
## Files To Edit:     8 
##  admin/admin_forumauth.php 
##  admin/admin_forums.php 
##  admin/admin_ug_auth.php 
##  viewtopic.php 
##  includes/auth.php 
##  includes/constants.php 
##  includes/topic_review.php 
##  templates/subSilver/viewtopic_body.tpl 
## 
## Included Files:    None 
## 
############################################################## 
## This MOD is released under the GPL License. 
## Intellectual Property is retained by the MOD Author(s) listed above 
############################################################## 
## For Security Purposes, Please Check: http://www.phpbb.com/mods/downloads/ for the 
## latest version of this MOD. Downloading this MOD from other sites could cause malicious code 
## to enter into your phpBB Forum. As such, phpBB will not offer support for MOD's not offered 
## in our MOD-Database, located at: http://www.phpbb.com/mods/downloads/ 
############################################################## 
## Author Notes: 
##    The mod is created and tested on phpBB 2.0.7a, but it must work on all versions. 
## 
############################################################## 
## MOD History: 
## 
## v1.0.0 (27st of Marth, 2004) 
##    - Initial Release 
## v1.0.1 (28st of Marth, 2004) 
##    - Security Fix 
##    - Admin Control Over Locks 
## 
############################################################## 
## Before Adding This MOD To Your Forum, You Should Back Up All Files Related To This MOD 
##############################################################

De hele mod is hier te downen.

//edit: let op, deze mod is veranderd toen ik hem op phpbb.com hem gezet, dit is dus de officieele versie 1.0.1!

[superman5000]

gaaf!!
echt handig joh!!
mooi werk wacht al op je volgende MOD :thumb:

[Luuk]

Bedankt 8)

Nog iemand op/aan-merkingen?

[mosymuis]

Nog iemand op/aan-merkingen?

Ten eerste een interessant idee, het verbaast me als dit nog niet eerder gemaakt is, al hoop ik van niet voor je.

Ik vind het wel wat vreemd dat je het topic knopje naast de edit/quote/del knopjes plaatst, het slaat immers op een topic en niet op een post. Mischien kan je hem beter wegzetten bij de al bestaande {S_TOPIC_ADMIN} knopjes.

Daarnaast heb je dit in je install staan:

Code: Selecteer alles

# 
#-----[ OPEN ]------------------------------------------ 
# 
templates/XXXX/viewtopic_body.tpl

NOTE: Replace XXXX with your template name

het zou duidelijker zijn als je het als volgt deed:

Code: Selecteer alles

# 
#-----[ OPEN ]------------------------------------------ 
# Make sure to edit this file for every theme you use!
#
templates/subSilver/viewtopic_body.tpl

omdat je immers uitgaat van een subSilver installatie. En officiëel moet je werken met FIND en IN-LINE FIND, dus op deze manier:

Code: Selecteer alles

# 
#-----[ FIND ]------------------------------------------ 
# 
				<td valign="top" nowrap="nowrap">{postrow.QUOTE_IMG} {postrow.EDIT_IMG} {postrow.DELETE_IMG} {postrow.IP_IMG}</td>

#
#-----[ IN-LINE FIND ]---------------------------------------------------
#
{postrow.DELETE_IMG}

# 
#-----[ IN-LINE AFTER, ADD ]---------------------------------------------------
# 
 {postrow.LOCK_IMG}

Misschien is het een idee om het knopje niet weer te geven voor moderators/admins? Die hebben dit icoon immers al onderaan elk onderwerp. Dan zou je dit:

Code: Selecteer alles

if ( $forum_topic_data['topic_poster'] == $userdata['user_id'] || $is_auth['auth_mod'] )

moeten veranderen in dit:

Code: Selecteer alles

if ( $forum_topic_data['topic_poster'] == $userdata['user_id'] && !$is_auth['auth_mod'] )

En tot slot vraag ik me af of viewtopic.php?mode= wel de meest efficiënte oplossing is, je hebt immers het grootste deel van de code uit modcp.php en als je daar de permissies had aangepast voor de topicstarter had je het kunnen doen met heel wat minder code. Maar dit heb ik verder niet precies bekeken, dus misschien heb ik wel ongelijk.

//edit
Er valt me nog wat op; het lijkt erop dat je een stukje in je install bent vergeten;

Code: Selecteer alles

# 
#-----[ FIND ]------------------------------------------ 
# 
		'DELETE' => $delpost,

# 
#-----[ AFTER, ADD ]------------------------------------------ 
# 
		'LOCK_IMG' => $lock_img,

[Luuk]

Ten eerste een interessant idee, het verbaast me als dit nog niet eerder gemaakt is, al hoop ik van niet voor je.

Dat hoop ik ook, heb gezoht op phpbb.com en phpbbhacks maar had nix kunnen vinden.

Ik vind het wel wat vreemd dat je het topic knopje naast de edit/quote/del knopjes plaatst, het slaat immers op een topic en niet op een post. Mischien kan je hem beter wegzetten bij de al bestaande {S_TOPIC_ADMIN} knopjes.

Misschien kan ik hem beter bovenaan het topic neerzetten ofzo, ik zal kijken. Dit is wel makkelijk maar het plaatje past er niet bij (is te groot).

omdat je immers uitgaat van een subSilver installatie. En officiëel moet je werken met FIND en IN-LINE FIND, dus op deze manier:

Hmm ja, zal ik veranderen, en de in-line find actie heb ik maar gegokt en niet opgezocht

Misschien is het een idee om het knopje niet weer te geven voor moderators/admins? Die hebben dit icoon immers al onderaan elk onderwerp.

Ja, maar als je hem op die plek hebt staat is het voor mods ook nog es makkelijk om het topic te sluiten.

En tot slot vraag ik me af of viewtopic.php?mode= wel de meest efficiënte oplossing is, je hebt immers het grootste deel van de code uit modcp.php en als je daar de permissies had aangepast voor de topicstarter had je het kunnen doen met heel wat minder code. Maar dit heb ik verder niet precies bekeken, dus misschien heb ik wel ongelijk.

Ik heb het in viewtopic gezet zodat ik niet met auth gedoe zit te *beep* in modcp, en in viewtopic gaf het geen problemen.

//edit
Er valt me nog wat op; het lijkt erop dat je een stukje in je install bent vergeten;

Raar dat het er niet in staat, heb het denk ik perongeluk verwijderd uit het install.txt bestand

Ik zal je opmerkingen morgen nog eens rustig doorlezen en kijken wat ik er mee (kan) doen. Iig dat laatste stuk er weer in zetten

[Luuk]

Geupdate, mod heeft geen knop meer (alleen in zijn eigen topic)
De plaats van de knop is veranderd en heb install.txt aangepast (wat mosymuis zei, dat zit er nu ook bij)

[V!ncent]

ik vind dit een zeer goed idee maar 1 vraagje:
als een admin of mod hem op slot doet kan de geen die het topic heeft geopend hem dan ook weer open maken?

[Luuk]

Eh, ja. Misschien ook nog een optie om dat te veranderen zodat het niet kan

[V!ncent]

Eh, ja. Misschien ook nog een optie om dat te veranderen zodat het niet kan

dat lijkt mij eeen zeer goed idee

[Duvelske]

Geen slechte mod, maar ik vind dat je zo de beheerders veel werk uit handen haalt. En dat eigenlijk beheerders moeten beslissen dat het op slot moet. (mijn mening) Maar de mod zelf is :thumb: Moeten we eens een tje op drinken

[Luuk]

Ga er denk ik nog van maken dat de admin kan beslissen in welke fora het wel en in welke het niet mag, net zoals berichten verwijderen.

[LazyTiger]

misschien dat de admin ook bij gebruikers uit kan zetten of hij wel of niet topics mag sluiten.
(net zoals prive berichten sturen en avatars kiezen wat in een gebruiker zijn profiel staat in de admin)

[Luuk]

heb al een hele verbetering, eerst kon iedereen het topic sluiten als je de url wist (met viewtopic.php?mode=lock&t={topic_id}) maar dat gaat nu niet meer.

[Luuk]

Er zijn sommige dingen veranderd in de mod, oa een functie dat je in het adminpaneel kan bepalen of de gebruiker wel of niet zijn ¨topic mag sluiten in bepaalde fora.

ps: let op, deze mod is veranderd toen ik hem op phpbb.com hem gezet, dit is dus de officieele versie 1.0.1!

[mosymuis]

Code: Selecteer alles

## Files To Edit:	9
##  admin/admin_forumauth.php
##  admin/admin_forums.php
##  admin/admin_ug_auth.php
##	viewtopic.php
##  includes/auth.php
##  includes/constants.php
##  includes/topic_review.php

Ik tel er maar 7

[Luuk]

lol, ik heb ook altijd wat