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

[4.0] Remove plg_fields_repeatable #23659

Closed
continga opened this issue Jan 24, 2019 · 13 comments
Closed

[4.0] Remove plg_fields_repeatable #23659

continga opened this issue Jan 24, 2019 · 13 comments

Comments

@continga
Copy link

As discussed in #22446 we need to remove the plg_fields_repeatable plugin as soon as #22446 is merged into 3.10-dev

Background: Pull request #22446 implements a new custom field type plg_fields_subfields which is capable of using predefined custom fields for use as sub fields of a subform containing those. This functionality is basically an expansion of the functionality of the plg_fields_repeatable type. Hence, to keep the Joomla! core clean, we need to remove the plg_fields_repeatable plugin.

What needs to be done is that the plg_fields_repeatable plugin needs to be removed for all new installations of Joomla!, and for all existing installations that do not use that plugin (so have no repeatable custom fields). For installations that use it, we need to either provide a transitioning-script, or we need to explain the user how to switch from the repeatable type to the subfields type. To accomplish this, we maybe first need to declare the plg_fields_subfields type as deprecated in a first step.

A first idea of how to remove the repeatable type from core and how to build a transitioning script can be found in this diff continga@71f831d
That diff was previously part of the beforementioned PR, but we decided to do the plg_fields_repeatable stuff in a separate issue and keep it outside of the scope of the plg_fields_subfields PR. That is the reason this issue got created.

So in a first place, we need to decide on the actions that need to be taken in regards to the plg_fields_repeatable plugin when #22446 got merged, and when we have decided and when #22446 is merged, we need to create an additional PR for this.

@ghost
Copy link

ghost commented Mar 3, 2019

@infograf768 can you please comment?

@ghost
Copy link

ghost commented Mar 16, 2019

@infograf768 reminder

@infograf768
Copy link
Member

As it looks that #22446 is not going to go into 3.x and is not changed to be targeted at 4.0, this Issue looks out of scope for now.
The proposed code should imho be added to #22446 if necessary in order to merge both together when proposed and accepted.

@ghost
Copy link

ghost commented Mar 16, 2019

@infograf768 thanks for Comment. So its for @HLeithner or @wilsonge

@HLeithner
Copy link
Member

We can't remove anything in a minor version. At least not following the semver definition.
Beside this, how should this work what happens to layout overrides?

@micker
Copy link

micker commented Mar 17, 2019

I think repetable field is already use in many extension ... Try to start migration now Can be interesting now for less work on j4 migration

@HLeithner
Copy link
Member

Migration seams not to be the problem (script is ready) but layout overrides may have to be changed.

@wilsonge
Copy link
Contributor

It would make sense in this case to merge #22446 so people have a version where they can migrate their fields out of repeatable and into the subform field. But yes the repeatable field cannot be removed until 4.0 - but we can of course mark it as deprecated as soon as that is merged

@wilsonge
Copy link
Contributor

wilsonge commented Mar 18, 2019

AMENDMENT TO ABOVE: Apologies. I missed there was a migration script. If there's a migration script then the correct place to do that migration is in 4.0 and not have this in 3.10! Sorry!

@ghost
Copy link

ghost commented Mar 28, 2019

changed Title cause of above Comment by @wilsonge

@joomla-cms-bot joomla-cms-bot changed the title [3.10] Remove plg_fields_repeatable [4.0] Remove plg_fields_repeatable Mar 28, 2019
@ghost ghost added the J4 Issue label Apr 4, 2019
@wilsonge
Copy link
Contributor

Adding release blocker label as this needs to be solved for 4.0 stable

@jwaisner
Copy link
Member

Increasing priority as per guidelines for beta-blockers.


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

@Quy
Copy link
Contributor

Quy commented Mar 11, 2020

See PR #28319 Draft.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants