diff options
author | David Rothstein <drothstein@gmail.com> | 2015-10-12 23:17:31 -0400 |
---|---|---|
committer | David Rothstein <drothstein@gmail.com> | 2015-10-12 23:17:31 -0400 |
commit | f12effc70c835e77bfdea394214df9247533d319 (patch) | |
tree | d8ccf0f126013ba29789d8a1bc5cee854f5987fa /misc/states.js | |
parent | ab86fe4810a2ec33386ec702e16f257feef3f45e (diff) | |
download | brdo-f12effc70c835e77bfdea394214df9247533d319.tar.gz brdo-f12effc70c835e77bfdea394214df9247533d319.tar.bz2 |
Issue #1592688 by jgtrescazes, tstoeckler, eric.chenchao, fietserwin, Heine, stefan.r, Dmitriy.trt, lachezar.valchev, bogdan khrupa: #states can cause the form "required" mark to appear more than once on the same element
Diffstat (limited to 'misc/states.js')
-rw-r--r-- | misc/states.js | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/misc/states.js b/misc/states.js index 6d98da81b..5aac65d20 100644 --- a/misc/states.js +++ b/misc/states.js @@ -493,7 +493,11 @@ $(document).bind('state:disabled', function(e) { $(document).bind('state:required', function(e) { if (e.trigger) { if (e.value) { - $(e.target).closest('.form-item, .form-wrapper').find('label').append('<span class="form-required">*</span>'); + var $label = $(e.target).closest('.form-item, .form-wrapper').find('label'); + // Avoids duplicate required markers on initialization. + if (!$label.find('.form-required').length) { + $label.append('<span class="form-required">*</span>'); + } } else { $(e.target).closest('.form-item, .form-wrapper').find('label .form-required').remove(); |