fat_free_crm 0.16.4 → 0.17.1

data/app/views/campaigns/_status.html.haml

@@ -15,7 +15,7 @@
         - if campaign.starts_on > Date.today
           = t(:starts_in, distance_of_time_in_words(Date.today, campaign.starts_on))
         - elsif campaign.starts_on < Date.today
-          %span.warn= t(:was_supposed_to_start, :time_ago => distance_of_time_in_words(campaign.starts_on, Date.today), :start_date => campaign.starts_on.strftime('%b %e, %Y'))
+          %span.warn= t(:was_supposed_to_start, time_ago: distance_of_time_in_words(campaign.starts_on, Date.today), start_date: campaign.starts_on.strftime('%b %e, %Y'))
         - else
           %span.cool #{t :starts_today}
       - else

data/app/views/campaigns/_title_bar.html.haml

@@ -1,11 +1,11 @@
 #confirm{ hidden }
 .title_tools#menu
-  = link_to_inline(:edit_campaign, edit_campaign_path(@campaign), :text => t(:edit)) + " | "
+  = link_to_inline(:edit_campaign, edit_campaign_path(@campaign), text: t(:edit)) + " | "
   = link_to_function(t(:delete) + '?', confirm_delete(@campaign))
 .title_tools#buttons
   = view_buttons
 .title#edit_campaign_title
   = h(@campaign.name)
-  = image_tag("loading.gif", :size => :thumb, :id => "loading", :style => "display: none;")
+  = image_tag("loading.gif", size: :thumb, id: "loading", style: "display: none;")
 
 %div#edit_campaign{ hidden }

data/app/views/campaigns/_top_section.html.haml

@@ -1,29 +1,29 @@
-= hook(:campaign_top_section, self, :f => f) do
+= hook(:campaign_top_section, self, f: f) do
   .section
     %table
       %tr
         %td(colspan="5")
           .label.top.req #{t :name}:
-          = f.text_field :name, :autofocus => true, :style => "width:500px"
+          = f.text_field :name, autofocus: true, style: "width:500px"
       %tr
         %td
           .label #{t :start_date}:
-          = f.text_field :starts_on, :value => f.object.starts_on, :style => "width:110px", :autocomplete => :off, :class => 'date'
+          = f.text_field :starts_on, value: f.object.starts_on, style: "width:110px", autocomplete: :off, class: 'date'
         %td= spacer
         %td
           .label #{t :end_date}:
-          = f.text_field :ends_on, :value => f.object.ends_on, :style => "width:110px", :autocomplete => :off, :class => 'date'
+          = f.text_field :ends_on, value: f.object.ends_on, style: "width:110px", autocomplete: :off, class: 'date'
         %td= spacer
         %td
           .label #{t :status}:
-          = f.select :status, Setting.unroll(:campaign_status), { :selected => (@campaign.status || "planned").to_sym }, { :style => "width:250px" }
+          = f.select :status, Setting.unroll(:campaign_status), { selected: (@campaign.status || "planned").to_sym }, { style: "width:250px", class: 'select2' }
 
       - if Setting.background_info && Setting.background_info.include?(:campaign)
         %tr
           %td(colspan="5")
-            .label= t(:background_info) << ':'
-            = f.text_area :background_info, :style =>"width:500px", :rows => 3
+            .label= t(:background_info) + ':'
+            = f.text_area :background_info, style: "width:500px", rows: 3
 
-      = render :partial => "/shared/tags", :locals => {:f => f, :span => 5}
+      = render partial: "/shared/tags", locals: {f: f, span: 5}
 
-      = hook(:campaign_top_section_bottom, self, :f => f)
+      = hook(:campaign_top_section_bottom, self, f: f)

data/app/views/campaigns/create.js.haml

@@ -6,13 +6,13 @@
   $('##{create_id}_arrow').html(crm.COLLAPSED);
   $('##{create_id}_title').html('#{ j t(entity_name.pluralize) }');
   $('##{create_id}').slideUp(250);
-  $('##{entity_name.pluralize}').prepend('#{ j render(:partial => entity_name, :collection => [ @entity ]) }');
+  $('##{entity_name.pluralize}').prepend('#{ j render(partial: entity_name, collection: [ @entity ]) }');
   $('##{dom_id(@entity)}').effect("highlight", { duration:1500 });
   = refresh_sidebar(:index, :filters)
