Skip to content

[5.1] Add Global Setting for Form Layout option to custom fields#37320

Merged
Razzo1987 merged 77 commits intojoomla:5.1-devfrom
AndySDH:patch-21
Nov 2, 2023
Merged

[5.1] Add Global Setting for Form Layout option to custom fields#37320
Razzo1987 merged 77 commits intojoomla:5.1-devfrom
AndySDH:patch-21

Conversation

@AndySDH
Copy link
Copy Markdown
Contributor

@AndySDH AndySDH commented Mar 19, 2022 •
edited
Loading

Pull Request for Issue #37162. Follow-up to PR #36551 and issue #36548.

Purpose of the PR

As discussed in #37162, this allows users to set a global preference for the HTML Select for custom fields of types: list, SQL, radio, integer.

I would have just removed the option, as the "raw" select is just ugly, but @laoneo noted that we need to keep it for Backwards Compatibility.

Summary of Changes

  • Added plugin settings for all relevant fields so that the field inherits from the global options by default (this allows users to set a global preference instead of having to set it for each field they create)
  • The global option will still use the old "raw" style by default (to keep Backwards Compatibility)
  • Implemented the same setting for the Integer field type as well (it was forgotten about in PR 36551 )
  • Renamed the option to "Form Layout" for it to be more explicative
  • Added new language strings in a shared language file to avoid duplicating them for all field types

Testing Instructions

First create the custom field:

  • Create Custom Field of type 'list' for the Content component. Make sure that the 'Multiple' setting is set to 'Yes'
  • Go to Options->Form Options: Keep the 'Form Layout' as 'Use Settings from Plugin'

Then:

  • Go to Plugin Manager, and in the plugin options for 'Fields - List', change the default 'Form Layout' option
  • Edit an article and check the form display of the field, it will change based on your selection of 'Form Layout'

Actual result BEFORE applying this Pull Request

There was no global setting to inherit from the plugin for 'Form Layout'

Expected result AFTER applying this Pull Request

There is now a global setting for 'Form Layout' that controls the default value when not specified in an individual field.

Changing the global setting will change the form layout of the field.

AndySDH added 16 commits March 19, 2022 11:59
@AndySDH
Rename option to more explicative name
5a7d86b
@AndySDH
Remove string from here, we need it for mulitple field types
cd69098
@AndySDH
Remove string from here, we need it for mulitple field types
7ecaf31
@AndySDH
Implement global language strings for select types
b198242
@AndySDH
Move setting to correct place, add global inherit
fd81516
@AndySDH
Move setting to correct place, add global inherit
0364134
@AndySDH
Move setting to correct place, add global inherit
266f792
@AndySDH
We need to implement it for integer field as well
008d415
@AndySDH
Implement global setting for integer
7bee728
@AndySDH
Implement global setting for list
4c07339
@AndySDH
Implement global setting for sql
50c7f80
@AndySDH
Implement global setting for radio
91e9ef7
@AndySDH
Fix a minor bug, sql query should not be required
0d9d043
@AndySDH
This is now in the fieldparams instead of the params
9b404f3
@AndySDH
Support for Language Strings from com_fields
1a4184b 
Allows for shared language string shared across multiple fields plugins (to avoid duplicating them for multiple fields)
@AndySDH
Fixing code
e242069
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.1-dev labels Mar 19, 2022
@AndySDH AndySDH mentioned this pull request Mar 19, 2022
Closed
brianteeman
brianteeman reviewed Mar 19, 2022
View reviewed changes
@AndySDH @brianteeman
order the strings alphabetically
9e65209 
Co-authored-by: Brian Teeman <brian@teeman.net>
brianteeman
brianteeman reviewed Mar 19, 2022
View reviewed changes
brianteeman
brianteeman reviewed Mar 19, 2022
View reviewed changes
brianteeman
brianteeman reviewed Mar 19, 2022
View reviewed changes
brianteeman
brianteeman reviewed Mar 19, 2022
View reviewed changes
AndySDH and others added 3 commits March 19, 2022 17:26
@AndySDH @brianteeman
Update plugins/fields/integer/params/integer.xml
3abd851 
Co-authored-by: Brian Teeman <brian@teeman.net>
@AndySDH @brianteeman
Update plugins/fields/list/params/list.xml
daabbd6 
Co-authored-by: Brian Teeman <brian@teeman.net>
@AndySDH @brianteeman
Update plugins/fields/sql/params/sql.xml
82febcd 
Co-authored-by: Brian Teeman <brian@teeman.net>
@obuisard
Copy link
Copy Markdown
Contributor

