Feeling Lucky
?
?
[#33540] - stop onchange handler being called twice when changing calendar field…
- Fixed in Code Base
- 15 May 2021
- Medium
- Build: staging
- # 33540
- Diff
- JEvents:onchange-twice
User tests: Successful: Unsuccessful:
… value - fixes issue #33538
Pull Request for Issue # .
Summary of Changes
Remove onchange function call from calendar.js - this is no longer needed since we now have a disptachEvent for the change event
Testing Instructions
edit the file administrator/components/com_content/forms/article.xml and find the publish_up field at line 129 and add the attrribute
onchange="alert(0);"
to this field and then edit an article and change the publish_up field (using the date picker)
Actual result BEFORE applying this Pull Request
you will get the alert(0) being called twice
Expected result AFTER applying this Pull Request
alert(0) is called once
Documentation Changes Required
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript |
Fedik
- comment
- 8 May 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33540.
Fedik
- comment
- 8 May 2021
can you please make the same for joomla 4?
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
|
||
richard67
- comment
- 8 May 2021
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33540.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-05-15 10:07:34 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
?
|
||
HLeithner
- comment
- 15 May 2021
thanks
I have tested this item✅ successfully on 7bc0667
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33540.