-  $('#paginate').html('#{ j render(:partial => "shared/paginate_with_per_page") }');
+  $('#paginate').html('#{ j render(partial: "shared/paginate_with_per_page") }');
   crm.flick('empty', 'remove');
 - else
-  $('##{create_id}').html('#{ j render(:partial => "new") }');
+  $('##{create_id}').html('#{ j render(partial: "new") }');
   $('##{create_id}').effect("shake", { duration:250, distance: 6 });
   - if @entity.errors[:name].blank? and @entity.errors[:ends_on].present?
     $('#campaign_ends_on').focus();

data/app/views/campaigns/destroy.js.haml

@@ -3,4 +3,4 @@
 
 $('##{dom_id(@entity)}').css('background-color', '#ffe4e1').slideUp(250)
 = refresh_sidebar(:index, :filters)
-$('#paginate').replaceWith('#{ j render(:partial => "shared/paginate_with_per_page") }');
+$('#paginate').replaceWith('#{ j render(partial: "shared/paginate_with_per_page") }');

data/app/views/campaigns/edit.js.haml

@@ -8,14 +8,14 @@
     crm.flip_form('edit_#{entity_name}');
     crm.set_title('edit_#{entity_name}', '#{h @entity.name}');
   - else                                            # Called from index page...
-    $('##{id}').replaceWith('#{ j render(:partial => entity_name, :collection => [ @entity ]) }');
+    $('##{id}').replaceWith('#{ j render(partial: entity_name, collection: [ @entity ]) }');
 
 - else # <----------------------------------------  Show [Edit] form.
 
   - if params[:cancel].blank?                       # Called from index page...
     - if @previous                                  # Hide open [Edit] form if any.
       - if @previous.is_a?(@entity.class)
-        $('##{dom_id(@previous)}').replaceWith('#{ j render(:partial => entity_name, :collection => [ @previous ]) }');
+        $('##{dom_id(@previous)}').replaceWith('#{ j render(partial: entity_name, collection: [ @previous ]) }');
       - else
         crm.flick('#{entity_name}_#{@previous}', 'remove');
     -# Disable onMouseOver for the list item.
@@ -23,9 +23,9 @@
     -# Hide [Create] form if any.
     crm.hide_form('create_#{entity_name}');
     -# Show [Edit] form.
-    $('##{id}').html('#{ j render(:partial => "edit") }');
+    $('##{id}').html('#{ j render(partial: "edit") }');
 
   - elsif params[:cancel].false?                    # Called from title of the landing page...
-    $('#edit_#{entity_name}').html('#{ j render(:partial => "edit") }');
+    $('#edit_#{entity_name}').html('#{ j render(partial: "edit") }');
     crm.flip_form('edit_#{entity_name}');
     crm.set_title('edit_#{entity_name}', "#{t :edit} #{h @entity.name}");

data/app/views/campaigns/index.js.haml

@@ -4,8 +4,8 @@
 - if @entities.any?
   $('##{entities}').html('#{ j render(@entities) }');
 - else
-  $('##{entities}').html('#{ j render(:partial => "shared/empty") }');
+  $('##{entities}').html('#{ j render(partial: "shared/empty") }');
 
-$('#paginate').html('#{ j render(:partial => "shared/paginate_with_per_page") }');
-$('#export').html('#{ j render(:partial => "shared/export") }');
-$('#search_results_count').html('#{ j render(:plain => t('search_results_count', :count => @search_results_count)) }');
+$('#paginate').html('#{ j render(partial: "shared/paginate_with_per_page") }');
+$('#export').html('#{ j render(partial: "shared/export") }');
+$('#search_results_count').html('#{ j render(plain: t('search_results_count', count: @search_results_count)) }');

data/app/views/campaigns/index.xls.builder

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 xml.Worksheet 'ss:Name' => I18n.t(:tab_campaigns) do
   xml.Table do
     unless @campaigns.empty?
@@ -39,25 +41,25 @@ xml.Worksheet 'ss:Name' => I18n.t(:tab_campaigns) do
       # Campaign rows.
       @campaigns.each do |campaign|
         xml.Row do
