[zKiLL]

Код: Выделить всё

############################################################## 
## MOD Title: Advanced Marital Status MOD
## MOD Author: zKiLL < zkill@zmail.ru > (N/A) http://www.phpbbguru.net
## MOD Description: The mod displays your marital status of users in viewprofile and viewtopic.
##                     
## MOD Version:	1.0.0
##
## Compatibility:	2.0.22
##
## Installation Level: 	Intermediate
## Installation Time: 	15 Minutes
##
## Files To Edit: 11
##			viewtopic.php
##			admin/admin_users.php
##			includes/usercp_avatar.php 
##			includes/usercp_register.php 
##			includes/usercp_viewprofile.php 
##			language/lang_english/lang_main.php 
##			templates/subSilver/profile_add_body.tpl
##			templates/subSilver/profile_view_body.tpl
##			templates/subSliver/subSilver.cfg
##			templates/subSliver/viewtopic_body.tpl
##			templates/subSilver/admin/user_edit_body.tpl
##
## Included Files: 8
##			db_update.php
##			root/templates/subSliver/images/icon_marriage_single.gif 
##			root/templates/subSliver/images/icon_marriage_divorced.gif 
##			root/templates/subSliver/images/icon_marriage_engaged.gif 
##			root/templates/subSliver/images/icon_marriage_long_relation.gif 
##			root/templates/subSliver/images/icon_marriage_married.gif 
##			root/templates/subSliver/images/icon_marriage_separated.gif 
##			root/templates/subSliver/images/icon_marriage_widowed.gif 
##
## Support:		http://www.phpbbguru.ru
## Copyright:		2007 © zKiLL [ Advanced Marital Status MOD ]
##
##############################################################
## License: http://opensource.org/licenses/gpl-license.php GNU General Public License v2
############################################################## 
## For Security Purposes, Please Check: http://www.phpbbguru.net/mods/ for the 
## latest version of this MOD. 
############################################################## 
## Author Notes: 
##
##	My First Mod )
## 
############################################################## 
## MOD History:
##
##	2007-05-21 - Version 1.0.0 
##	- Initial Release 
## 
############################################################## 
## Before Adding This MOD To Your Forum, You Should Back Up All Files Related To This MOD 
############################################################## 

Данный мод отражает ваше семейное положение.
Идентично личным настройкам ICQ / QIP.

Мой первый мод. Прошу помочь в тестировании. Жду адекватной критики, предложений.

Добавлено спустя 1 минуту 45 секунд:

Примечание: В архиве НЕТ картинок! Созданы только названия для них. Картинки придется рисовать!

Если все хорошо, прошу добавить мод в базу данных.

[Xpert]

А с какой целью запрещено указывать семейное положение после регистрации? Кроме того именно в реализации данного запрета присутствует ошибка...

Код: Выделить всё

# 
#-----[ AFTER, ADD ]------------------------------------------ 
# 

	// Start add - Advanced Marital Status MOD
	$marriage = ( isset($HTTP_POST_VARS['marriage']) ) ? intval ($HTTP_POST_VARS['marriage']) : 0;
	// End add - Advanced Marital Status MOD

# 
#-----[ FIND ]------------------------------------------------ 
# 

$sql = "UPDATE " . USERS_TABLE . "
SET

# 
#-----[ IN-LINE FIND ]---------------------------------------- 
# 

$avatar_sql . "

# 
#-----[ IN-LINE AFTER, ADD ]---------------------------------- 
# 

, user_marriage = '$marriage'

Недостаточно только деактивировать элемент формы, злоумышленник может ее скачать и подменить код. Соотвественно нужна проверка на mode != 'register' в коде. Если указания семейного положения в моде после регистрации вообще не предусмотрено, тогда я не понимаю к чему кусок кода выше.

[Поручик]

Получается, зарегистрировался на таком форуме холостым и всё, жениться уже нельзя. Типа обет безбрачия взял.

[zKiLL]

мой косяк ) подправлю.

еще подумал над:
1) поженить мод на гендере.
2) сделать админку.
3) возможность выбора вашей ПАРЫ с форума (с ее(его) согласия).

подумаю...

но в принципе так нормально да? )

Добавлено спустя 10 минут 51 секунду:

Ну вы это тоже... помогайте мне ) я нуб )) я буду сам долго очень думать, и вполне возможно накосячу еще больше )

Добавлено спустя 7 минут 9 секунд:

Xpert писал(а):Недостаточно только деактивировать элемент формы, злоумышленник может ее скачать и подменить код. Соотвественно нужна проверка на mode != 'register' в коде. Если указания семейного положения в моде после регистрации вообще не предусмотрено, тогда я не понимаю к чему кусок кода выше.

если не трудно, пишите мне КАК это исправлять ) хотя бы примерно. я слегонца путаюсь. пока, как и у всех начинающих, у меня работа с кодом обстоит таким образом, что я черпаю знания из чужих кодов, если не нахожу там ответов, вхожу в ступор )

[Xpert]

zKiLL писал(а):3) возможность выбора вашей ПАРЫ с форума (с ее(его) согласия).

Есть такой...

Код: Выделить всё

############################################################## 
## MOD Title: Marriage MOD 
## MOD Author: WyriHaximus < webmaster@wyrihaximus.net > (Cees-Jan Kiewiet) http://www.WyriHaximus.net/ 
## MOD Description: This MOD allows forum members to marry. 
## MOD Version: SNAPSHOT06 
## 
## Installation Level: Advanced 
## Installation Time: 5 Minutes 
############################################################## 

Причем сильно навороченный.

zKiLL писал(а):если не трудно, пишите мне КАК это исправлять

Я бы вообще от идеи запрета смены настроек отказался.

[zKiLL]

Давай может, ты мод возьмешь на себя, и доведешь до ума?

[Xpert]

Второе подобное предложение за неделю
Оставь себе, у меня и так хватает. Только выкинь оттуда проверки на заполненность. Каждый мужчина хотя бы раз в год мечтает побыть холостяком (с)

[Derjavin]

Та ну, холостяком нехорошо, эт я за 4 года уже понял хорошо