Quote:
Originally Posted by deathemperor
Thanks for your understanding.
|
yeah, i understand it completely, i knew it was not a final version...
i will report to you what is not done actually:
1- you did not use the proper $vbulletin->GPC[] structure for the globalised inputs... so it's impossible this script was tested for 3.5... look:
PHP Code:
// get values in this way it's faster and more secure using vb fuction globalsize()
$vbulletin->input->clean_array_gpc('p',array(
'keyword' => TYPE_NOHTML,
'searchfor' => TYPE_NOHTML,
'searchdate' => TYPE_NOHTML,
'_order' => TYPE_NOHTML,
'orderby' => TYPE_NOHTML,
'beforeafter' => TYPE_NOHTML
));
// if string search is too short (less than 3 char)
if (strlen($keyword) < 3)
{
eval(print_standard_error('searchnametooshort'));
}
the "if (strlen($keyword) < 3)" will never find its content, as we need to use "$vbulletin->GPC['keyword']" instead... and this is it abroad your script actually... some changes are to be planned..
EDIT: for the calendar_quickreply, it's not compatible with the new editor, so half the features are not working... will have to be fixed!
for the others, just phrases, but actually it's not a problem i think... just the permissions sentences are needed.
and if you have time, i don't think it would be useful to wait 'til 3.5 Gold, your code use the minimum of vB codes actually, it will be easy to integrate in the final...