-          data    = [campaign.id,
-                     campaign.user.try(:name),
-                     campaign.assignee.try(:name),
-                     campaign.name,
-                     campaign.access,
-                     campaign.status,
-                     campaign.budget,
-                     campaign.target_leads,
-                     campaign.target_conversion,
-                     campaign.target_revenue,
-                     campaign.leads_count,
-                     campaign.opportunities_count,
-                     campaign.revenue,
-                     campaign.starts_on,
-                     campaign.ends_on,
-                     campaign.objectives,
-                     campaign.background_info,
-                     campaign.created_at,
-                     campaign.updated_at]
+          data = [campaign.id,
+                  campaign.user.try(:name),
+                  campaign.assignee.try(:name),
+                  campaign.name,
+                  campaign.access,
+                  campaign.status,
+                  campaign.budget,
+                  campaign.target_leads,
+                  campaign.target_conversion,
+                  campaign.target_revenue,
+                  campaign.leads_count,
+                  campaign.opportunities_count,
+                  campaign.revenue,
+                  campaign.starts_on,
+                  campaign.ends_on,
+                  campaign.objectives,
+                  campaign.background_info,
+                  campaign.created_at,
+                  campaign.updated_at]
 
           # Append custom field values.
           Campaign.fields.each do |field|
@@ -67,7 +69,7 @@ xml.Worksheet 'ss:Name' => I18n.t(:tab_campaigns) do
           data.each do |value|
             xml.Cell do
               xml.Data value,
-                       'ss:Type' => "#{value.respond_to?(:abs) ? 'Number' : 'String'}"
+                       'ss:Type' => (value.respond_to?(:abs) ? 'Number' : 'String').to_s
             end
           end
         end

data/app/views/campaigns/new.js.haml

@@ -5,7 +5,7 @@ crm.flick('empty', 'toggle');
 crm.flip_form('#{create_id}');
 
 - unless params[:cancel].true?
-  $('##{create_id}').html('#{ j render(:partial => "new") }');
+  $('##{create_id}').html('#{ j render(partial: "new") }');
   crm.set_title('#{create_id}', '#{ j t(create_id) }');
 - else
   crm.set_title('#{create_id}', '#{ j t(entity_name.pluralize) }');

data/app/views/campaigns/show.html.haml

@@ -1,26 +1,26 @@
 - if (template = template_for_current_view)
-  = render(template, :campaign => @campaign)
+  = render(template, campaign: @campaign)
 - else
 
   -# default view
 
   = styles_for :task, :lead, :opportunity
 
-  = render 'campaigns/title_bar', :campaign => @campaign
+  = render 'campaigns/title_bar', campaign: @campaign
 
-  = render "comments/new", :commentable => @campaign
-  = render :partial => "shared/timeline", :collection => @timeline
+  = render "comments/new", commentable: @campaign
+  = render partial: "shared/timeline", collection: @timeline
 
   = hook(:show_campaign_bottom, self, {entity: @campaign}) do
 
-    = render "tasks/tasks", :object => @campaign
+    = render "tasks/tasks", object: @campaign
 
-    = render "leads/leads", :object => @campaign
+    = render "leads/leads", object: @campaign
 
-    = render "opportunities/opportunities", :object => @campaign
+    = render "opportunities/opportunities", object: @campaign
 
-    = render "versions/versions", :object => @campaign
+    = render "versions/versions", object: @campaign
 
     = load_select_popups_for(@campaign, :tasks, :leads, :opportunities)
 
-  #export= render "shared/export", :action => :show
+  #export= render "shared/export", action: :show

data/app/views/campaigns/show.js.haml

@@ -1,5 +1,5 @@
 - entity_name = controller.controller_name.singularize.underscore #account
 - @entity = instance_variable_get("@#{entity_name}")
 
-$('#main').html('#{ j (render :template => "#{entity_name.pluralize}/show.html", entity_name => @entity) }');
+$('#main').html('#{ j (render template: "#{entity_name.pluralize}/show.html", entity_name => @entity) }');
 = raw generate_js_for_popups(@entity, :tasks, :leads, :opportunities)

data/app/views/campaigns/update.js.haml

@@ -8,11 +8,11 @@
     crm.set_title('edit_#{entity_name}', '#{h @entity.name}');
     = refresh_sidebar(:show, :summary)
   - else
-    $('##{id}').replaceWith('#{ j render(:partial => entity_name, :collection => [ @entity ]) }');
+    $('##{id}').replaceWith('#{ j render(partial: entity_name, collection: [ @entity ]) }');
     $('##{id}').effect("highlight", { duration:1500 });
     = refresh_sidebar(:index, :filters)
 - else
