User tests: Successful: Unsuccessful:
Pull Request for Issue introduced (by me 🥇 ) in PR #40761.
Affected Joomla versions: 4.4.0 and 5.0.0
Quick FIX : Time select broken if calendar form field has weeknumbers
attribute on false
.
Check if weeknumbers is true or false to listen to correct number of childNodes.
Calendar form field with 12h (AM/PM) time format is not affected by the issue, as well as calendar picker with week numbers (so Joomla core not affected).
Set 2 form fields of type calendar like this (with and without week numbers):
Note: default timeformat is 24 and default weeknumbers is true.
<field
name="no_week_numbers"
type="calendar"
label="Test"
description="Test without weeknumbers and time 24h"
translateformat="true"
showtime="true"
weeknumbers="false"
/>
<field
name="week_numbers"
type="calendar"
label="Test"
description="Test with weeknumbers and time 24h"
translateformat="true"
showtime="true"
/>
You can't change Time (hour and/or minutes) with the no_week_numbers
calendar picker.
It's working with the other form field.
Note
EDIT 2025-08-23 PBF
The issue is if weeknumbers
attribute is set to false
you can change the time, click on close, but the time is not changed. Still the previous time display, not the new one.
It's working in both cases.
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
Status | New | ⇒ | Pending |
Category | ⇒ | JavaScript Repository NPM Change |
Labels |
Added:
NPM Resource Changed
PR-4.4-dev
|
Title |
|
Title |
|
Labels |
Added:
bug
PR-5.2-dev
|
Labels |
Removed:
PR-4.4-dev
|
This pull request has been automatically rebased to 5.3-dev.
Title |
|
Labels |
Added:
PR-5.3-dev
Removed: PR-5.2-dev |
I have tested this item 🔴 unsuccessfully on 4c1d10b
Sadly... ;( I tried it twice and failed both times... I could NOT replicate the BEFORE (or AFTER) condition - the fields DID show up from the .xml in my plugin!
BUT without the PR applied, NONE of the 2 date pickers showed the time. (And even after the Download Package was applied with the PR rolled in), NEITHER of the 2 date pickers showed the time ;( ;(
Sorry @cyrezdev
Imo, it's possible that i didn't explain clearly enough the issue. (2 years ago...)
So, before to patch, if you set add attribute "weeknumbers=false", you can see time, select one, but if you change it, click on close, it won't be saved.
This is what this patch is supposed to fix.
Simple example:
A. Before patch, open an existed article, go to created date field, open calendar picker and change the time, then click on close. Result the time is changed.
Then, edit article.xml and add weeknumbers=false to the "created" form field.
Try again A.
There, you may see that the time field is not update. You just can't. (just tested it while chilling in the garden).
B. Apply the patch, test A. with weeknumbers=false
I've update PR with a Note to add better instruction. Hope this helps?
I have tested this item ✅ successfully on 4c1d10b
I have tested this successfully - thank you @cyrezdev for doing Garden style PBF and fixing the Testing Instructions so we can get past the finish line 💜
I have tested this item ✅ successfully on 4c1d10b
works for me!
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
RTC
|
Thanks
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-08-24 11:45:03 |
Closed_By | ⇒ | rdeutz |
This pull request has been automatically rebased to 5.2-dev.