Commit c8e5e931

mo khan <mo@mokhan.ca>
2014-10-01 03:10:30
remove unused columns from user.
1 parent 798e574
app/views/admin/sessions/index.html.erb
@@ -21,7 +21,7 @@
           <tr>
             <td><%= link_to user_session.user.name, admin_user_path(user_session.user) %></td>
             <td><%= user_session.ip %></td>
-            <td><%= user_session.city %></td>
+            <td><%= user_session.location.try(:city) %></td>
             <td><%= user_session.user_agent %></td>
             <td><%= time_ago_in_words(user_session.accessed_at) %></td>
             <td><%= time_ago_in_words(user_session.revoked_at) if user_session.revoked_at %></td>
app/views/admin/users/_index.html.erb
@@ -4,7 +4,6 @@
       <td>cakes</td>
       <td>name</td>
       <td>email</td>
-      <td>location</td>
       <td>created at</td>
     </tr>
   </thead>
@@ -14,7 +13,6 @@
         <td><%= user.creations_count %></td>
         <td><%= link_to user.name, admin_user_path(user) %></td>
         <td><%= mail_to user.email %></td>
-        <td><%= user.latitude %>, <%= user.longitude %></td>
         <td><%= time_ago_in_words(user.created_at) %></td>
       </tr>
     <% end %>
app/views/admin/users/show.html.erb
@@ -1,17 +1,3 @@
-<% content_for :javascript do %>
-<script type="text/javascript" src="https://maps.google.com/maps/api/js?sensor=false"></script>
-<script>
-  function initialize_map() {
-    var latitude = <%= @user.latitude %>;
-    var longitude = <%= @user.longitude %>;
-    var mapOptions = { zoom: 14, center: new google.maps.LatLng(latitude, longitude), mapTypeId: google.maps.MapTypeId.ROADMAP };
-    var map = new google.maps.Map(document.getElementById('map-canvas'), mapOptions);
-    var marker = new google.maps.Marker({ map: map, position: new google.maps.LatLng(latitude, longitude)});
-  }
-  google.maps.event.addDomListener(window, 'load', initialize_map);
-</script>
-<% end %>
-
 <div class="row-fluid">
   <div class="span2">
     <%= render partial: "my/shared/my_nav" %>
@@ -23,7 +9,6 @@
         <tr>
           <th>name</th>
           <th>email</th>
-          <th>city</th>
           <th>website</th>
           <th>twitter</th>
           <th>facebook</th>
@@ -33,7 +18,6 @@
         <tr>
           <td><%= link_to @user.name, admin_user_path(@user) %></td>
           <td><%= mail_to @user.email %></td>
-          <td><%= @user.city %></td>
           <td><%= link_to @user.website %></td>
           <td><%= @user.twitter %></td>
           <td><%= link_to @user.facebook %></td>
@@ -92,10 +76,5 @@
       <% end %>
       </tbody>
     </table>
-
-    <div style="overflow:hidden;height:500px;width:600px;">
-      <div id="map-canvas" style="height:500px;width:600px;"></div>
-      <style> img{max-width:none!important;background:none!important} </style>
-    </div>
   </div>
 </div>
db/migrate/20141001025113_remove_location_info_from_user_sessions.rb
@@ -0,0 +1,26 @@
+class RemoveLocationInfoFromUserSessions < ActiveRecord::Migration
+  def change
+    remove_column :user_sessions, :latitude
+    remove_column :user_sessions, :longitude
+    remove_column :user_sessions, :city
+    remove_column :user_sessions, :country
+
+    remove_column :users, :remember_created_at
+    remove_column :users, :sign_in_count
+    remove_column :users, :current_sign_in_at
+    remove_column :users, :last_sign_in_at
+    remove_column :users, :current_sign_in_ip
+    remove_column :users, :last_sign_in_ip
+    remove_column :users, :city
+    remove_column :users, :latitude
+    remove_column :users, :longitude
+    remove_column :users, :confirmation_token
+    remove_column :users, :confirmed_at
+    remove_column :users, :confirmation_sent_at
+    remove_column :users, :unconfirmed_email
+    remove_column :users, :failed_attempts
+    remove_column :users, :unlock_token
+    remove_column :users, :locked_at
+    remove_column :users, :invitation_token
+  end
+end
db/schema.rb
@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20140929024135) do
+ActiveRecord::Schema.define(version: 20141001025113) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -31,8 +31,8 @@ ActiveRecord::Schema.define(version: 20140929024135) do
 
   create_table "avatars", force: true do |t|
     t.integer  "user_id"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",        null: false
+    t.datetime "updated_at",        null: false
     t.string   "avatar"
     t.boolean  "avatar_processing"
     t.string   "avatar_tmp"
@@ -85,8 +85,8 @@ ActiveRecord::Schema.define(version: 20140929024135) do
     t.datetime "failed_at"
     t.string   "locked_by"
     t.string   "queue"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",             null: false
+    t.datetime "updated_at",             null: false
   end
 
   add_index "delayed_jobs", ["priority", "run_at"], name: "delayed_jobs_priority", using: :btree
@@ -103,8 +103,8 @@ ActiveRecord::Schema.define(version: 20140929024135) do
 
   create_table "interests", force: true do |t|
     t.string   "name"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
   end
 
   create_table "locations", id: :uuid, default: "uuid_generate_v4()", force: true do |t|
@@ -157,15 +157,13 @@ ActiveRecord::Schema.define(version: 20140929024135) do
     t.integer "taggings_count", default: 0
   end
 
-  add_index "tags", ["name"], name: "index_tags_on_name", unique: true, using: :btree
-
   create_table "tutorials", force: true do |t|
     t.string   "heading"
     t.text     "description"
     t.string   "url"
     t.integer  "user_id"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",  null: false
+    t.datetime "updated_at",  null: false
     t.string   "image_url"
     t.string   "author"
     t.string   "author_url"
@@ -182,10 +180,6 @@ ActiveRecord::Schema.define(version: 20140929024135) do
     t.string   "user_agent"
     t.datetime "accessed_at"
     t.datetime "revoked_at"
-    t.float    "latitude"
-    t.float    "longitude"
-    t.string   "city"
-    t.string   "country"
   end
 
   add_index "user_sessions", ["accessed_at"], name: "index_user_sessions_on_accessed_at", using: :btree
@@ -194,46 +188,27 @@ ActiveRecord::Schema.define(version: 20140929024135) do
   add_index "user_sessions", ["user_id"], name: "index_user_sessions_on_user_id", using: :btree
 
   create_table "users", force: true do |t|
-    t.string   "email",                  default: "", null: false
-    t.string   "encrypted_password",     default: "", null: false
+    t.string   "email",                              default: "", null: false
+    t.string   "encrypted_password",     limit: 128, default: "", null: false
     t.string   "reset_password_token"
     t.datetime "reset_password_sent_at"
-    t.datetime "remember_created_at"
-    t.integer  "sign_in_count",          default: 0
-    t.datetime "current_sign_in_at"
-    t.datetime "last_sign_in_at"
-    t.string   "current_sign_in_ip"
-    t.string   "last_sign_in_ip"
     t.datetime "created_at"
     t.datetime "updated_at"
     t.string   "name"
     t.string   "website"
     t.string   "twitter"
     t.string   "facebook"
-    t.string   "city"
-    t.float    "latitude"
-    t.float    "longitude"
     t.string   "password_salt"
-    t.string   "confirmation_token"
-    t.datetime "confirmed_at"
-    t.datetime "confirmation_sent_at"
-    t.string   "unconfirmed_email"
-    t.integer  "failed_attempts",        default: 0
-    t.string   "unlock_token"
-    t.datetime "locked_at"
     t.string   "authentication_token"
-    t.string   "invitation_token"
     t.string   "full_address"
-    t.integer  "creations_count",        default: 0
+    t.integer  "creations_count",                    default: 0
     t.boolean  "admin"
   end
 
   add_index "users", ["authentication_token"], name: "index_users_on_authentication_token", unique: true, using: :btree
-  add_index "users", ["confirmation_token"], name: "index_users_on_confirmation_token", unique: true, using: :btree
   add_index "users", ["created_at"], name: "index_users_on_created_at", using: :btree
   add_index "users", ["email"], name: "index_users_on_email", unique: true, using: :btree
   add_index "users", ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true, using: :btree
-  add_index "users", ["unlock_token"], name: "index_users_on_unlock_token", unique: true, using: :btree
 
   create_table "users_interests", id: false, force: true do |t|
     t.integer "user_id"