tkh_authentication 0.1.10 → 0.1.11

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YTI1YWNhMzk1MmJkYTIwZjMzMWE0ZTFmMTg3OGE2NDMwMjEwZDUzNg==
+    ODU5ZDNhMDM0ZmQ0ZDI4MzhmYjRhODZhZDUzNWY4ZjJlNDk0YzFlMg==
   data.tar.gz: !binary |-
-    OTYwYjBjMDEwYThmNDZlYzkyNzA2ZTNjZDQ0ZmFkYTZlNzI4OTBiNg==
+    MTkxMmM0ZDZlMmE0MWE2YmVhN2MwYWY5ZDcwMzgyMzBhM2EwMjNjYw==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    YjUzYTJmYmM2Mjc2YTMyMTJkYTBhNTYxMDcxNDZjZmM2OGU2M2VlZWU5MjYw
-    ODZiOTA0NGRkNmFmNDlmMGFjOGUwZDNhYzk3ZTAxNmEyMmVjMjE5MWFiNWI5
-    NGVhZDE0ZGJhODAyN2JhY2MwZDFhYTQ1OGUzNzVmNmRkODhlMjI=
+    MWE5MTk3MjMzNThjYTNlNDliMTJiYTU0OGQ5NmY1OWQwYmJkOTczMjMzOTRi
+    MGQzMDk2MTlmOWE0NDBhYjI3OGQ3ODg3NjY0ZTZjMzZiODI5OWM5YjM4ZWVj
+    MmRiNWU2OWMyZTdkZjg0MGM4NTQ0YmY4NGY5NTQwZmMzNzQyMGY=
   data.tar.gz: !binary |-
-    YmI4NTZkODZiMTBmNmI5Njk2NzE5MmMxZmUzNGFlNTMwNTRjZGM2ODFjNmM0
-    Mzc3MTNhYjI3ZWM0NmE5NjM5OTQ0NmRkNTJlZDIwNjRlY2VmN2EzZDRiNWM3
-    N2U3ZjgyYTliMjAxMzc3YjdhZDQxYzIxNWY4MDFlODdmZTQ0NGE=
+    M2NjZTNmZTliMTMwZDJlZTkxZjMyZTcwNTNmNGYxNzFiOTJkN2ZjNzNhNDE5
+    NmViNzQzZmNmODNjNGRjNTc4MGRkZDBkNDg1Yjc2ODhmNDdmOWVkNDFiNGIw
+    MTg2YjRjZWRhMTYyYTNmZTg2MjU3ZTljZDg1OTBlN2FkNTUwZGE=

data/CHANGELOG.md

@@ -1,6 +1,12 @@
 # TKH Authentication
 
 
+## 0.1.11
+
+* Ensure the target page is always set and removed when needed to improve the click-flow of users signing in or creating an account.
+* When user is redirected to login page from the detect address view, his entered email address follows with.
+
+
 ## 0.1.10
 
 * Debugged the user has comments dependent destroy bug

data/app/controllers/password_resets_controller.rb

@@ -21,9 +21,16 @@ class PasswordResetsController < ApplicationController
     elsif @user.update_attributes(params[:user])
       cookies[:auth_token] = @user.auth_token # logging in the user
       redirect_to session[:target_page] || safe_root_url, notice: t('authentication.password_reset_confirmation')
+      destroy_target_page
     else
       render :edit
     end
   end
 
+  private
+
+  def destroy_target_page
+    session[:target_page] = nil
+  end
+
 end

data/app/controllers/sessions_controller.rb

@@ -15,7 +15,7 @@ class SessionsController < ApplicationController
         cookies[:auth_token] = user.auth_token
       end
       redirect_to (session[:target_page] || safe_root_url), notice: t('authentication.login_confirmation')
-      session[:target_page] = nil
+      destroy_target_page
     else
       flash.now.alert = t('authentication.warning.email_or_password_invalid')
       render "new"
@@ -24,6 +24,7 @@ class SessionsController < ApplicationController
 
   def destroy
     cookies.delete(:auth_token)
+    destroy_target_page
     redirect_to safe_root_url, notice: t('authentication.logout_confirmation')
   end
 
@@ -33,4 +34,8 @@ class SessionsController < ApplicationController
     session[:target_page] = request.referer unless session[:target_page] # && !request.referer.nil?
   end
 
+  def destroy_target_page
+    session[:target_page] = nil
+  end
+
 end

data/app/controllers/users_controller.rb

@@ -26,10 +26,11 @@ class UsersController < ApplicationController
 
   def detect_existence
     set_target_page
-    user = User.where('email = ?', params[:user][:email]).first
+    @detected_email = params[:user][:email]
+    user = User.where('email = ?',@detected_email).first
     if user && !user.password_digest.blank? && user.password != 'temporary'
       flash[:notice] = "Our records show you have an account with us. Please login."
-      redirect_to login_path
+      redirect_to login_path(email: @detected_email)
     else
       if user
         newbie = Newbie.where('email = ?', params[:user][:email]).first

data/app/views/sessions/_form.html.erb

@@ -1,7 +1,6 @@
 <%= form_tag sessions_path, class: 'simple_form form-horizontal' do %>
 
 	<div class="form-inputs">
-		
 	  <div class="control-group email required">
 	    <%= label_tag :email, t('activerecord.attributes.user.email'), class: "email required control-label" %>
 			<div class="controls">
@@ -9,7 +8,7 @@
 				<p class="help-block"><%= link_to t('authentication.not_registered'), signup_path %></p>
 			</div>
 	  </div>
-	
+
 		<div class="control-group password required">
 	    <%= label_tag :password, t('activerecord.attributes.user.password'), class: "password required control-label" %>
 			<div class="controls">
@@ -23,11 +22,11 @@
 				<label class="checkbox"><input class="boolean optional" id="remember_me" name="remember_me" type="checkbox" value="1" /><%= t('authentication.remember_me') %></label>
 			</div>
 		</div>
-	
+
 	</div>
-	
+
 	<div class="form-actions">
     <%= submit_tag t('authentication.login_now'), class: 'btn btn-primary' %>
   </div>
-	
-<% end %>
+
+<% end %>

data/lib/tkh_authentication/version.rb

@@ -1,3 +1,3 @@
 module TkhAuthentication
-  VERSION = "0.1.10"
+  VERSION = "0.1.11"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tkh_authentication
 version: !ruby/object:Gem::Version
-  version: 0.1.10
+  version: 0.1.11
 platform: ruby
 authors:
 - Swami Atma
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-04 00:00:00.000000000 Z
+date: 2013-08-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails