kaui 3.0.1 → 3.0.2

data/app/controllers/kaui/home_controller.rb

@@ -52,15 +52,23 @@ module Kaui
     end
 
     def invoice_search(search_query, search_by = nil, fast = 0, options = {})
-      if search_by == 'ID'
+      case search_by
+      when 'ID'
         begin
           invoice = Kaui::Invoice.find_by_id(search_query, 'NONE', options)
           redirect_to account_invoice_path(invoice.account_id, invoice.invoice_id) and return
         rescue KillBillClient::API::NotFound => _e
           search_error("No invoice matches \"#{search_query}\"")
         end
-      elsif search_by == 'EXTERNAL_KEY'
-        unsupported_external_key_search('INVOICE')
+      when 'EXTERNAL_KEY'
+        unsupported_search_field('INVOICE', search_by)
+      when 'NUMBER'
+        begin
+          invoice = Kaui::Invoice.find_by_number(search_query, 'NONE', options)
+          redirect_to account_invoice_path(invoice.account_id, invoice.invoice_id) and return
+        rescue KillBillClient::API::NotFound, KillBillClient::API::BadRequest => _e
+          search_error("No invoice matches \"#{search_query}\"")
+        end
       else
         invoice = Kaui::Invoice.list_or_search(search_query, 0, 1, options).first
         if invoice.blank?
@@ -157,7 +165,7 @@ module Kaui
           search_error("No credit matches \"#{search_query}\"")
         end
       else
-        unsupported_external_key_search('CREDIT')
+        unsupported_search_field('CREDIT', search_by)
       end
     end
 
@@ -170,7 +178,7 @@ module Kaui
           redirect_to custom_fields_path(q: search_query, fast:)
         end
       else
-        unsupported_external_key_search('CUSTOM FIELD')
+        unsupported_search_field('CUSTOM FIELD', search_by)
       end
     end
 
@@ -183,7 +191,7 @@ module Kaui
           search_error("No invoice payment matches \"#{search_query}\"")
         end
       else
-        unsupported_external_key_search('INVOICE PAYMENT')
+        unsupported_search_field('INVOICE PAYMENT', search_by)
       end
     end
 
@@ -196,7 +204,7 @@ module Kaui
           search_error("No subscription matches \"#{search_query}\"")
         end
       else
-        unsupported_external_key_search('SUBSCRIPTION')
+        unsupported_search_field('SUBSCRIPTION', search_by)
       end
     end
 
@@ -209,7 +217,7 @@ module Kaui
           redirect_to tags_path(q: search_query, fast:)
         end
       else
-        unsupported_external_key_search('TAG')
+        unsupported_search_field('TAG', search_by)
       end
     end
 
@@ -222,7 +230,7 @@ module Kaui
           search_error("No tag definition matches \"#{search_query}\"")
         end
       elsif search_by == 'EXTERNAL_KEY'
-        unsupported_external_key_search('TAG DEFINITION')
+        unsupported_search_field('TAG DEFINITION', search_by)
       else
         tag_definition = Kaui::TagDefinition.find_by_name(search_query, 'NONE', options)
         if tag_definition.blank?
@@ -238,8 +246,9 @@ module Kaui
       end
     end
 
-    def unsupported_external_key_search(object_type)
-      search_error("\"#{object_type}\": Search by \"EXTERNAL KEY\" is not supported.")
+    def unsupported_search_field(object_type, object_field)
+      field_name = object_field.gsub('_', ' ')
+      search_error("\"#{object_type}\": Search by \"#{field_name}\" is not supported.")
     end
 
     def search_error(message)
@@ -267,7 +276,7 @@ module Kaui
         '0'
       end
 
-      search_error("\"#{search_by}\" is not a valid search by value") if !search_by.blank? && !%w[ID EXTERNAL_KEY].include?(search_by)
+      search_error("\"#{search_by}\" is not a valid search by value") if !search_by.blank? && !search_by.in?(Kaui::ObjectHelper::ADVANCED_SEARCH_OBJECT_FIELDS)
 
       [object_type, search_for, search_by, fast]
     end

data/app/controllers/kaui/invoice_tags_controller.rb