-  $('##{id}').html('#{ j render(:partial => "edit") }');
+  $('##{id}').html('#{ j render(partial: "edit") }');
   $('##{id}').effect("shake", { duration:250, distance: 6 });
   - if @campaign.errors[:name].blank? and @campaign.errors[:ends_on].present?
     $('#campaign_ends_on').focus();

data/app/views/comments/_comment.html.haml

@@ -1,11 +1,11 @@
-- truncated = simple_format(truncate(comment.comment.gsub("\n", " "), :length => 125))
+- truncated = simple_format(truncate(comment.comment.gsub("\n", " "), length: 125))
 - formatted = defined?(RedCloth) ? RedCloth.new(comment.comment).to_html : simple_format(comment.comment)
 - collapsable = formatted.size > 150
 - commentable = comment.commentable
 
 %li.comment.highlight[comment]
   .body
-    = link_to avatar_for(comment.user, :size => :small), user_path(comment.user)
+    = link_to avatar_for(comment.user, size: :small), user_path(comment.user)
     %ul.tools
       - if can?(:update, commentable)
         %li= link_to_edit(comment)
@@ -17,13 +17,13 @@
       = t(:added_note, value: timeago(comment.created_at) ).html_safe
       - if collapsable && can?(:read, commentable)
         = " | "
-        = link_to_function(comment.collapsed? ? t(:more) : t(:less), "crm.flip_note_or_email(this, '#{t(:more)}', '#{t(:less)}')", :class => "toggle")
+        = link_to_function(comment.collapsed? ? t(:more) : t(:less), "crm.flip_note_or_email(this, '#{t(:more)}', '#{t(:less)}')", class: "toggle")
 
     - if can?(:read, commentable)
       - if collapsable
-        %dt{ hidden_if(comment.expanded?), :id => dom_id(comment, :truncated) }
+        %dt{ hidden_if(comment.expanded?), id: dom_id(comment, :truncated) }
           = truncated
-        %dt.textile{ hidden_if(comment.collapsed?), :id => dom_id(comment, :formatted) }
+        %dt.textile{ hidden_if(comment.collapsed?), id: dom_id(comment, :formatted) }
           = sanitize auto_link(formatted)
       - else
         %dt.textile= sanitize auto_link(formatted)

data/app/views/comments/_edit.html.haml

@@ -1,11 +1,11 @@
 - class_name = commentable.class.name.underscore
-.comment{:style => "margin: 5px 0px 0px 0px"}
-  = link_to avatar_for(current_user, :size => :small), user_path(current_user)
-  = form_for(@comment, :remote => true) do |f|
+.comment{style: "margin: 5px 0px 0px 0px"}
+  = link_to avatar_for(current_user, size: :small), user_path(current_user)
+  = form_for(@comment, remote: true) do |f|
     = hidden_field_tag "comment[commentable_id]", commentable.id
     = hidden_field_tag "comment[commentable_type]", class_name.classify
-    = f.text_area :comment, :id => dom_id(@comment, :text)
+    = f.text_area :comment, id: dom_id(@comment, :text)
     .buttons
       = f.submit t(:save_note)
       #{t :or}
-      = link_to(t(:cancel), edit_comment_path(@comment, "#{class_name}_id" => commentable.id, :cancel => true), :remote => true)
+      = link_to(t(:cancel), edit_comment_path(@comment, "#{class_name}_id" => commentable.id, cancel: true), remote: true)

data/app/views/comments/_new.html.haml

@@ -1,7 +1,7 @@
 - class_name = commentable.class.name.underscore
 - id_prefix = "#{class_name}_#{commentable.id}"
 - @comment = @comment || Comment.new
-.comment.highlight.new_comment{ :id => "#{id_prefix}_comment_new" }
+.comment.highlight.new_comment{ id: "#{id_prefix}_comment_new" }
   - subscribed_users = commentable.subscribed_users.map{|uid| User.find_by_id(uid) unless uid == current_user.id }.compact
   - if notification_emails_configured? && subscribed_users.any?
     = t(:following_users_will_be_notified) << ":"
@@ -10,26 +10,26 @@
 
   -# Two hidden fields store the IDs of notes and emails shown for the asset. These IDs are used
   -# by [Expand/Collapse All]. The contents gets updated by actions such as [Add] or [Delete].
