RTC Language Change bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar krishnagandhicode
krishnagandhicode
30 Mar 2026

Pull Request resolves #33327

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

This fixes the silent duplicate filed removal behavior for Subform fields

this fix now : on saving a Subform field, check for duplicate selected custom fields in the options list, if found any the duplicates are removed during during save (existing behavior), but now a warning message is shown to the user so the change is not silent and the user is aware of it.

Testing Instructions

Navigate to : Administrator > fields
create a field only give it a title and save and close
now create a new field or type subform (i.e select Subform (subform) in type field) > now go to Fields (at end in General tab) select the same custom field more than once > Save & close
(see the video in Before and after Patch section in this disc. if unclear)

Actual result BEFORE applying this Pull Request

Duplicate subform fields will be removed on save without any warning,
so after save when you check your subform field each custom field appears only once. (but the user is unaware why)

Before.mp4

Expected result AFTER applying this Pull Request

Duplicate subform fields are still cleaned up on save, but users now receive a warning message saying that duplictes were removed and that each field can only be used once

After.mp4

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar krishnagandhicode krishnagandhicode - open - 30 Mar 2026
avatar krishnagandhicode krishnagandhicode - change - 30 Mar 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Mar 2026
Category Administration com_fields Language & Strings
avatar krishnagandhicode krishnagandhicode - change - 30 Mar 2026
Labels Added: Language Change PR-5.4-dev
avatar exlemor exlemor - test_item - 31 Mar 2026 - Tested successfully
avatar exlemor
exlemor - comment - 31 Mar 2026

I have tested this item ✅ successfully on 6055428

I have tested this successfully - thanks @krishnagandhicode


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

avatar SRV-KILLER09 SRV-KILLER09 - test_item - 31 Mar 2026 - Tested successfully
avatar SRV-KILLER09
SRV-KILLER09 - comment - 31 Mar 2026

I have tested this item ✅ successfully on 6055428


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

avatar richard67 richard67 - change - 1 Apr 2026
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 1 Apr 2026

RTC


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

avatar muhme muhme - change - 1 Apr 2026
Labels Added: RTC bug
avatar muhme
muhme - comment - 2 Apr 2026

✅ Final test before merge with JBT

  • Seen silent deletion of duplicate fields in subform field
  • Applied PR with Patch Tester
    • Seen yellow warning 'Duplicate subform fields were removed. Each field can only be used once.'
    • Only one warning is displayed if two of the three duplicate fields are removed
    • No warning if two different fields are used in one subform field and the fields are still exist
    • No warning if two subform fields are using the same field
avatar muhme muhme - change - 2 Apr 2026
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2026-04-02 06:06:15
Closed_By muhme
avatar muhme muhme - close - 2 Apr 2026
avatar muhme muhme - merge - 2 Apr 2026
avatar muhme
muhme - comment - 2 Apr 2026

Thank you very much @krishnagandhicode for your contribution. Thanks to @brianteeman for your support. Thanks to @SRV-KILLER09 and @exlemor for testing.

avatar krishnagandhicode
krishnagandhicode - comment - 2 Apr 2026

Thanks for the merge, and thanks to everyone who helped test.

Add a Comment

Login with GitHub to post a comment