You may want to look at the various modal.php used in multianguage, when modifying an edit.php
for example:
/administrator/components/com_content/views/article/tmpl/modal.php

I meant that any chnage in an edit.php should also be done in the related modal.php

Thanks @infograf768! I must have missed one. I'll check that in a bit.

Thanks @infograf768 . I think I got them all now.

@test ok
The custom fieldsets are displayed in the above mentioned views.

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

@test: The custom fields show in the views. The only thing I don't know how to test is the modals. Any guidance on how to check those?

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

These are used in multilanguage with associations.
Editing an item will display the tab associations where an item can be selected in another language.
Once this is done and saved, that associated item can be edited via the Edit button: it then opens a new window: this is the "modal".

Would you all consider this RTC?

Thanks @phproberto!

This needs to be reverted as at least in com_newsfeeds it is resulting in duplicated tabs and fields

Set to "open" on behalf of @brianteeman by The JTracker Application at issues.joomla.org/joomla-cms/4284

Reverting now

done
df127e1

Good catch @brianteeman. I've updated the PR to correct that.

Newsfeed tabs are no longer being duplicated etc

Added the PBF14 label as this should be easy to test and get to RTC again.

@test not successful

com_categories.category: ok
com_contact.contact: tabs "form" and "display" missing, but tab "metadata" added
com_content.article: ok
com_finder.filter: tab "filter timeline" missing
com_menus.item: tab "associations" duplicated
com_modules.module: ok
com_newsfeeds.newsfeed: ok
com_plugins.plugin: ok
com_tags.tag: tab "metadata" added
com_templates.style: ok

on the above tab "metadata" always contains fields "author" and "robots"

com_weblinks.weblink: tab "metadata" added (fields: "robots" and "contant rights")


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

on com_contact.contact tab "associations" is duplicated, too

Thanks @lunalars! I'm a bit embarrassed for missing those. I'll get this updated in a bit.

Thanks again @lunalars, this should be all set now.

@test OK,

Applied diff, checked views, installed and enabled plugin, and re-checked views.

• com_categories.category
• com_contact.contact
• com_content.article
• com_finder.filter
• com_menus.item
• com_modules.module
• com_newsfeeds.newsfeed
• com_plugins.plugin
• com_tags.tag
• com_templates.style
• com_weblinks.weblink
  
  This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4284.

@test not ok - sorry
using com_patchtester and the test plugin mentioned in the test instructions

com_categories.category: tab "associations" duplicated
com_content.article: tab "associations" duplicated
com_menus.item: tab "required settings" added

things have turned around :-)
a noob question: is it ok to test even if travis doesn't like this?


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

(re: travis yes its ok to test. sometimes the error with travis is
unrelated to the issue)

On 18 October 2014 08:36, lunalars notifications@github.com wrote:

@test https://github.com/test not ok - sorry
using com_patchtester and the test plugin mentioned in the test
instructions

com_categories.category: tab "associations" duplicated
com_content.article: tab "associations" duplicated
com_menus.item: tab "required settings" added

things have turned around :-)
a noob question: is it ok to test even if travis doesn't like this?

This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/4284
http://issues.joomla.org/tracker/joomla-cms/4284.

—
Reply to this email directly or view it on GitHub
#4284 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

@test
Associations tabs are doubled.
$this->ignore_fieldsets = array('item_associations');
is missing for article edit, category edit, newsfeed edit, same for the modal.php files concerned
Also missing for contact modal.php

Also I suggest to add the code with all fieldsets to be ignored at the top of the files instead of in the middle.

Another matter : is this really B/C?
At seing the changes we have to add to each page concerned, I worry about 3pd components.

Removing test results because there are a number of unsuccessful tests and B/C is in question.

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

To address the BC concerns, the intent of this PR is to correct a previous backwards incompatible change that broke documented functionality in both Joomla 2.5 and 3.1+. Further details are provided in the description of this PR.

If there is consensus that this would be accepted, I will update this PR to correct any test failures.

From your description it sounds like we need this
On 28 Nov 2014 00:13, "Matt Thomas" notifications@github.com wrote:

To address the BC concerns, the intent of this PR is to correct a previous
backwards incompatible change that broke documented functionality in both
Joomla 2.5 and 3.1+. Further details are provided in the description of
this PR.

If there is consensus that this would be accepted, I will update this PR
to correct any test failures.

—
Reply to this email directly or view it on GitHub
#4284 (comment).

I agree with @brianteeman that it sounds like we need this since B/C broke. @Bakual @chrisdavenport Can this be accepted?

@betweenbrain Looks like this is out of sync. I'd also like to see the association bug stuff to be fixed before merging.

The JLayouts were introduced in 3.2 - so if there was a b/c effect of these fields being removed I think we would have seen it with plugin reports. However I agree with Matt that without these a lot of the JForm functionality is lost so it's important to me that we reimplement the ability to do that.

Closing as we have now a new version by @pollen8 here: #7126