Hi,
GA Login has to attach itself to the login validators. Therfore, it hooks into the login form and overwrites the validator variable. However, this does bot work well, if the site is using different 2 factor systems at the same time. I used for example YubiKey and GA login. The better idea would be to unshift the validators and place the ga login validator at the correct position.
here the patch:
/**
@@ -265,12 +266,8 @@ function ga_login_test_form_submit($form, $form_state) {
*/
function ga_login_form_alter(&$form, &$form_state, $form_id) {
if ($form_id == 'user_login_block' || $form_id == 'user_login') {
- $form['#validate'] = array(
- 'user_login_name_validate',
- 'ga_login_user_login_validate',
- 'user_login_authenticate_validate',
- 'user_login_final_validate',
- );
+ array_unshift($form['#validate'],$form['#validate'][0]);
+ $form['#validate'][1] = 'ga_login_user_login_validate';
Comments
Comment #1
Jelle_Sfixed in latest dev
PS: for future reference, it's easier for us if you upload the patch as a file ;-)
Comment #2
Jelle_SComment #3
Martin Klinkigt CreditAttribution: Martin Klinkigt commentedok. in future I will provide the files ;).
Only problem is, that I patched many things at once, so I would have to split the patches for the different issues.
thank you for your support.
Comment #4.0
(not verified) CreditAttribution: commentedprovided patch