Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed argument prefix warnings #3594

Merged
merged 1 commit into from
Jan 21, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= @form.text_field field.id,
value: field.value.to_s,
class: classes("w-full"),
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<% if field.value.present? %>
<%= area_map field.map_data, **field.mapkick_options %>
<% else %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args, flush: true do %>
<%= index_field_wrapper(**field_wrapper_args, flush: true) do %>
<%= render Avo::Fields::Common::BadgeViewerComponent.new value: @field.value, options: @field.options %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= render Avo::Fields::Common::BadgeViewerComponent.new value: @field.value, options: @field.options %>
<% end %>
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
data-association="<%= @field.id %>"
data-association-class="<%= @field&.target_resource&.model_class || nil %>"
>
<%= field_wrapper **field_wrapper_args, label_for: @field.polymorphic_form_field_label, help: @field.polymorphic_help || '' do %>
<%= field_wrapper(**field_wrapper_args, label_for: @field.polymorphic_form_field_label, help: @field.polymorphic_help || '') do %>
<%= @form.select @field.type_input_foreign_key, @field.types.map { |type| [Avo.resource_manager.get_resource_by_model_class(type.to_s).name, type.to_s] },
{
value: @field.value,
Expand All @@ -32,7 +32,7 @@
<% @field.types.each do |type| %>
<template data-belongs-to-field-target="type" data-type="<%= type %>">
<div data-polymorphic-type="<%= type %>">
<%= field_wrapper **field_wrapper_args.merge!(data: reload_data), label: Avo.resource_manager.get_resource_by_model_class(type.to_s).name do %>
<%= field_wrapper(**field_wrapper_args.merge!(data: reload_data), label: Avo.resource_manager.get_resource_by_model_class(type.to_s).name) do %>
<% if @field.is_searchable? %>
<%= render Avo::Pro::SearchableAssociations::AutocompleteComponent.new form: @form,
disabled: disabled,
Expand Down Expand Up @@ -88,7 +88,7 @@
<% end %>
</div>
<% else %>
<%= field_wrapper **field_wrapper_args.merge!(data: reload_data) do %>
<%= field_wrapper(**field_wrapper_args.merge!(data: reload_data)) do %>
<% if @field.is_searchable? %>
<%= render Avo::Pro::SearchableAssociations::AutocompleteComponent.new form: @form,
field: @field,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<%= link_to @field.label, helpers.resource_view_path(
record: @field.index_link_to_record,
resource: @field.index_link_to_resource
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= link_to @field.label, resource_view_path, data: {turbo_frame: @field.target} %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<div class="h-8 flex items-center">
<%= @form.check_box @field.id,
value: @field.value,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args, flush: true do %>
<%= index_field_wrapper(**field_wrapper_args, flush: true) do %>
<%= render Avo::Fields::Common::BooleanCheckComponent.new checked: @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= render Avo::Fields::Common::BooleanCheckComponent.new checked: @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<div class="flex items-center">
<div class="space-y-2">
<% @field.options.each do |id, label| %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args, dash_if_blank: false do %>
<%= index_field_wrapper(**field_wrapper_args, dash_if_blank: false) do %>
<%= render Avo::Fields::Common::BooleanGroupComponent.new options: @field.options, value: @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args, dash_if_blank: false do %>
<%= field_wrapper(**field_wrapper_args, dash_if_blank: false) do %>
<%= render Avo::Fields::Common::BooleanGroupComponent.new options: @field.options, value: @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args, full_width: true do %>
<%= field_wrapper(**field_wrapper_args, full_width: true) do %>
<div data-controller="code-field">
<%= @form.text_area @field.id,
value: @field.value,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args, full_width: true do %>
<%= field_wrapper(**field_wrapper_args, full_width: true) do %>
<div data-controller="code-field" style="--height: <%= @field.height %>">
<%= text_area_tag @field.id, @field.value,
class: helpers.input_classes('w-full'),
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= @form.select @field.id, @field.select_options, {
value: @field.value,
selected: @field.value,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<% if @field.display_code %>
<%= @field.value %>
<% else %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<% if @field.display_code %>
<%= @field.value %>
<% else %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= content_tag :div, data: {
controller: "date-field",
date_field_view_value: @view,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<%= content_tag :div, data: {
controller: "date-field",
date_field_view_value: @view,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= content_tag :div, data: {
controller: "date-field",
date_field_view_value: @view,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= content_tag :div, class: "flex relative", data: {
controller: "date-field",
date_field_view_value: @view,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<%= content_tag :div, data: {
controller: "date-field",
date_field_view_value: @view,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= content_tag :div, data: {
controller: "date-field",
date_field_view_value: @view,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= @form.text_field @field.id,
class: classes("w-full"),
data: @field.get_html(:data, view: view, element: :input),
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= index_field_wrapper **field_wrapper_args, flush: true do %>
<%= index_field_wrapper(**field_wrapper_args, flush: true) do %>
<% if @field.value.present? %>
<%= link_to_if @field.link_to_record.present?,
image_tag(@field.value,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<% if @field.value.present? %>
<%= image_tag @field.value, style: 'max-width: 340px;' %>
<% end %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<% if @field.value.present? %>
<div class="mb-2">
<%= render Avo::Fields::Common::Files::ViewType::GridItemComponent.new resource: @resource, field: @field %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= index_field_wrapper **field_wrapper_args, flush: flush? do %>
<%= index_field_wrapper(**field_wrapper_args, flush: flush?) do %>
<% if @field.value.present? %>
<% if @field.value.attached? && @field.value.representable? && @field.is_image %>
<%= link_to_if @field.link_to_record, image_tag(helpers.main_app.url_for(@field.value), class: 'h-10'), resource_view_path, class: 'block' %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= render Avo::Fields::Common::Files::ViewType::GridItemComponent.new resource: @resource, field: @field %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args, full_width: true do %>
<%= field_wrapper(**field_wrapper_args, full_width: true) do %>
<%= render Avo::Fields::Common::Files::ListViewerComponent.new(field: @field, resource: @resource) if @field.value.present? %>

<% if can_upload_file? %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<%= "#{length} #{t('avo.file', count: length)}" %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args, full_width: true do %>
<%= field_wrapper(**field_wrapper_args, full_width: true) do %>
<%= render Avo::Fields::Common::Files::ListViewerComponent.new(field: @field, resource: @resource) if @field.value.present? %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= index_field_wrapper **field_wrapper_args, flush: true do %>
<%= index_field_wrapper(**field_wrapper_args, flush: true) do %>
<%= render Avo::Fields::Common::GravatarViewerComponent.new(
md5: @field.md5,
default: @field.default,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= render Avo::Fields::Common::GravatarViewerComponent.new(
md5: @field.md5,
default: @field.default,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<%= link_to @field.label, helpers.resource_path(record: @field.value, resource: @field.target_resource) %>
<% end %>
2 changes: 1 addition & 1 deletion app/components/avo/fields/id_field/edit_component.html.erb
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args, class: 'whitespace-no-wrap w-[1%]' do %>
<%= index_field_wrapper(**field_wrapper_args, class: 'whitespace-no-wrap w-[1%]') do %>
<% link_to_if (@field.link_to_record or Avo.configuration.id_links_to_resource), @field.value, resource_view_path, title: t('avo.view_item', item: @resource.name).humanize %>
<% end %>
2 changes: 1 addition & 1 deletion app/components/avo/fields/id_field/show_component.html.erb
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args, full_width: true do %>
<%= field_wrapper(**field_wrapper_args, full_width: true) do %>
<%= render Avo::Fields::Common::KeyValueComponent.new field: @field, form: @form, view: view %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args, full_width: true do %>
<%= field_wrapper(**field_wrapper_args, full_width: true) do %>
<%= render Avo::Fields::Common::KeyValueComponent.new(
field: @field,
view: Avo::ViewInquirer.new(:show)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<% if field.value_as_array? %>
<div class="flex gap-4">
<%= @form.fields_for @field.id do |coordinates_form| %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<% if field.value_present? %>
<%= js_map [{latitude: field.value[0], longitude: field.value[1]}], id: "location-map", zoom: field.zoom, controls: true, **field.mapkick_options %>
<% else %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args, full_width: true do %>
<%= field_wrapper(**field_wrapper_args, full_width: true) do %>
<div data-controller="easy-mde">
<%= @form.text_area @field.id,
value: @field.value,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args, full_width: true do %>
<%= field_wrapper(**field_wrapper_args, full_width: true) do %>
<div data-controller="easy-mde">
<%= text_area_tag @field.id, @field.value,
class: helpers.input_classes('w-full js-has-easy-mde-editor'),
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= @form.number_field @field.id,
value: @field.value,
class: classes("w-full"),
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<%= @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<div class="relative" data-controller="<%= @field.revealable ? 'password-visibility' : nil %>">
<%= @form.password_field @field.id,
value: @field.value,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= index_field_wrapper **field_wrapper_args, dash_if_blank: false, class: 'whitespace-no-wrap w-[1%]', flush: true, center_content: true do %>
<%= index_field_wrapper(**field_wrapper_args, dash_if_blank: false, class: 'whitespace-no-wrap w-[1%]', flush: true, center_content: true) do %>
<%= link_to resource_view_path,
title: t('avo.view_item', item: @resource.name).humanize,
data: {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= content_tag :div, data: {controller: "progress-bar-field"} do %>
<% if @field.display_value %>
<div class="text-center text-sm font-semibold w-full leading-none mb-1">
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args, flush: true do %>
<%= index_field_wrapper(**field_wrapper_args, flush: true) do %>
<%= render Avo::Fields::Common::ProgressBarComponent.new value: @field.value, display_value: @field.display_value, value_suffix: @field.value_suffix, max: @field.max, view: view %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= render Avo::Fields::Common::ProgressBarComponent.new value: @field.value, display_value: @field.display_value, value_suffix: @field.value_suffix, max: @field.max, view: view %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<div class="flex flex-col gap-2">
<% @field.options.each do |key, value| %>
<div>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<%== @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%== @field.value %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= @form.select @field.id, options_for_select(@field.options_for_select, selected: @field.value), {
include_blank: @field.include_blank
},
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<%= @field.label %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args.merge(dash_if_blank: false) do %>
<%= field_wrapper(**field_wrapper_args.merge(dash_if_blank: false)) do %>
<%= @field.value.nil? ? "—" : @field.label %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= @form.text_field @field.id,
class: classes("w-full"),
data: @field.get_html(:data, view: view, element: :input),
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= index_field_wrapper **field_wrapper_args do %>
<%= index_field_wrapper(**field_wrapper_args) do %>
<%= render Avo::Fields::Common::StatusViewerComponent.new label: @field.value, status: @field.status %>
<% end %>
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<%= render Avo::Fields::Common::StatusViewerComponent.new label: @field.value, status: @field.status %>
<% end %>
4 changes: 2 additions & 2 deletions app/components/avo/fields/tags_field/edit_component.html.erb
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args, data: {
<%= field_wrapper(**field_wrapper_args, data: {
controller: "tags-field",
tags_field_mode_value: @field.mode,
tags_field_whitelist_items_value: @field.whitelist_items,
Expand All @@ -8,7 +8,7 @@
tags_field_close_on_select_value: @field.close_on_select,
tags_field_delimiters_value: @field.delimiters,
tags_field_fetch_values_from_value: @field.fetch_values_from,
} do %>
}) do %>
<%# We use a dummy field so the back action from Turbo does not brake the field %>
<%# dummy field %>
<%= text_field_tag "#{@field.id}-dummy", '',
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= index_field_wrapper **field_wrapper_args, flush: true do %>
<%= index_field_wrapper(**field_wrapper_args, flush: true) do %>
<div class="flex gap-1 items-center flex-nowrap">
<% value.take(3).each do |item| %>
<%= render Avo::Fields::TagsField::TagComponent.new(label: label_from_item(item)) %>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<%= field_wrapper **field_wrapper_args do %>
<%= field_wrapper(**field_wrapper_args) do %>
<div class="flex gap-1 items-center flex-wrap">
<% @field.field_value.each do |item| %>
<%= render Avo::Fields::TagsField::TagComponent.new(label: label_from_item(item)) %>
Expand Down
Loading
Loading