![]() |
set minimal length for password for pw-change
This little modification ensures that users use passwords with at least an amount of characters you define. This length is currently set to 8 characters but can be modified ofcourse.
If you want to change that length, just look at the PHP-code and the new phrase below, the corresponding places are marked green. Thanks to Mystics for help with the javascript :) open /profile.php find Code:
globalize($_POST, array('currentpassword' => STR, 'currentpassword_md5' => STR, 'newpassword' => STR, 'newpasswordconfirm' => STR, 'newpassword_md5' => STR, 'newpasswordconfirm_md5' => STR, 'email' => STR, 'emailconfirm' => STR)); Code:
globalize($_POST, array('currentpassword' => STR, 'currentpassword_md5' => STR, 'newpassword' => STR, 'newpasswordconfirm' => STR, 'newpassword_md5' => STR, 'newpasswordconfirm_md5' => STR, 'npwlength' => INT, 'email' => STR, 'emailconfirm' => STR)); Code:
if (!empty($newpassword) OR !empty($newpassword_md5)) Code:
// ### Password-Security-Hack Edit the template modifypassword and replace all of it with the following code: Code:
<script type="text/javascript" src="clientscript/vbulletin_md5.js"></script> Now enter the phrase-manager, select the phrase-type Front-End Error Messages and add a new phrase: VARNAME: unsecurepassword TEXT: Code:
The password you chose does not fit our standard of security, please use a password with <b>at least 8 characters.</b><br><br>Click here to <a href="profile.php?$session[sessionurl]do=editpassword">change your password</a>. That's all, you're done :smoke: |
red = green here, remember (we have a colorblind bbcode here!)
|
Nice hack. How about applying this to registrations too?
|
Err... No offsense, dude, but do you check your hacks before posting them?
You have both syntax and logic error in your hack... Syntax error: if ($npwlength > 0 AND $npwlength <= 7{ Don't you need a closing ')' before '{'??? Logic error: It says: if ($npwlength > 0 AND $npwlength <= 7{ If you want your password to be not less than 7 characters in length, then it logically follows that it will be more than 0 characters. Therefore, the first part of the condition does not make any sense. A final, more valid, condition would be: if ($npwlength <= 7) { :) But anyway, this is a nice hack, and I am going to install it when you finish developing it :) |
The $npwlength > 0 part of his condition is probably so that the default error message is shown if no password is entered, instead of this custom one about the password length.. ;)
|
Oblivion Knight
In that case, it would be best to put this check into IF..ELSEIF, instead of two separate IF's. That would make it easier to understand. |
its a hack so it is probably better putting it in a seperate if rather than to play around with vb core if,else structure... Is there a reason that you are checking if the length is less than or equal to seven? why not just use < 8?
|
This is a simple if..else structure you are talking about here, so I don't think it will hurt making some intergration...
|
Please post this in a text file so others can have it on an upgrade. ;)
|
I updated the missing ) and named the color "green" ...
|
All times are GMT. The time now is 12:27 PM. |
Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2025, vBulletin Solutions Inc.
X vBulletin 3.8.12 by vBS Debug Information | |
---|---|
|
|
![]() |
|
Template Usage:
Phrase Groups Available:
|
Included Files:
Hooks Called:
|