@@ -18,7 +18,7 @@ module Kaui
       invoice_id = params.require(:invoice_id)
 
       tags = []
-      params.each do |tag, _tag_name|
+      params.each_key do |tag|
         tag_info = tag.split('_')
         next if (tag_info.size != 2) || (tag_info[0] != 'tag')
 

data/app/controllers/kaui/invoices_controller.rb

@@ -39,7 +39,7 @@ module Kaui
         end
         formatter = lambda do |invoice|
           row = [view_context.link_to(invoice.invoice_number, view_context.url_for(controller: :invoices, action: :show, account_id: invoice.account_id, id: invoice.invoice_id))]
-          row += Kaui.invoice_search_columns.call(invoice, view_context)[1]
+          row += Kaui.invoice_search_columns.call(invoice, view_context, cached_options_for_klient)[1]
           row
         end
       else
@@ -147,6 +147,11 @@ module Kaui
       redirect_to account_invoice_path(invoice.account_id, invoice.invoice_id)
     end
 
+    def restful_show_by_number
+      invoice = Kaui::Invoice.find_by_number(params.require(:number), 'NONE', options_for_klient)
+      redirect_to account_invoice_path(invoice.account_id, invoice.invoice_id)
+    end
+
     def show_html
       render html: Kaui::Invoice.as_html(params.require(:id), options_for_klient).html_safe
     end

data/app/controllers/kaui/payment_methods_controller.rb

@@ -47,7 +47,7 @@ module Kaui
       }
 
       @plugin_properties = begin
-        params[:plugin_properties].values.reject { |item| (item['value'].blank? || item['key'].blank?) }
+        params[:plugin_properties].values.reject { |item| item['value'].blank? || item['key'].blank? }
       rescue StandardError
         @plugin_properties = nil
       end

data/app/controllers/kaui/subscriptions_controller.rb

@@ -193,10 +193,11 @@ module Kaui
     def update_tags
       subscription_id = params.require(:id)
       subscription = Kaui::Subscription.find_by_id(subscription_id, options_for_klient)
-
       tags = []
-      params.each do |tag|
-        tag_info = tag.split('_')
+      params.each_key do |key|
+        next unless key.include? 'tag'
+
+        tag_info = key.split('_')
         next if (tag_info.size != 2) || (tag_info[0] != 'tag')
 
         tags << tag_info[1]
@@ -210,7 +211,7 @@ module Kaui
 
     def lookup_bundle_and_plan_details(subscription, base_product_name = nil)
       if subscription.product_category == 'ADD_ON'
-        bundle = Kaui::Bundle.find_by_id(@subscription.bundle_id, options_for_klient)
+        bundle = Kaui::Bundle.find_by_id(subscription.bundle_id, options_for_klient)
         if base_product_name.blank?
           bundle.subscriptions.each do |sub|
             if sub.product_category == 'BASE'
@@ -222,17 +223,17 @@ module Kaui
         plans_details = Kaui::Catalog.available_addons(base_product_name, subscription.account_id, options_for_klient)
       else
         bundle = nil
-        plans_details = catalog_plans(subscription.product_category == 'BASE' ? nil : subscription.product_category)
+        plans_details = catalog_plans(subscription.product_category == 'BASE' ? nil : subscription.product_category, subscription.account_id)
       end
       [bundle, plans_details]
     end
 
-    def catalog_plans(product_category = nil)
-      return Kaui::Catalog.available_base_plans(@subscription.account_id, options_for_klient) if product_category == 'BASE'
+    def catalog_plans(product_category = nil, account_id = nil)
+      return Kaui::Catalog.available_base_plans(account_id, options_for_klient) if product_category == 'BASE'
 
       options = options_for_klient
 
-      catalog = Kaui::Catalog.get_tenant_catalog_json(DateTime.now.to_s, @subscription.account_id, options)
+      catalog = Kaui::Catalog.get_tenant_catalog_json(DateTime.now.to_s, account_id, options)
       return [] if catalog.blank?
 
       plans = []

data/app/controllers/kaui/transactions_controller.rb

@@ -26,7 +26,7 @@ module Kaui
     def create
       transaction = Kaui::Transaction.new(params[:transaction].delete_if { |_key, value| value.blank? })
 
-      plugin_properties = params[:plugin_properties].values.reject { |item| (item['value'].blank? || item['key'].blank?) } unless params[:plugin_properties].blank?
+      plugin_properties = params[:plugin_properties].values.reject { |item| item['value'].blank? || item['key'].blank? } unless params[:plugin_properties].blank?
       unless plugin_properties.blank?
         plugin_properties.map! do |property|
           KillBillClient::Model::PluginPropertyAttributes.new(property)

data/app/helpers/kaui/object_helper.rb

@@ -2,6 +2,13 @@
 
 module Kaui
   module ObjectHelper
+    ADVANCED_SEARCH_OBJECT_FIELDS = %w[ID EXTERNAL_KEY NUMBER].freeze
+    ADVANCED_SEARCH_OBJECT_FIELDS_MAP = {
+      # ID is supported by all object types, hence not listed.
+      EXTERNAL_KEY: %w[ACCOUNT PAYMENT TRANSACTION BUNDLE],
+      NUMBER: %w[INVOICE]
+    }.freeze
+
     # Because we don't have access to the account_id, we use the restful_show routes
     def url_for_object(object_id, object_type)
       case object_type
@@ -27,7 +34,15 @@ module Kaui
     end
 
     def object_types_for_advanced_search
-      %i[ACCOUNT BUNDLE INVOICE CREDIT CUSTOM_FIELD INVOICE_PAYMENT INVOICE PAYMENT SUBSCRIPTION TRANSACTION TAG TAG_DEFINITION]
+      %i[ACCOUNT BUNDLE INVOICE CREDIT CUSTOM_FIELD INVOICE_PAYMENT PAYMENT SUBSCRIPTION TRANSACTION TAG TAG_DEFINITION]
+    end
+
+    def object_fields_for_advanced_search
+      [' '] + ADVANCED_SEARCH_OBJECT_FIELDS
+    end
+
+    def advanced_search_object_fields_map
+      ADVANCED_SEARCH_OBJECT_FIELDS_MAP
     end
   end
 end

data/app/models/kaui/admin_tenant.rb

@@ -33,7 +33,7 @@ module Kaui
 
         raw_tenant_config.each_with_object({}) do |e, hsh|
           # Strip prefix '/PLUGIN_CONFIG_'
-          plugin_name = e.key.gsub!(/PLUGIN_CONFIG_/, '')
+          plugin_name = e.key.gsub!('PLUGIN_CONFIG_', '')
 
           # Construct simple hash with one property (first value)
           hsh[plugin_name] = e.values[0]

data/app/models/kaui/catalog.rb

@@ -35,8 +35,8 @@ module Kaui
         raise e
       end
 
-      def get_catalog_json(latest, requested_date, options)
-        catalogs = get_tenant_catalog_json(requested_date, nil, options)
+      def get_catalog_json(latest, requested_date, account_id, options)
+        catalogs = get_tenant_catalog_json(requested_date, account_id, options)
         return catalogs.length.positive? ? catalogs[catalogs.length - 1] : nil if latest
 
         # Order by latest

data/app/models/kaui/invoice_payment.rb

@@ -21,7 +21,7 @@ module Kaui
           result.send("#{attr}=", raw_payment.send(attr))
         end
         # Use  Kaui::Transaction to benefit from additional fields (e.g next_retry_date)
-        original_transactions = (result.transactions || [])
+        original_transactions = result.transactions || []
         result.transactions = []
         original_transactions.each do |transaction|
           new_transaction = Kaui::Transaction.new

data/app/models/kaui/overdue.rb

@@ -19,7 +19,7 @@ module Kaui
             state.subscription_cancellation_policy = nil
           else
             state.is_disable_entitlement = true
-            state.subscription_cancellation_policy = state_model['subscription_cancellation_policy'].blank? ? :NONE : state_model['subscription_cancellation_policy'].to_s.gsub!(/POLICY_/, '')
+            state.subscription_cancellation_policy = state_model['subscription_cancellation_policy'].blank? ? :NONE : state_model['subscription_cancellation_policy'].to_s.gsub!('POLICY_', '')
           end
 
           if state_model['condition']
