? Pending

User tests: Successful: Unsuccessful:

avatar cheetabhai
cheetabhai
2 Jul 2019

Pull Request for Issue #25387

Summary of Changes

The value of id attribute of input element is checked for non-word characters and if non-word characters are found then replacing with them with underscore

Testing Instructions

Check the value of id attribute of input element in repetable subform after clicking the plus (+) button to add new clone of sub-form

Expected result

The value of id attribute of input element should not contain any non-word character

Actual result

The non-word characters in the value of id attribute of input element are being replaced with underscore

Documentation Changes Required

no change required in documentation

avatar cheetabhai cheetabhai - open - 2 Jul 2019
avatar cheetabhai cheetabhai - change - 2 Jul 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jul 2019
Category JavaScript
avatar cheetabhai cheetabhai - change - 2 Jul 2019
Labels Added: ?
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 Jul 2019
avatar SharkyKZ
SharkyKZ - comment - 2 Jul 2019

Now you need to generate a minified file (media/system/js/subform-repeatable.js).

avatar ankush-maherwal
ankush-maherwal - comment - 2 Jul 2019

updated the related minified js file

avatar manojLondhe
manojLondhe - comment - 2 Jul 2019

@SharkyKZ what tools shall be used for minification?

avatar SharkyKZ
SharkyKZ - comment - 2 Jul 2019

Is manually modified minified file fine in this case?

avatar cheetabhai
cheetabhai - comment - 2 Jul 2019

as of now I have manually modified the minified JS

avatar richard67 richard67 - test_item - 2 Jul 2019 - Tested successfully
avatar richard67
richard67 - comment - 2 Jul 2019

I have tested this item successfully on 9570c0d

Solves both issues #25387 and #25384 .


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

avatar HLeithner HLeithner - close - 2 Jul 2019
avatar HLeithner HLeithner - merge - 2 Jul 2019
avatar HLeithner HLeithner - change - 2 Jul 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-07-02 16:52:40
Closed_By HLeithner
avatar HLeithner
HLeithner - comment - 2 Jul 2019

thx

Add a Comment

Login with GitHub to post a comment