<div dir="ltr"><div>Hi Alan,<br><br></div>Please do commit the migration changes through new file so we can pull the changes.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 5, 2015 at 4:54 PM, Alan Bell <span dir="ltr"><<a href="mailto:alan.bell@libertus.co.uk" target="_blank">alan.bell@libertus.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
I have a branch here <a href="http://code.vtiger.com/alanbell/vtigercrm/tree/secure_portal_passwords" rel="noreferrer" target="_blank">http://code.vtiger.com/alanbell/vtigercrm/tree/secure_portal_passwords</a> which contains this change <a href="http://code.vtiger.com/alanbell/vtigercrm/tree/secure_portal_passwords" rel="noreferrer" target="_blank">http://code.vtiger.com/alanbell/vtigercrm/tree/secure_portal_passwords</a><br>
which should provide encrypted passwords for the customer portal. It does require the user_password column in the vtiger_portalinfo table to be a bit longer than it is at the moment, varchar(30) is not enough to store a hashed password, varchar(128) is OK. How should I propose this schema change? modules/Migration/schema/640_to_650.php doesn't exist yet.<br>
<br>
On a related note, I think it would be good if *everything* in the migration files would always be safe to repeat. Anything that inserts stuff should first check to see if it is already there, anything that transforms things should check to see if they have already been transformed, if doing it twice would be harmful. Often a migration script fails somewhere due to a change in a specific instance, I fix the problem and run the migration script again and all the previous steps that were OK now cause havoc!<br>
<br>
Alan.<br>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a><br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr">With<br>Best Regards<br>Uma.S<br><div>Vtiger Team</div></div></div>
</div>