@@ -60,7 +60,7 @@ module Kaui
             attr_accessor :subscription_cancellation
           end
           state.subscription_cancellation = if state.is_disable_entitlement
-                                              state.subscription_cancellation_policy ? "POLICY_#{state.subscription_cancellation_policy}".to_sym : :NONE
+                                              state.subscription_cancellation_policy ? :"POLICY_#{state.subscription_cancellation_policy}" : :NONE
                                             else
                                               :NONE
                                             end

data/app/models/kaui/tag.rb

@@ -13,12 +13,12 @@ module Kaui
     class << self
       %i[account bundle subscription invoice].each do |model|
         define_method "all_for_#{model}" do |model_id, included_deleted, audit, options|
-          instance = Kaui.const_get(model.to_s.camelize).new("#{model}_id".to_sym => model_id)
+          instance = Kaui.const_get(model.to_s.camelize).new("#{model}_id": model_id)
           instance.tags(included_deleted, audit, options)
         end
 
         define_method "set_for_#{model}" do |model_id, tags, user, reason, comment, options|
-          instance = Kaui.const_get(model.to_s.camelize).new("#{model}_id".to_sym => model_id)
+          instance = Kaui.const_get(model.to_s.camelize).new("#{model}_id": model_id)
           instance.set_tags(tags, user, reason, comment, options)
         end
       end

data/app/models/kaui/tag_definition.rb

@@ -25,7 +25,7 @@ module Kaui
 
     ALL_OBJECT_TYPES.each do |object_type|
       define_singleton_method "all_for_#{object_type.downcase}" do |options_for_klient|
-        (all('NONE', options_for_klient).delete_if { |tag_definition| !tag_definition.applicable_object_types.include?(object_type) || tag_definition.is_control_tag }).sort
+        (all('NONE', options_for_klient).delete_if { |tag_definition| !tag_definition.applicable_object_types.include?(object_type) }).sort
       end
     end
 

data/app/views/kaui/accounts/_billing_info.html.erb

@@ -110,7 +110,7 @@
               <% end %>
             </table>
 
-            <% if can? :trigger, Kaui::Invoice %>
+            <% if (can? :trigger, Kaui::Invoice) || (can? :dry_run, Kaui::Invoice) %>
                 <div style="padding-bottom: 50px;">
                   <%= form_tag kaui_engine.trigger_invoice_path(params.to_h), :method => :post do %>
                       <div class="form-group">
@@ -118,13 +118,11 @@
                         <div class="col-sm-3">
                           <input class="form-control" id="target_date" name="target_date" type="text" data-provide="datepicker" data-date-format="yyyy-mm-dd" data-date-today-highlight="true">
                         </div>
-                        <% if can? :dry_run, Kaui::Invoice %>
                           <div style="width: 14.5%; float: left; padding: 0;">
                             <%= label_tag :dry_run do %>
-                                <%= check_box_tag :dry_run, '1', true %>&nbsp;<small>Dry-run</small>
+                                <%= check_box_tag :dry_run, '1', true, :disabled => !can?(:trigger, Kaui::Invoice) %>&nbsp;<small>Dry-run</small>
                             <% end %>
                           </div>
-                        <% end %>
                         <div class="col-xs-1" style="padding: 0;">
                           <%= button_tag '<i class="fa fa-magic"></i>'.html_safe, :class => 'btn btn-xs' %>
                         </div>

data/app/views/kaui/accounts/_form.html.erb

@@ -45,7 +45,7 @@
         <div class="form-group">
           <%= f.label :bill_cycle_day_local, 'Bill cycle day', :class => 'col-sm-3 control-label' %>
           <div class="col-sm-9">
-            <%= f.text_field :bill_cycle_day_local, :class => 'form-control' %>
+            <%= f.number_field :bill_cycle_day_local, :class => 'form-control', :min => 0, max: 31 %>
           </div>
         </div>
         <div class="form-group">
@@ -111,13 +111,13 @@
     <div class="form-group">
       <%= f.label :country, 'Country', :class => 'col-sm-3 control-label' %>
       <div class="col-sm-9">