-  = hidden_field_tag :shown_notes, "#{commentable.comment_ids.join(',')}", :id => "#{id_prefix}_shown_notes"
-  = hidden_field_tag :shown_emails, "#{commentable.email_ids.join(',')}", :id => "#{id_prefix}_shown_emails"
+  = hidden_field_tag :shown_notes, "#{commentable.comment_ids.join(',')}", id: "#{id_prefix}_shown_notes"
+  = hidden_field_tag :shown_emails, "#{commentable.email_ids.join(',')}", id: "#{id_prefix}_shown_emails"
 
-  = link_to avatar_for(current_user, :size => :small), user_path(current_user)
+  = link_to avatar_for(current_user, size: :small), user_path(current_user)
   .tools
     = link_to_function(t(:expand_all), "crm.flip_notes_and_emails('#{t(:expanded)}', '#{t(:more)}', '#{t(:less)}', '#{id_prefix}')") << " | "
     = link_to_function(t(:collapse_all), "crm.flip_notes_and_emails('#{t(:collapsed)}', '#{t(:more)}', '#{t(:less)}', '#{id_prefix}')")
 
-  %div{ {:id => "#{id_prefix}_post"}.merge(hidden_if(true))}
-    = form_for(@comment, :remote => true, :html => {:id => "#{id_prefix}_new_comment"}) do |f|
-      = hidden_field_tag "comment[commentable_id]", commentable.id, :id => "#{id_prefix}_comment_commentable_id"
-      = hidden_field_tag "comment[commentable_type]", class_name.classify, :id => "#{id_prefix}_comment_commentable_type"
-      = f.text_area :comment, :id => "#{id_prefix}_comment_comment"
+  %div{ {id: "#{id_prefix}_post"}.merge(hidden_if(true))}
+    = form_for(@comment, remote: true, html: {id: "#{id_prefix}_new_comment"}) do |f|
+      = hidden_field_tag "comment[commentable_id]", commentable.id, id: "#{id_prefix}_comment_commentable_id"
+      = hidden_field_tag "comment[commentable_type]", class_name.classify, id: "#{id_prefix}_comment_commentable_type"
+      = f.text_area :comment, id: "#{id_prefix}_comment_comment"
       .buttons
-        = image_tag("loading.gif", :size => :thumb, :class => "spinner", :style => "display: none;")
-        = f.submit t(:add_note), :id => "#{id_prefix}_comment_submit"
+        = image_tag("loading.gif", size: :thumb, class: "spinner", style: "display: none;")
+        = f.submit t(:add_note), id: "#{id_prefix}_comment_submit"
         #{t :or}
-        = link_to(t(:cancel), '#', :class => 'cancel')
-  %div{ {:id => "#{id_prefix}_ask"}.merge(hidden_if(false))}
-    = text_field_tag :post_new_note, t(:add_note_help), :id => "#{id_prefix}_post_new_note"
+        = link_to(t(:cancel), '#', class: 'cancel')
+  %div{ {id: "#{id_prefix}_ask"}.merge(hidden_if(false))}
+    = text_field_tag :post_new_note, t(:add_note_help), id: "#{id_prefix}_post_new_note"
 
   - if notification_emails_configured?
-    = render :partial => "comments/subscription_links", :locals => {:entity => commentable}
+    = render partial: "comments/subscription_links", locals: {entity: commentable}

data/app/views/comments/_subscription_links.html.haml

@@ -8,6 +8,6 @@
 - else
   - link_text, action = [t(:subscribe_via_email), "subscribe"]
 
-%div{:id => "#{id_prefix}_subscribe", :class => "comment_subscriptions"}
-  = image_tag "notifications.png", :title => t(:notifications_tooltip)
-  = link_to link_text, url_for(:controller => class_name.pluralize, :action => action, :id => entity.id), :remote => true, :method => :post
+%div{id: "#{id_prefix}_subscribe", class: "comment_subscriptions"}
+  = image_tag "notifications.png", title: t(:notifications_tooltip)
+  = link_to link_text, url_for(controller: class_name.pluralize, action: action, id: entity.id), remote: true, method: :post

data/app/views/comments/create.js.haml

@@ -2,7 +2,7 @@
 - id_prefix = "#{class_name}_#{@comment.commentable.id}"
 
 - if @comment.valid?
