Thanks for pointing that out, and sorry for the trouble. The code I did above was from looking at the flow, and not actually testing. It's not always possible to test every code change (and usually not necessary for seemingly simple stuff).
I went through and developed and tested how to do this, fixing the issue you mention and finding another that you would have run into right after. Below is all the code changes that need to be done in Joomla 2.5 to make this work.
In the /components/com_jlinked/views/loginregister/view.html.php, around line 44, you'll see:
// Setup the fields we can pre-populate
// To do: Give option to show/hide the name on the form
$this->form->setValue('name', null, $profile->get('first-name') . " " . $profile->get('last-name'));Add the following line after that the
following 2 lines (above I mentioned only the first):
$this->form->setFieldAttribute('username', 'type', 'hidden');
$this->form->setFieldAttribute('username', 'class', 'non-required');
Then, in the /components/com_jlinked/controllers/loginregister file, around line 60, you'll see:
$userModel = JModel::getInstance('Registration', 'UsersModel');
$requestData = JRequest::getVar('jform', array(), 'post', 'array');Add the following line directly after that:
$requestData['username'] = $requestData['email1'];
If you followed all the steps in the threads above, please
undo the change I mentioned in the /com_jlinked/models/loginregister.php file. That's unnecessary.
This has been tested on our development systems, but not for every possible JLinked setting or with the profile plugins. Nothing should affect it, but obviously, do your own testing
Please let us know how it goes!
Alex
Email as a username