-        <%= f.country_select :country, {:priority_countries => %w(US CA), :selected => @account.country}, :class => 'form-control' %>
+        <%= f.country_select :country, {:priority_countries => %w(US CA), :selected => @account.country, :sort_provided => false}, :class => 'form-control' %>
       </div>
     </div>
     <div class="form-group">
         <%= f.label :phone, 'Phone', :class => 'col-sm-3 control-label' %>
         <div class="col-sm-9">
-            <%= f.text_field :phone, :class => 'form-control' %>
+            <%= f.number_field :phone, :class => 'form-control' %>
         </div>
     </div>
     <div class="form-group">

data/app/views/kaui/accounts/index.html.erb

@@ -65,7 +65,6 @@ $(document).ready(function() {
   <!-- When we don't know the total number of pages, we need to hide the legend and next button manually -->
   <% if @max_nb_records.nil? %>
     $('#accounts-table').on('draw.dt', function() {
-      $.noConflict();
       var noMoreData = table.column(0)
                             .data()
                             .length == 0;

data/app/views/kaui/admin_tenants/new_overdue_config.html.erb

@@ -38,11 +38,11 @@
                     <td><%= state_form.select :is_block_changes, options_for_select([true, false ], state.is_block_changes), :class => 'form-control'  %></td>
                     <td><%= state_form.select :subscription_cancellation_policy, options_for_select([:NONE, :POLICY_NONE, :POLICY_IMMEDIATE, :POLICY_END_OF_TERM], state.subscription_cancellation), :class => 'form-control' %></td>
                     <%= state_form.fields_for 'condition' do |condition| %>
-                      <td><%= condition.number_field :time_since_earliest_unpaid_invoice_equals_or_exceeds, :value => state.condition.time_since_earliest_unpaid_invoice_equals_or_exceeds.number %></td>
-                      <td><%= condition.select :control_tag_inclusion, options_for_select([:NONE, :AUTO_PAY_OFF, :AUTO_INVOICING_OFF, :OVERDUE_ENFORCEMENT_OFF, :MANUAL_PAY, :TEST, :PARTNER], state.condition.control_tag_inclusion), :class => 'form-control'  %></td>
-                      <td><%= condition.select :control_tag_exclusion, options_for_select([:NONE, :AUTO_PAY_OFF, :AUTO_INVOICING_OFF, :OVERDUE_ENFORCEMENT_OFF, :MANUAL_PAY, :TEST, :PARTNER], state.condition.control_tag_exclusion), :class => 'form-control'%></td>
-                      <td><%= condition.number_field :number_of_unpaid_invoices_equals_or_exceeds, :value => state.condition.number_of_unpaid_invoices_equals_or_exceeds %></td>
-                      <td><%= condition.number_field :total_unpaid_invoice_balance_equals_or_exceeds, :step => :any, :value => state.condition.total_unpaid_invoice_balance_equals_or_exceeds %></td>
+                      <td><%= condition.number_field :time_since_earliest_unpaid_invoice_equals_or_exceeds, :value => state.condition.time_since_earliest_unpaid_invoice_equals_or_exceeds&.number %></td>
+                      <td><%= condition.select :control_tag_inclusion, options_for_select([:NONE, :AUTO_PAY_OFF, :AUTO_INVOICING_OFF, :OVERDUE_ENFORCEMENT_OFF, :MANUAL_PAY, :TEST, :PARTNER], state.condition&.control_tag_inclusion), :class => 'form-control'  %></td>
+                      <td><%= condition.select :control_tag_exclusion, options_for_select([:NONE, :AUTO_PAY_OFF, :AUTO_INVOICING_OFF, :OVERDUE_ENFORCEMENT_OFF, :MANUAL_PAY, :TEST, :PARTNER], state.condition&.control_tag_exclusion), :class => 'form-control'%></td>
+                      <td><%= condition.number_field :number_of_unpaid_invoices_equals_or_exceeds, :value => state.condition&.number_of_unpaid_invoices_equals_or_exceeds %></td>
+                      <td><%= condition.number_field :total_unpaid_invoice_balance_equals_or_exceeds, :step => :any, :value => state.condition&.total_unpaid_invoice_balance_equals_or_exceeds %></td>
                     <% end %>
                   <% end %>
                 <% end %>

data/app/views/kaui/home/_advanced_search_modal.html.erb

@@ -23,7 +23,7 @@
           <div class="form-group">
             <%= label_tag :search_by, 'Search by', :class => 'col-sm-4 control-label' %>
             <div class="col-sm-8">
-              <%= select_tag :search_by, options_for_select(['', 'ID', 'EXTERNAL_KEY'],''), :class => 'form-control' %>
+              <%= select_tag :search_by, options_for_select(object_fields_for_advanced_search,''), :class => 'form-control' %>
             </div>
           </div>
           <div class="form-group">
@@ -68,13 +68,34 @@
             $('#advancedQuery').val(query);
         }
 
+        const searchByFieldsMap = <%= advanced_search_object_fields_map.to_json.html_safe %>;
+        function refresh_search_by() {
+            var objectType = $('#object_type').val();
+
+            // Clears the selected field if it's not supported by the selected object type
+            var searchBy = $('#search_by').val();
+            if (searchByFieldsMap[searchBy] && !searchByFieldsMap[searchBy].includes(objectType)) {
+              $('#search_by').val(' ');
+              refresh_query();
+            }
+
+            // Updates search options to only show fields supported by the selected object type
+            for (field in searchByFieldsMap) {
+                var isSupported = searchByFieldsMap[field].includes(objectType);
+                var fieldOption = $(`#search_by > option[value=${field}]`);
+                isSupported ? fieldOption.show() : fieldOption.hide();
+            }
+        }
+
         $('#search_for').keyup(function(){ refresh_query(); });
         $('#search_for').change(function(){ refresh_query(); });
         $('#object_type').change(function(){ refresh_query(); });
+        $('#object_type').change(function(){ refresh_search_by(); });
         $('#search_by').change(function(){ refresh_query(); });
         $('#fast').change(function(){ refresh_query(); });
 
         refresh_query();
+        refresh_search_by();
 
     });
 <% end %>

data/app/views/kaui/invoices/show.html.erb

@@ -4,8 +4,9 @@
 
     <h1>
       Invoice <%= @invoice.invoice_number %>
-      <% unless @invoice.invoice_number.blank? %>
+      <% if !@invoice.invoice_number.blank? %>
         <%= Kaui.customer_invoice_link.call(@invoice, self) %>
+        <%= Kaui.additional_invoice_links.call(@invoice, self) %>
       <% end %>
     </h1>
 

data/app/views/kaui/layouts/kaui_header.html.erb

@@ -4,8 +4,8 @@
   <meta name="viewport" content="width=device-width, initial-scale=1">
 
   <title>Kaui</title>
-  <%= stylesheet_link_tag 'kaui_application', :media => 'all' %>
-
+  <%= stylesheet_link_tag 'application', :media => 'all' %>
+  <%= javascript_include_tag 'application' %>
 
   <%= csrf_meta_tags %>
 
@@ -14,21 +14,5 @@
   <!--[if lt IE 9]>
   <script src="https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js"></script>
   <script src="https://oss.maxcdn.com/respond/1.4.2/respond.min.js"></script>
-
   <![endif]-->
-
-  <%= javascript_include_tag "routes" %>
-  <%= javascript_include_tag "jquery" %>
-  <%= javascript_include_tag "jquery-ui" %>
-  <%= javascript_include_tag "popper" %>
-  <%= javascript_include_tag "spin" %>
-  <%= javascript_include_tag "jquery.spin" %>
-  <%= javascript_include_tag "bootstrap-sprockets" %>
-  <%= javascript_include_tag "bootstrap-datepicker" %>
-  <%= javascript_include_tag "dataTables/jquery.dataTables" %>
-  <%= javascript_include_tag "dataTables/bootstrap/3/jquery.dataTables.bootstrap" %>
-  <%= javascript_include_tag "mustache"  %>
-  <%= javascript_include_tag "bootstrap-datepicker"  %>
-  <%= javascript_include_tag "kaui/kaui" %>
- 
 </head>

data/app/views/kaui/refunds/_form.html.erb

@@ -23,6 +23,13 @@
     </div>
 
     <div id="invoiceItems" style="display:none">
+        <div class="form-group">
+            <label class="col-sm-2 control-label"></label>
+            <div class="col-sm-10">
+                <input type="checkbox" id="selectAll">
+                <%="Select All" %>
+            </div>
+        </div>
       <% @invoice.items.each_with_index do |ii, index| %>
           <% if ii.amount > 0 %>
               <div id=<%= "div_#{ii.invoice_item_id}" %> class="form-group">
@@ -31,7 +38,7 @@
                 <% bundle_result = @bundles.find { |bundle| bundle.bundle_id == ii.bundle_id } %>
                 <div class="col-sm-10">
                   <input type="checkbox" id=<%= "cb_adj_#{ii.invoice_item_id}" %>>
-                  <%= "Item #{index + 1} : #{ii.description} #{"(bundle #{bundle_result.external_key})" unless bundle_result.nil?}" %>
+                  <%= Kaui.refund_invoice_description.call(index, ii, bundle_result) %>
                   <%= text_field_tag "adjustments[#{ii.invoice_item_id}]", index, :id => "tf_adj_#{ii.invoice_item_id}", :value => ii.amount, :class => 'form-control' %>
                 </div>
               </div>
@@ -144,9 +151,23 @@
         }
         validateRefundAmount();
     };
