Fix broken stimulus controllers after better_together namespace added to files

Author: rsmithlal

app/helpers/better_together/share_buttons_helper.rb

@@ -32,7 +32,7 @@ def share_buttons(platforms: BetterTogether::Metrics::Share::SHAREABLE_PLATFORMS
             link_to share_button_content(platform).html_safe, '#',
                     class: "share-button share-#{platform}",
                     data: {
-                      action: 'click->share#share',
+                      action: 'click->better_together--share#share',
                       platform: platform,
                       url: url,
                       title: title,

app/helpers/better_together/translatable_fields_helper.rb

@@ -10,16 +10,18 @@ def translation_tab_button(attribute:, locale:, temp_id:, model:) # rubocop:todo
       translation_present = model.public_send(locale_attribute).present?
 
       content_tag(:li, class: 'nav-item', role: 'presentation',
-                       data: { attribute:, translation_target: 'tab', locale: }) do
+                       data: { attribute:, locale: },
+                       'data-better_together--translation-target' => 'tab') do
         content_tag(:button,
                     id: "#{unique_locale_attribute}-tab",
                     class: ['nav-link', ('active' if locale.to_s == I18n.locale.to_s),
                             ('text-success' if translation_present)],
                     data: { bs_toggle: 'tab',
                             bs_target: "##{unique_locale_attribute}-field",
-                            action: 'click->translation#syncLocaleAcrossFields',
-                            locale:,
-                            translation_target: 'tabButton' },
+                            action: 'click->better_together--translation#syncLocaleAcrossFields',
+                            locale:
+                          },
+                    'data-better_together--translation-target' => 'tabButton',
                     role: 'tab',
                     type: 'button',
                     aria: { controls: "#{unique_locale_attribute}-field",

app/views/better_together/addresses/_address_fields.html.erb

@@ -62,7 +62,7 @@
     </div>
   </div>
   <div class="form-group mt-3 mt-sm-0">
-    <%= link_to '#', class: 'btn btn-outline-danger btn-sm', data: { action: 'click->dynamic-fields#remove' } do %>
+    <%= link_to '#', class: 'btn btn-outline-danger btn-sm', data: { action: 'click->better_together--dynamic-fields#remove' } do %>
       <i class="fas fa-minus"></i> <%= t('better_together.remove') %>
     <% end %>
   </div>

app/views/better_together/content/blocks/fields/shared/_translatable_field.html.erb

@@ -3,7 +3,7 @@
 <%- temp_id = temp_id_for(model, temp_id:) %>
 
 <div id="<%= dom_id(model) %>-<%= attribute %>-translation-fields" class="localized-fields" data-controller="better_together--translation">
-  <ul class="nav nav-tabs px-2" id="<%= attribute %>-translation-tabs-<%= temp_id %>" role="tablist" data-translation-target="tab">
+  <ul class="nav nav-tabs px-2" id="<%= attribute %>-translation-tabs-<%= temp_id %>" role="tablist" data-better_together--translation-target="tab">
     <%- I18n.available_locales.each do |locale| %>
       <%= translation_tab_button(attribute: attribute, locale: locale, temp_id: temp_id, model: model) %>
     <% end %>
@@ -18,7 +18,7 @@
         role="tabpanel"
         aria-labelledby="<%= "#{unique_locale_attribute}-tab" %>"
         data-locale="<%= locale %>"
-        data-translation-target="tabContent">
+        data-better_together--translation-target="tabContent">
 
         <%= label_tag "#{scope}[#{locale_attribute}]", 
                       model.class.human_attribute_name(locale_attribute) %>

app/views/better_together/content/blocks/fields/shared/_translatable_rich_text_field.html.erb

@@ -4,12 +4,12 @@
 
 <%= render layout: 'better_together/content/blocks/fields/shared/translatable_field', 
            locals: { model: model, scope: scope, temp_id: temp_id, attribute: attribute } do |locale_attribute| %>
-  
+
   <%=
     rich_text_area_tag "#{scope}[#{locale_attribute}]", model.public_send(locale_attribute),
     class: "form-control#{' is-invalid' if model.errors[locale_attribute].any?}",
-    data: { translation_target: 'trix' },
+    'data-better_together-translation-target' => 'trix',
     id: "#{dom_id(model)}-#{locale_attribute}"
   %>
-  
+
 <% end %>

app/views/better_together/content/blocks/fields/shared/_translatable_string_field.html.erb

@@ -9,6 +9,7 @@
         model.public_send(locale_attribute),
         id: "#{dom_id(model)}-#{locale_attribute}",
         class: "form-control#{' is-invalid' if model.errors[locale_attribute].any?}",
-        data: { translation_target: "input", action: "input->translation#updateTranslationStatus" } %>
+        data: {action: "input->better_together--translation#updateTranslationStatus" },
+'data-better_together-translation-target' => "input" %>
 
 <% end %>

app/views/better_together/content/blocks/fields/shared/_translatable_text_field.html.erb

@@ -9,6 +9,7 @@
         model.public_send(locale_attribute),
         id: "#{dom_id(model)}-#{locale_attribute}",
         class: "form-control#{' is-invalid' if model.errors[locale_attribute].any?}",
-        data: { translation_target: "input", action: "input->translation#updateTranslationStatus" } %>
+        data: {action: "input->better_together--translation#updateTranslationStatus" },
+'data-better_together-translation-target' => "input" %>
 
 <% end %>

app/views/better_together/content/page_blocks/_form_fields.html.erb

@@ -1,6 +1,6 @@
 <% temp_id = page_block.persisted? ? page_block.id : SecureRandom.uuid %>
 
-<div id="<%= dom_id(page_block, temp_id) %>" class="page-block-fields <%= dom_class(page_block) %>" data-page-blocks-target="pageBlock">
+<div id="<%= dom_id(page_block, temp_id) %>" class="page-block-fields <%= dom_class(page_block) %>" data-better_together--page-blocks-target="pageBlock">
   <div class="d-flex justify-content-between align-items-center mb-3">
     <!-- Title and control buttons in a toolbar -->
     <h5 class="mb-0"><%= page_block.block.class.model_name.human %></h5>
@@ -16,11 +16,11 @@
       </div>
 
       <div class="btn-group me-2" role="group" aria-label="Position controls">
-        <button type="button" class="btn btn-outline-secondary" data-action="click->page-blocks#moveUp"
+        <button type="button" class="btn btn-outline-secondary" data-action="click->better_together--page-blocks#moveUp"
                 aria-label="Move up" data-bs-toggle="tooltip" title="Move Up">
           <i class="fas fa-arrow-up"></i>
         </button>
-        <button type="button" class="btn btn-outline-secondary" data-action="click->page-blocks#moveDown"
+        <button type="button" class="btn btn-outline-secondary" data-action="click->better_together--page-blocks#moveDown"
                 aria-label="Move down" data-bs-toggle="tooltip" title="Move Down">
           <i class="fas fa-arrow-down"></i>
         </button>

app/views/better_together/email_addresses/_email_address_fields.html.erb

@@ -18,7 +18,7 @@
     <%= form.email_field :email, class: 'form-control', required: true %>
   </div>
   <div class="col-md-2 mt-3 mt-md-0">
-    <%= link_to '#', class: 'btn btn-outline-danger btn-sm', data: { action: 'click->dynamic-fields#remove' } do %>
+    <%= link_to '#', class: 'btn btn-outline-danger btn-sm', data: { action: 'click->better_together--dynamic-fields#remove' } do %>
       <i class="fas fa-minus"></i> <%= t('better_together.remove') %>
     <% end %>
   </div>

app/views/better_together/phone_numbers/_phone_number_fields.html.erb

@@ -18,7 +18,7 @@
     <%= form.telephone_field :number, class: 'form-control', required: true %>
   </div>
   <div class="col-md-2 mt-3 mt-md-0">
-    <%= link_to '#', class: 'btn btn-outline-danger btn-sm', data: { action: 'click->dynamic-fields#remove' } do %>
+    <%= link_to '#', class: 'btn btn-outline-danger btn-sm', data: { action: 'click->better_together--dynamic-fields#remove' } do %>
       <i class="fas fa-minus"></i> <%= t('better_together.remove') %>
     <% end %>
   </div>