? ? Pending

User tests: Successful: Unsuccessful:

avatar hikashop-nicolas
hikashop-nicolas
12 Nov 2019

Pull Request for Issue #27054 .

Summary of Changes

We add the trigger to the onchange event after clearing the input field when the "clear" button is pressed.

Testing Instructions

install HikaShop Starter
go to Components->HikaShop->Orders
select a date in one of the calendar field on the top right of the listing
the page is refreshed with the calendar field selected date set
click on the "clear" button of that calendar field

Expected result

the value is cleared and the page is refreshed with the calendar field empty

Actual result

the value is cleared and the page is not refreshed

Documentation Changes Required

avatar hikashop-nicolas hikashop-nicolas - open - 12 Nov 2019
avatar hikashop-nicolas hikashop-nicolas - change - 12 Nov 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Nov 2019
Category JavaScript
avatar Quy
Quy - comment - 20 Nov 2019

Edit an article.
Clear Start Publishing.

See in Console.

TypeError: self.inputField.onchange is not a function calendar.js:801:21

avatar hikashop-nicolas hikashop-nicolas - change - 20 Nov 2019
Labels Added: ?
avatar hikashop-nicolas
hikashop-nicolas - comment - 20 Nov 2019

Thank you for your feedback. I've added a check to call onchange only when defined. That will prevent the javascript error when the onchange is not defined, like on the Joomla article edition page.

avatar hikashop-nicolas
hikashop-nicolas - comment - 20 Nov 2019

It's done

avatar Quy Quy - test_item - 20 Nov 2019 - Tested successfully
avatar Quy
Quy - comment - 20 Nov 2019

I have tested this item successfully on 4909026

Also can be tested here: Components > Banners > Tracks > Search Tools


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

avatar viocassel viocassel - test_item - 21 Nov 2019 - Tested successfully
avatar viocassel
viocassel - comment - 21 Nov 2019

I have tested this item successfully on 4909026


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

avatar alikon alikon - change - 21 Nov 2019
Status Pending Ready to Commit
avatar alikon
alikon - comment - 21 Nov 2019

RTC


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

avatar rdeutz rdeutz - change - 24 Nov 2019
Labels Added: ?
avatar HLeithner
HLeithner - comment - 5 Dec 2019

Thanks for the for fix.

avatar HLeithner HLeithner - change - 5 Dec 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-12-05 22:38:20
Closed_By HLeithner
avatar HLeithner HLeithner - close - 5 Dec 2019
avatar HLeithner HLeithner - merge - 5 Dec 2019
avatar Quy
Quy - comment - 9 Dec 2019

@hikashop-nicolas Can you please check the minified file as the calendar is not displaying?

SyntaxError: expected expression, got keyword 'if' calendar.min.js:1:15600

avatar Quy
Quy - comment - 10 Dec 2019

See PR #27233

Add a Comment

Login with GitHub to post a comment