+    /*
+     * Check status of all items check-box check/uncheck status for select all chec-kbox 
+     */
+    var checkSelectAllCheckboxStatus = function(){
+        var checkedCheckBoxCount = 0;
+        var checkboxList = $('input').filter(function() { return this.id.match(/cb_adj_/) });
+        var checkboxListCount = checkboxList.length;
+        checkboxList.each(function() {
+            if ($(this).is(':checked')) {
+                checkedCheckBoxCount++;
+            }
+        });
+        $("#selectAll").prop('checked', checkedCheckBoxCount == checkboxListCount);
+    }
 
     /*
-    * When clicking checkbox for each item, disable amount and recompute total refund amount
+    * When clicking checkbox for each item, disable amount, Changes SelectAll status and recompute total refund amount
     */
     var onClickInvoiceItemAdjustment = function(event) {
         var id = checkboxToTextId(this.id);
@@ -158,8 +179,30 @@
         }
         recomputeRefundAmountAndValidateAmount();
         validateInvoiceItemAmount(id);
+        checkSelectAllCheckboxStatus();
     };
 
+    /*
+    * When clicking select all checkbox - select each items, disable amount and recompute total refund amount
+    */
+    var onClickInvoiceItemsSelectAll = function(event){
+        var isChecked = $('#selectAll').prop('checked');
+        $('input').filter(function() { return this.id.match(/tf_adj_/) }).each(function() {
+            var textFieldId = this.id;
+            var textFieldIdElm = $("#" + textFieldId);
+            var checkboxId = textToCheckboxId(this.id);
+            if(checkboxId){
+                $("#" + checkboxId).prop('checked', isChecked);
+                textFieldIdElm.prop('readonly', isChecked);
+                if(!isChecked){
+                    textFieldIdElm.attr('value', textFieldIdElm.attr('originalValue'));
+                }
+                validateInvoiceItemAmount(textFieldId);
+            }
+        });
+        recomputeRefundAmountAndValidateAmount();
+    }
+
     /*
     * When selecting Invoice Adjustment or No Adjustment, hide invoice items and recompute refund Amount
     */
