[ FINAL ] Advanced Marital Status MOD

Здесь авторы могут постить бета-версии своих модификаций для phpBB 2.0.x. Внимание! Не устанавливайте бета-версии модов на работающие форумы!

[ FINAL ] Advanced Marital Status MOD

Сообщение zKiLL 21.04.2007 14:40

Код: Выделить всё
##############################################################
## 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 секунд:

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

Если все хорошо, прошу добавить мод в базу данных.
Вложения
Advanced_Marital_Status_1.0.0.zip
(6.26 Кб) Скачиваний: 132
zKiLL
phpBB 1.4.1
 
Сообщения: 43
Зарегистрирован: 12.08.2006 12:46


Сообщение Xpert 21.04.2007 14:47

А с какой целью запрещено указывать семейное положение после регистрации? Кроме того именно в реализации данного запрета присутствует ошибка...
Код: Выделить всё
#
#-----[ 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' в коде. Если указания семейного положения в моде после регистрации вообще не предусмотрено, тогда я не понимаю к чему кусок кода выше.
Эксперт - это человек, который избегает мелких ошибок на пути к грандиозному провалу.
Любая более-менее сложная задача имеет несколько простых, изящных, лёгких для понимания неправильных решений
Аватара пользователя
Xpert
phpBB Guru
 
Сообщения: 5710
Зарегистрирован: 13.03.2004 22:27
Откуда: msk.ru

Сообщение Поручик 21.04.2007 15:28

Получается, зарегистрировался на таком форуме холостым и всё, жениться уже нельзя. Типа обет безбрачия взял.
Профессионал - тот же дилетант, только знающий, где ошибётся.
Генератор db_update.php для phpBB2 с некоторыми удобствами. Многие моды я беру или ищу здесь, здесь, тут
Все консультации только на форуме, приваты и стук в аську по таким вопросам игнорируются!
FAQ-phpBB2 | FAQ-phpBB3 | Ошибки новичков, или как не поссориться с модератором | Правила конференции

наш форум http://forum.aeroion.ru/
Аватара пользователя
Поручик
Его Благородие
Его Благородие
 
Сообщения: 3663
Зарегистрирован: 12.05.2005 17:25
Откуда: Оренбург (Южный Урал)

Сообщение zKiLL 21.04.2007 18:42

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

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

подумаю...

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

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

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

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

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


если не трудно, пишите мне КАК это исправлять ) хотя бы примерно. я слегонца путаюсь. пока, как и у всех начинающих, у меня работа с кодом обстоит таким образом, что я черпаю знания из чужих кодов, если не нахожу там ответов, вхожу в ступор )
Newbie ) Не обижать )
zKiLL
phpBB 1.4.1
 
Сообщения: 43
Зарегистрирован: 12.08.2006 12:46

Сообщение Xpert 21.04.2007 23:00

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 писал(а):если не трудно, пишите мне КАК это исправлять

Я бы вообще от идеи запрета смены настроек отказался.
Эксперт - это человек, который избегает мелких ошибок на пути к грандиозному провалу.
Любая более-менее сложная задача имеет несколько простых, изящных, лёгких для понимания неправильных решений
Аватара пользователя
Xpert
phpBB Guru
 
Сообщения: 5710
Зарегистрирован: 13.03.2004 22:27
Откуда: msk.ru

Сообщение zKiLL 22.04.2007 16:04

Давай может, ты мод возьмешь на себя, и доведешь до ума?
Newbie ) Не обижать )
zKiLL
phpBB 1.4.1
 
Сообщения: 43
Зарегистрирован: 12.08.2006 12:46

Сообщение Xpert 22.04.2007 16:06

Второе подобное предложение за неделю :)
Оставь себе, у меня и так хватает. Только выкинь оттуда проверки на заполненность. Каждый мужчина хотя бы раз в год мечтает побыть холостяком (с)
Эксперт - это человек, который избегает мелких ошибок на пути к грандиозному провалу.
Любая более-менее сложная задача имеет несколько простых, изящных, лёгких для понимания неправильных решений
Аватара пользователя
Xpert
phpBB Guru
 
Сообщения: 5710
Зарегистрирован: 13.03.2004 22:27
Откуда: msk.ru

Сообщение Derjavin 09.05.2007 3:23

Та ну, холостяком нехорошо, эт я за 4 года уже понял хорошо :D
"Только когда плывешь против течения,
Понимаешь, чего стоит свободное мнение"
Аватара пользователя
Derjavin
phpBB 1.4.3
 
Сообщения: 69
Зарегистрирован: 22.05.2006 19:13
Откуда: Ukraine, Kiev


Вернуться в Бета-версии модов для phpBB 2.0.x

Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и гости: 0