obuisard commented Oct 2, 2023

I have tested this and have not triggered any errors this time. I think it is OK but do not know how to create a ticked pass box as I usually do that with the Joomla test interface which is currently down.

You will have to wait for the issue tracker to get back online, I am afraid. This is the only way you can cast your test. In the meantime, Richard set the PR as RTC, which acknowledges your test.

@AndySDH
Copy link
Copy Markdown
Contributor Author

AndySDH commented Oct 2, 2023

I have tested this and have not triggered any errors this time. I think it is OK but do not know how to create a ticked pass box as I usually do that with the Joomla test interface which is currently down.

Thank you :)

@Quy Quy removed the PBF Pizza, Bugs and Fun label Oct 2, 2023
@richard67 richard67 changed the title [5.1] Add Global Setting for Form Layout option to custom fields [5.0] Add Global Setting for Form Layout option to custom fields Oct 3, 2023
@richard67 richard67 added PBF Pizza, Bugs and Fun PR-5.0-dev and removed RTC This Pull Request is Ready To Commit PR-5.1-dev labels Oct 3, 2023
@richard67
Copy link
Copy Markdown
Member

richard67 commented Oct 3, 2023

RTC

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37320.

@joomla-cms-bot joomla-cms-bot added PR-5.1-dev RTC This Pull Request is Ready To Commit labels Oct 3, 2023
@richard67 richard67 removed PBF Pizza, Bugs and Fun PR-5.0-dev labels Oct 3, 2023
@richard67 richard67 changed the title [5.0] Add Global Setting for Form Layout option to custom fields [5.1] Add Global Setting for Form Layout option to custom fields Oct 3, 2023
@LadySolveig LadySolveig added this to the Joomla! 5.1.0 milestone Nov 1, 2023
@LadySolveig
Copy link
Copy Markdown
Contributor

LadySolveig commented Nov 2, 2023

Many thanks for this contribution @AndySDH and the very nice teamwork!

@Razzo1987
Copy link
Copy Markdown
Contributor

Razzo1987 commented Nov 2, 2023

I have tested this item ✅ successfully on 9137eee

Very nice work! We love it!

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37320.

@Razzo1987 Razzo1987 merged commit 5eb45ba into joomla:5.1-dev Nov 2, 2023
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 2, 2023
@jgerman-bot jgerman-bot mentioned this pull request Nov 3, 2023
Closed
heelc29 added a commit to heelc29/joomla-cms that referenced this pull request Dec 31, 2023
@heelc29
[5.1] add language deprecation for joomla#37320
055caef
@heelc29 heelc29 mentioned this pull request Dec 31, 2023
Merged
2 tasks
Razzo1987 pushed a commit that referenced this pull request Jan 5, 2024
@heelc29
[5.1] add language deprecation for #37320 (#42584)
6847b5a
Kostelano added a commit to JPathRu/localisation that referenced this pull request Feb 24, 2024
@Kostelano
Новое в Joomla 5.1.0-Alpha4
cac4459 
joomla/joomla-cms#42220 - (только для en-GB)
joomla/joomla-cms#37320 +
joomla/joomla-cms#42390 +
joomla/joomla-cms#42393 +
joomla/joomla-cms#42584 +
joomla/joomla-cms#42607 +
joomla/joomla-cms#42402 +
joomla/joomla-cms#42334 +
joomla/joomla-cms#42610 +
joomla/joomla-cms#42663 +
joomla/joomla-cms#42747 +
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@laoneo laoneo laoneo left review comments

@Quy Quy Quy left review comments

@obuisard obuisard obuisard left review comments

@HLeithner HLeithner Awaiting requested review from HLeithner

+2 more reviewers

@brianteeman brianteeman brianteeman left review comments

@regularlabs regularlabs regularlabs left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Feature Language Change This is for Translators

Projects

None yet

Milestone

Joomla! 5.1.0

Development

Successfully merging this pull request may close these issues.