Welcome, Guest. Please login or register.
Login with username, password and session length


VirtueMart 1.1.8 - [SECURITY RELEASE] is available! Read more....

  Advanced search

247038 Posts in 67506 Topics- by 258314 Members - Latest Member: aniketana
Pages: [1]   Go Down
Print
Author Topic: Required fields in registration form  (Read 1227 times)
ultracandido
Newbie
*
Posts: 2


« on: November 18, 2010, 10:41:15 AM »

Description:
When one of the following fields is the former of the registration form, the JS validator shows an alert telling us that some of the required fields are not filled in, even if registration type is optional:
fields are:
* username
* email
* password
* password2


VirtueMart Version:
1.1.5

Joomla/Mambo Version:
Joomla 1.5.20

Steps to replicate:
In the administrator, go to "Manage user field" and modify the order of the fields and put the password field at the top of the list.
In the frontend, place an order, check it out, and in the registration form set the "register and account" checkbox unchecked. Fields usename, password and retype password should now disappear. Fill in all the required fields and submit the form. You will get the error.

Proposed fix(es):
There is weird compositon of the $field_list variable (line 641 administrator/components/com_virtuemart/classes/ps_userfield.php) using the implode function so on the next few lines the str_replace function fail to replace the string 'password' (with the starting ') because it isn't in the string.
 

Bugtracker task #:


System info:
UBUNTU 9.10 LAMP server

cheers
« Last Edit: November 18, 2010, 10:46:17 AM by ultracandido » Logged
zanardi
Development Team
Hero Member
*
Posts: 812


WWW
« Reply #1 on: November 18, 2010, 14:16:44 PM »

Thank you for the info, if anyone else can confirm this behaviour i will surely insert it in the list of bugs for VM 1.1.7 (VM 1.1.6 is so close to release that it's impossible to ad new fixes).
Logged

ultracandido
Newbie
*
Posts: 2


« Reply #2 on: November 19, 2010, 05:32:21 AM »

The patch is really simple, i can post it if you trust me...   Tongue
Logged
zanardi
Development Team
Hero Member
*
Posts: 812


WWW
« Reply #3 on: November 19, 2010, 07:36:22 AM »

Ok, send the patch here on the forum and i will open a tracker item with it.
Logged

LarsLB
Newbie
*
Posts: 9


« Reply #4 on: December 30, 2010, 14:13:17 PM »

would love the patch ..
I experience the same thing ..

I hate a digital store ..
really only need people to fill in their email address ..
but even when Register Account is unchecked, the form returns as not completed

lb.

Logged
Pages: [1]   Go Up
Print
Jump to: