created models to clear outheres classests.

Author: raphox

README.md

@@ -23,6 +23,20 @@ Things you may want to cover:
 
 * ...
 
+## Debugging with `byebug`
+
+## Docker compos config
+
+```yaml
+app:
+  tty: true
+  stdin_open: true
+```
+
+This also makes it possible to after the containers are up do docker attach project_app_1 which seems to work fine.
+
+https://github.com/docker/compose/issues/423#issuecomment-141995398
+
 ## Neo4j
 
 ### Removing data

app/controllers/application_controller.rb

@@ -8,6 +8,6 @@ def set_session_user_id
 
     @current_user = User.first if @current_user.nil?
 
-    session[:session_user_id] = @current_user.id.to_s
+    session[:session_user_id] = @current_user&.id&.to_s
   end
 end

app/controllers/users_controller.rb

@@ -12,33 +12,20 @@ def index
   # GET /users/1
   # GET /users/1.json
   def show
-    @user_neo4j = UserNeo4j.find(@user.neo4j_uuid)
+    @user_neo4j = UserNeo4j.find(@user.neo4j_uuid) if @user.neo4j_uuid.present?
     @friends_path = {}
 
     # TODO: optimize queries to locate Users
-    @friends =
+    @friends = begin
       if params[:q].present?
-        uuids = @user_neo4j.
-          friends(:l, :r, rel_length: 5).
-          where("ANY(title IN l.titles WHERE toLower(title) CONTAINS toLower({title}))").
-          params({ title: params[:q] }).
-          pluck(:uuid)
-
-        uuids.each do |uuid|
-          @friends_path[uuid] = Neo4j::ActiveBase.current_session.
-            query("MATCH (r:UserNeo4j { uuid: '#{@user.neo4j_uuid}' }),
-              (l:UserNeo4j { uuid: '#{uuid}' }),
-              p = shortestPath((r)-[FRIEND*..15]->(l))
-              RETURN p", limit: 1).first.p.nodes.
-            map { |node| User.find_by({ neo4j_uuid: node.properties[:uuid] }) }
-        end
-
-        User.where(:neo4j_uuid.in => uuids)
-      else
+        byebug
+        uuids = @user_neo4j.friendsBySearch(params[:q]).pluck(:uuid)
+      elsif @user_neo4j.present?
         uuids = @user_neo4j.friends.pluck(:uuid)
-
-        User.where(:neo4j_uuid.in => uuids)
       end
+
+      uuids.present? ? User.where(:neo4j_uuid.in => uuids) : []
+    end
   end
 
   # GET /users/new

app/helpers/users_helper.rb

@@ -6,6 +6,8 @@ def invite_link_to(user)
     link_to invite_user_path(user), class: "card-footer-item btn-invite #{'is-active' if invited}" do
       content_tag(:div, nil, class: 'heart') + ' Invite'
     end
+  rescue
+    nil
   end
 
   def friends_path(path)

app/models/concerns/user_sync.rb

@@ -0,0 +1,39 @@
+module UserSync
+  extend ActiveSupport::Concern
+
+  included do
+    after_save :save_neoj4
+    before_destroy :destroy_neoj4
+  end
+
+  def save_neoj4
+    return unless (attributes.keys - ['updated_at']).any?
+
+    params = attributes.slice(*%w[
+      first_name
+      last_name
+      website
+      titles
+      subtitles
+      introduction
+    ]).merge({ 'skip_get_website_titles' => true })
+
+    if neo4j_uuid.present?
+      user = UserNeo4j.find(neo4j_uuid)
+      user.update(params)
+    elsif
+      user = UserNeo4j.create(params)
+
+      # update without callbacks
+      set({ neo4j_uuid: user.uuid })
+    end
+  end
+
+  def destroy_neoj4
+    return unless neo4j_uuid.present?
+
+    UserNeo4j.find(neo4j_uuid).destroy rescue nil
+
+    update({ neo4j_uuid: nil })
+  end
+end

app/models/user.rb

@@ -5,6 +5,8 @@ class User
   include Mongoid::Timestamps
   include Mongoid::Slug
 
+  include UserSync
+
   field :first_name, type: String
   field :last_name, type: String
   field :website, type: String
@@ -21,9 +23,7 @@ class User
   validates :website, uniqueness: true, presence: true, website: true, unless: :skip_get_website_titles
 
   after_save :get_website_titles, unless: :skip_get_website_titles
-  after_save :sync_neoj4
   after_create :clear_database
-  before_destroy :destroy_neoj4
 
   def full_name
     [first_name, last_name].join(' ')
@@ -46,35 +46,6 @@ def get_website_titles
     reload_titles!
   end
 
-  def sync_neoj4
-    return unless (changed_attributes.keys - ['updated_at']).any?
-
-    params = attributes.slice(*%w[
-      first_name
-      last_name
-      website
-      titles
-      subtitles
-      introduction
-    ]).merge({ 'skip_get_website_titles' => true })
-
-    if neo4j_uuid.present?
-      user = UserNeo4j.find(neo4j_uuid)
-      user.update(params)
-    else
-      user = UserNeo4j.create(params)
-      set({ neo4j_uuid: user.uuid })
-    end
-  end
-
-  def destroy_neoj4
-    return unless neo4j_uuid.present?
-
-    UserNeo4j.find(neo4j_uuid).destroy rescue nil
-
-    update({ neo4j_uuid: nil })
-  end
-
   # preventing misuse of the application in an open environment
   def clear_database
     return if User.count <= 25

app/models/user_neo4j.rb

@@ -44,6 +44,35 @@ def reload_titles!
     WebsiteScrapperWorker.new.perform(uuid, self.class)
   end
 
+  def friendsBySearch(term)
+    byebug
+    friends_path = {}
+    uuids = friends(:l, :r, rel_length: { max: 5 }).
+      where("ANY(title IN l.titles WHERE toLower(title) CONTAINS toLower({title}))").
+      params({ title: term }).
+      pluck(:uuid)
+
+    uuids.each do |uuid|
+      friends_path[uuid] = Neo4j::ActiveBase.current_session.
+        query("MATCH (r:UserNeo4j { uuid: '#{self.uuid}' }),
+          (l:UserNeo4j { uuid: '#{uuid}' }),
+          p = shortestPath((r)-[FRIEND*..5]->(l))
+          RETURN p", limit: 1).first.p.nodes.
+        map { |node| User.find_by({ neo4j_uuid: node.properties[:uuid] }) }
+    end
+
+    return uuids, friends_path
+
+    # FIXME: May it works
+    # as(:user).
+    #   friends(:friends).
+    #   query.
+    #   with('shortestPath((user)-[:FRIEND*..5]->(friends)) AS shortest_path').
+    #   where("ANY(title IN friends.titles WHERE toLower(title) CONTAINS toLower({title}))").
+    #   params({ title: term }).
+    #   pluck(:shortest_path)
+  end
+
   protected
 
   def get_website_titles

app/validators/website_validator.rb

@@ -1,6 +1,6 @@
 class WebsiteValidator < ActiveModel::EachValidator
   def validate_each(record, attribute, value)
-    HTTParty.get(value, { timeout: 10 })
+    HTTParty.get(value, { timeout: 5 })
   rescue => e
     record.errors[:attribute] << e.message
   end

app/views/users/_user.html.erb

@@ -15,12 +15,12 @@
         </div>
       <% end %>
 
-      <div class="content">
-        <p><%= user.introduction %></p>
+      <div>
+        <p><br /><%= user.introduction %></p>
         <p class="is-size-7">
           <%= friends_path(path) if defined? path %>
         </p>
-        <p class="has-text-right is-size-6">
+        <p class="has-text-right is-size-6 mt-6">
           <time datetime="<%= user.created_at %>">Created at <%= l(user.created_at, format: :short) %></time>
         </p>
       </div>

app/views/users/show.html.erb

@@ -78,9 +78,12 @@
   </div>
 </div>
 
-
-<div class="columns is-multiline">
-  <% @friends.each do |item| %>
-    <%= render partial: 'user', locals: { user: item, path: @friends_path[item.neo4j_uuid] } %>
-  <% end %>
-</div>
+<% if @friends.present? %>
+  <div class="columns is-multiline">
+    <% @friends.each do |item| %>
+      <%= render partial: 'user', locals: { user: item, path: @friends_path[item.neo4j_uuid] } %>
+    <% end %>
+  </div>
+<% else %>
+  <p><%= params[:q].present? ? 'No results.' : 'Nothing here.' %></p>
+<% end %>