-  $('##{id_prefix}_comment_new').after('#{ j (render :partial => "comment", :locals => { :comment => @comment }) }');
+  $('##{id_prefix}_comment_new').after('#{ j (render partial: "comment", locals: { comment: @comment }) }');
   $('##{dom_id(@comment)}').effect("highlight", { duration:1500 });
   $('##{id_prefix}_post').hide();
   $('##{id_prefix}_ask').show();

data/app/views/comments/edit.js.haml

@@ -1,11 +1,11 @@
 - id = dom_id(@comment)
 
 - if params[:cancel].true?     # <----------------- Hide [Edit Comment]
-  $('##{id}').replaceWith('#{ j render(:partial => "comment", :collection => [ @comment ]) }');
+  $('##{id}').replaceWith('#{ j render(partial: "comment", collection: [ @comment ]) }');
 
 - else # <----------------------------------------- Show [Edit Comment] form.
 
   -# Disable onMouseOver for the list item.
   crm.highlight_off('#{id}');
-  $('##{id}').html('#{ j render(:partial => "edit", :locals => { :commentable => @comment.commentable }) }');
+  $('##{id}').html('#{ j render(partial: "edit", locals: { commentable: @comment.commentable }) }');
   $('##{dom_id(@comment, :text)}').focus();

data/app/views/comments/update.js.haml

@@ -1,9 +1,9 @@
 - id = dom_id(@comment)
 
 - if @comment.errors.empty?
-  $('##{id}').replaceWith('#{ j render(:partial => "comment", :collection => [ @comment ]) }');
+  $('##{id}').replaceWith('#{ j render(partial: "comment", collection: [ @comment ]) }');
   $('##{id}').effect("highlight", { duration:1500 });
 - else
-  $('##{id}').html('#{ j render(:partial => "edit") }');
+  $('##{id}').html('#{ j render(partial: "edit") }');
   $('##{id}').effect("shake", { duration:250, distance: 6 });
   $('#comment_comment').focus();

data/app/views/contacts/_contact.html.haml

@@ -1,5 +1,5 @@
-- if (params['controller'] == 'contacts') and (template = template_for_current_view).present?
-  = render(template, :contact => contact)
+- if (controller.controller_name == 'contacts') and (template = template_for_current_view).present?
+  = render(template, contact: contact)
 - else
   -# default view
-  = render 'contacts/index_long', :contact => contact
+  = render 'contacts/index_long', contact: contact

data/app/views/contacts/_contacts.html.haml

@@ -1,7 +1,7 @@
 
 = section(object, :contacts)
 .list#contacts
-  - contacts = object.contacts.order('updated_at desc').paginate(:page => 1, :per_page => 20)
-  = render :partial => "contacts/contact", :collection => contacts
+  - contacts = object.contacts.order('updated_at desc').paginate(page: 1, per_page: 20)
+  = render partial: "contacts/contact", collection: contacts
 
 = paginate( collection: contacts, id: 'contacts_pagination', params: { action: 'contacts' } )

data/app/views/contacts/_edit.html.haml

@@ -1,19 +1,19 @@
 .remote
-  = simple_form_for(@contact, :html => one_submit_only, :remote => true) do |f|
+  = simple_form_for(@contact, html: one_submit_only, remote: true) do |f|
     = link_to_close edit_contact_path(@contact)
     = f.hidden_field :user_id
 
-    = f.error_messages :object_name => t('contact')
+    = f.error_messages object_name: t('contact')
 
-    = render "contacts/top_section", :f => f, :edit => true
-    = render "fields/edit_custom_field_group", :f => f, :edit => true
-    = render "contacts/extra",       :f => f, :edit => true
-    = render "contacts/web",         :f => f, :edit => true
-    = render "fields/groups",        :f => f, :edit => true
-    = render "entities/permissions", :f => f, :edit => true, :entity => @contact
+    = render "contacts/top_section", f: f, edit: true
+    = render "fields/edit_custom_field_group", f: f, edit: true
+    = render "contacts/extra",       f: f, edit: true
+    = render "contacts/web",         f: f, edit: true
+    = render "fields/groups",        f: f, edit: true
+    = render "entities/permissions", f: f, edit: true, entity: @contact
     = hook(:entity_form, self, {f: f, entity: @contact})
 
     .buttonbar
-      = f.submit t(:save_contact), :onclick => "crm.save_contact()"
+      = f.submit t(:save_contact), onclick: "crm.save_contact()"
       #{t :or}
       = link_to_cancel edit_contact_path(@contact)