@@ -213,6 +256,11 @@
             return this.id.match(/cb_adj_/);
         }).click(onClickInvoiceItemAdjustment);
 
+        /*
+        * Attach handler onClickInvoiceItemsSelectAll for selectall checkbox
+        */
+        $('#selectAll').click(onClickInvoiceItemsSelectAll);
+
         /*
         * Attach handler for all invoice item text areas so that:
         * - We disable posting form when pressing 'ENTER'

data/app/views/kaui/subscriptions/_cancel_by_date_modal.html.erb

@@ -10,7 +10,7 @@
           <div class="form-group">
             <%= label_tag :requested_date, 'Date', :class => 'col-sm-2 control-label' %>
             <div class="col-sm-10">
-              <%= text_field_tag :requested_date, Date.parse(Time.now.to_s).to_s, :class => 'form-control date-picker' %>
+              <input class="form-control" value="<%=Date.parse(Time.now.to_s).to_s%>" name="requested_date" type="text" data-provide="datepicker" data-date-format="yyyy-mm-dd" data-date-today-highlight="true">
             </div>
           </div>
           <div class="form-group">

data/app/views/kaui/subscriptions/_edit_form.html.erb

@@ -37,7 +37,7 @@
     <div class="form-group", id="form_requested_date" >
         <%= label_tag :requested_date, 'Change Date', :class => 'col-sm-3 control-label' %>
         <div class="col-sm-9">
-            <%= text_field_tag :requested_date, Date.parse(Time.now.to_s).to_s, :class => 'form-control date-picker' %>
+          <input class="form-control" value="<%=Date.parse(Time.now.to_s).to_s%>" name="requested_date" type="text" data-provide="datepicker" data-date-format="yyyy-mm-dd" data-date-today-highlight="true">
         </div>
     </div>
     <div class="form-group", id="form_policy">

data/app/views/kaui/subscriptions/_form.html.erb

@@ -59,7 +59,7 @@
     <div class="form-group" id="form_requested_date">
       <%= label_tag :requested_date, 'Date', :class => 'col-sm-2 control-label' %>
       <div class="col-sm-10">
-        <%= text_field_tag :requested_date, Date.parse(Time.now.to_s).to_s, :class => 'form-control date-picker' %>
+      <input class="form-control" value="<%=Date.parse(Time.now.to_s).to_s%>" name="requested_date" type="text" data-provide="datepicker" data-date-format="yyyy-mm-dd" data-date-today-highlight="true">
       </div>
     </div>
     <div class="form-group">

data/app/views/kaui/subscriptions/edit_bcd.erb

@@ -15,7 +15,7 @@
                 <div class="form-group" , id="form_effective_from_date">
                     <%= label_tag :effective_from_date, 'Effective Date', :class => 'col-sm-2 control-label' %>
                     <div class="col-sm-10">
-                        <%= text_field_tag :effective_from_date, Date.parse(Time.now.to_s).to_s, :class => 'form-control date-picker' %>
+                        <input class="form-control" value="<%=Date.parse(Time.now.to_s).to_s%>" id="effective_from_date" name="effective_from_date" type="text" data-provide="datepicker" data-date-format="yyyy-mm-dd" data-date-today-highlight="true">
                     </div>
                 </div>
                 <div class="form-group">

data/app/views/kaui/tag_definitions/_form.html.erb

@@ -10,8 +10,7 @@
       <div class="col-sm-10" id="object_types">
         <% (@tag_definition.applicable_object_types || [:account]).each_with_index do |object_type, index| %>
         <div id="object_type_line_<%= index %>">
-          <%= f.select "applicable_object_types[#{index}]", object_types,
-                       {:selected => object_type, :id => "object_type_#{index}"}, :class => 'form-control tag-definition-select' %>
+        <%= select_tag "tag_definition[applicable_object_types][#{index}]", options_for_select(object_types),  {:selected => object_type, :id => "object_type_#{index}", :class => 'form-control tag-definition-select'} %>
           <a class='btn btn-xs' href="javascript:void(0);" onclick="delete_object_type(this);" id="delete_object_type_<%= index %>" <%= "style='display: none;'".html_safe if index == 0 %>>
             <%= '<i class="fa fa-times"></i>'.html_safe %>
           </a>

data/config/routes.rb

@@ -98,6 +98,7 @@ Kaui::Engine.routes.draw do
   scope '/invoices' do
     match '/pagination' => 'invoices#pagination', :via => :get, :as => 'invoices_pagination'
     match '/:id/show_html' => 'invoices#show_html', :via => :get, :as => 'show_html_invoice'
+    match '/:number' => 'invoices#restful_show_by_number', :via => :get, :constraints => { number: /\d+/ }
     match '/:id' => 'invoices#restful_show', :via => :get, :as => 'invoice'
     match '/commit' => 'invoices#commit_invoice', :via => :post, :as => 'commit_invoice'
     match '/void' => 'invoices#void_invoice', :via => :delete, :as => 'void_invoice'