Offending code caches function in a variable so the TextField.value never update the model.

gistfile1.txt

updateUserModel = function(){ 
  // this is bad because it caches the function in the variable, 
  // so the Textfield.value won't get the latest value 
  // but only the original cached. 
  // This bad practice can happen if being lazy after getting rid of the exports prefix of a commonjs exports.functionname.
  // it will still work for most cases but cause nasty caching bugs for when textfield values need to be updated

  	var currentUser ={
			firstname: $.firstName.value, // these are textfields
			surname: $.surname.value,
			address1: $.address1.value,
			address2: $.address2.value,
			suburb: $.suburb.value,
			state: $.state.text,
			pc: $.postCode.value,
			email: $.email.value
		}
		
		Alloy.Globals.ModelUser.save(currentUser);
		Alloy.Globals.ModelUser.trigger('change');
}


// # Solution
// function updateUserModel(){ }
// needs to have the traditional function. 
// rather than
// updateUserModel = function(){ }
// and then later use
// exports.updateUserModel = updateUserModel;
// if we need to expose it.