Merge lp:~mhall119/wordpress-launchpad-integration/auto-submit-login into lp:wordpress-launchpad-integration

Proposed by Michael Hall
Status: Needs review
Proposed branch: lp:~mhall119/wordpress-launchpad-integration/auto-submit-login
Merge into: lp:wordpress-launchpad-integration
Diff against target: 16 lines (+5/-1)
1 file modified
launchpad_login.php (+5/-1)
To merge this branch: bzr merge lp:~mhall119/wordpress-launchpad-integration/auto-submit-login
Reviewer Review Type Date Requested Status
Matthew Nuzum (community) Approve
Review via email: mp+69351@code.launchpad.net

Commit message

Auto-submits the login form that takes you to SSO if there are no errors or messages to display

Description of the change

Auto-submits the login form that takes you to SSO if there are no errors or messages to display

To post a comment you must log in.
Revision history for this message
Matthew Nuzum (newz) :
review: Approve

Unmerged revisions

21. By Michael Hall

Auto-redirect to SSO if there isn't anything to display to the user

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'launchpad_login.php'
2--- launchpad_login.php 2010-04-26 15:02:31 +0000
3+++ launchpad_login.php 2011-07-26 19:54:10 +0000
4@@ -117,7 +117,11 @@
5 </p>
6
7 <script type="text/javascript">
8-try{document.getElementById('lplogin').focus();}catch(e){}
9+<?php if (!empty($errors) || !empty($messages)) { ?>
10+ try{document.getElementById('lplogin').focus();}catch(e){}
11+<?php } else { ?>
12+ try{document.getElementById('lplogin').click();}catch(e){}
13+<?php } ?>
14 </script>
15 </body>
16 </html>

Subscribers

People subscribed via source and target branches