Merge pull request #626 from coopdevs/feat/tags-postcode-members

feat: tags and postal code attributes

Author: sseerrggii

app/admin/user.rb

@@ -30,13 +30,15 @@
   filter :email
   filter :username
   filter :phone
+  filter :postcode
 
   form do |f|
     f.semantic_errors *f.object.errors.keys
     f.inputs do
       f.input :username
       f.input :email
       f.input :phone
+      f.input :postcode
       f.input :gender, as: :select, collection: User::GENDERS
       f.input :identity_document
     end

app/assets/javascripts/application/tags.js

@@ -7,6 +7,10 @@ $(function() {
     loadTags('inquiry');
   });
 
+  $(".switch_member-js").on("click", function() {
+    loadTags('user');
+  });
+
   function loadTags(type){
     $.get({
       url: `/tags/alpha_grouped_index.js?post_type=${type}`,
@@ -28,7 +32,7 @@ $(function() {
     ajax: {
       url: '/tags.json',
       data: function(params) {
-        return { term: params.term };
+        return { term: params.term, model: $(this).data("model") };
       },
       processResults: function(data, params) {
         // parse the data into the format expected by Select2

app/assets/stylesheets/application/member-card.scss

@@ -37,6 +37,13 @@
         }
       }
 
+    &__tags {
+      a {
+        margin-left: 4px;
+        color: white;
+      }
+    }
+
       &__activity {
         font-size: 14px;
         color: #78adb9;

app/controllers/tags_controller.rb

@@ -2,27 +2,30 @@ class TagsController < ApplicationController
   before_action :authenticate_user!, :member_should_be_active
 
   def index
-    posts = Post.by_organization(current_organization)
-    @all_tags = posts.find_like_tag(params[:term])
+    model = params[:model].classify.constantize
+    posts = model.by_organization(current_organization)
+    @tags = posts.find_like_tag(params[:term])
 
-    render json: @all_tags
+    render json: @tags
   end
 
   def alpha_grouped_index
     redirect_to users_path && return unless current_organization
 
     post_type = params[:post_type] || "offer"
-    @alpha_tags = case post_type
-                  when "offer" then Offer
-                  when "inquiry" then Inquiry
-                  end.by_organization(current_organization).
-                  active.of_active_members.
-                  alphabetical_grouped_tags
+    @tags = case post_type
+            when "offer"
+              Offer.by_organization(current_organization).active.of_active_members
+            when "inquiry"
+              Inquiry.by_organization(current_organization).active.of_active_members
+            when "user"
+              Member.by_organization(current_organization).active
+            end.alphabetical_grouped_tags
 
     respond_to do |format|
       format.html
       format.js do
-        render partial: "grouped_index", locals: { alpha_tags: @alpha_tags, post_type: post_type }
+        render partial: "grouped_index", locals: { alpha_tags: @tags, post_type: post_type }
       end
     end
   end

app/controllers/users_controller.rb

@@ -1,8 +1,13 @@
 class UsersController < ApplicationController
   before_action :authenticate_user!, :member_should_be_active
 
+  has_scope :tagged_with, as: :tag
+
   def index
-    search_and_load_members current_organization.members.active, { s: 'user_last_sign_in_at DESC' }
+    members = current_organization.members.active
+    members = apply_scopes(members)
+
+    search_and_load_members members, { s: 'user_last_sign_in_at DESC' }
   end
 
   def manage
@@ -37,9 +42,12 @@ def create
 
     if @user.persisted?
       @user.tune_after_persisted(current_organization)
+      @user.add_tags(current_organization, params[:tag_list] || [])
+
       redirect_to_after_create
     else
       @user.email = "" if empty_email
+
       render action: "new"
     end
   end
@@ -49,6 +57,8 @@ def update
     authorize @user
 
     if @user.update(user_params)
+      @user.add_tags(current_organization, params[:tag_list] || [])
+
       redirect_to @user
     else
       render action: :edit, status: :unprocessable_entity
@@ -76,7 +86,7 @@ def scoped_users
 
   def user_params
     fields_to_permit = %w"gender username email date_of_birth phone
-                          alt_phone active description notifications push_notifications"
+                          alt_phone active description notifications push_notifications postcode"
     fields_to_permit += %w"admin registration_number
                            registration_date" if admin?
     fields_to_permit += %w"organization_id superadmin" if superadmin?

app/decorators/member_decorator.rb

@@ -1,5 +1,5 @@
 class MemberDecorator < ViewModel
-  delegate :user, :member_uid, :active?, to: :object
+  delegate :user, :member_uid, :tags, :active?, to: :object
   delegate :phone, :alt_phone, :username, :description, :last_sign_in_at, to: :user
 
   def manager?

app/models/concerns/taggable.rb

@@ -6,7 +6,7 @@ module Taggable
   extend ActiveSupport::Concern
 
   included do
-    scope :tagged_with, ->(tag) { where("? = ANY (tags)", tag) }
+    scope :tagged_with, ->(tag) { where("? = ANY (#{table_name}.tags)", tag) }
   end
 
   def tag_list

app/models/member.rb

@@ -1,8 +1,22 @@
 class Member < ApplicationRecord
+  include Taggable
+
   # Cast the member_uid integer to a string to allow pg ILIKE search (from Ransack *_contains)
   ransacker :member_uid_search do
     Arel.sql("member_uid::text")
   end
+  # Convert array of tags to string
+  ransacker :member_tags do
+    Arel.sql("array_to_string(tags, ',')")
+  end
+  ransack_alias :member_search, %w(
+    user_username
+    user_email
+    user_phone
+    user_alt_phone
+    member_uid_search
+    member_tags
+  ).join("_or_")
 
   belongs_to :user
   belongs_to :organization
@@ -15,6 +29,7 @@ class Member < ApplicationRecord
 
   scope :by_month, -> (month) { where(created_at: month.beginning_of_month..month.end_of_month) }
   scope :active, -> { where active: true }
+  scope :by_organization, ->(org) { where(organization_id: org) if org }
 
   validates :organization_id, presence: true
   validates :member_uid,

app/models/user.rb

@@ -110,6 +110,11 @@ def tune_after_persisted(organization)
     save
   end
 
+  def add_tags(organization, tag_list)
+    member = as_member_of(organization)
+    member.update(tag_list: tag_list)
+  end
+
   def has_valid_email?
     !email.include? "example.com"
   end

app/views/shared/_post_form.html.erb

@@ -34,7 +34,7 @@
                  f.object.tags || [],
                  { selected: f.object.tags },
                  multiple: true,
-                 data: { placeholder: t('application.tips.entertag') },
+                 data: { placeholder: t("application.tips.entertag"), model: "post" },
                  id: "tags-js",
                  class: "form-control" %>
   </div>